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

Vamos24 replication #8

Merged
merged 97 commits into from
Dec 7, 2023
Merged

Vamos24 replication #8

merged 97 commits into from
Dec 7, 2023

Conversation

guethilu
Copy link
Owner

@guethilu guethilu commented Dec 7, 2023

No description provided.

guethilu and others added 30 commits July 12, 2022 16:05
Merge VariantSync/DiffDetective into guethilu/DiffDetective
…ncy + remove deprecated assertSemanticConsistency
Behavioural changes:

- `EditClassValidationTask` doesn't add export options to the metadata
  anymore because it doesn't actually export anything.

- Bug fix: The tree preprocessing in `MiningTask` was run twice:
  First for the line graph export and second directly by the `call`
  method.
For example `#if defined()` was accepted.
In general, `HashMap`s are not deterministic because the default
implementation of the hash function might not be deterministic (as in,
for example, openjdk 18.0.2). This matters in this case because it's
incorrect to replace '>' before '>='.
According to C11, constant expressions (used in macro conditions) can
contain all operators available in normal code except assignment,
increment, decrement, function-call, or comma operators. (§ 6.6
paragraph 3)
An example where this now works is `#ifdef ifndef`.
Previously, the macro `#if defined A` was not parsed correctly because
first the space was removed and afterwards it was tried to replace the
`defined` operator.
Note: This changes the abstracted names of macro calls in an backwards
incompatible way.
@guethilu guethilu merged commit 344c063 into main Dec 7, 2023
1 check passed
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 this pull request may close these issues.

4 participants