Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Diff mode output is empty on "large" files #264

Closed
gOATiful opened this issue Apr 30, 2024 · 1 comment · Fixed by #265
Closed

Diff mode output is empty on "large" files #264

gOATiful opened this issue Apr 30, 2024 · 1 comment · Fixed by #265

Comments

@gOATiful
Copy link

gOATiful commented Apr 30, 2024

Running the command:
java -classpath ./coming.jar fr.inria.coming.main.ComingMain -input filespair -location Util_before.java:Util_after.java -mode diff -output out

Returns without and error, but the fields in the created JSON are empty:

{
  "frequency": [],
  "frequencyParent": [],
  "probability": [],
  "probabilityParent": []
}

Which is not expected, since there are changes in L147 and L161.

However, removing unchanged lines from both files to a file length of about 700 lines, produces out again:

{
  "frequency": [
    {
      "c": "Class",
      "f": "1"
    }
  ],
  "frequencyParent": [
    {
      "c": "update-node_Class_Package",
      "f": "1"
    }
  ],
  "probability": [
    {
      "c": "Class",
      "f": "1.0"
    }
  ],
  "probabilityParent": [
    {
      "c": "update-node_Class_Package",
      "f": "1.0"
    }
  ]
}

This output is also not expected, since the change involves the removal of boolean literals in a function invocation.

Please find the used files attached.
used_files.zip

@monperrus
Copy link
Contributor

thanks for the bug report.

Fix WIP in progress in #265

Was partially due to a bug in gumtree-spoon-ast-diff fixed in SpoonLabs/gumtree-spoon-ast-diff#313

@monperrus monperrus mentioned this issue May 1, 2024
monperrus added a commit that referenced this issue May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants