Skip to content

Conversation

michaelnebel
Copy link
Contributor

@michaelnebel michaelnebel commented Aug 29, 2024

In this PR we introduce a new model generator query for Java and C#. The work is based on #8667.

This should not in any way be considered the final version of these queries.

@michaelnebel michaelnebel force-pushed the java/modelgenfieldbased branch 2 times, most recently from 6b21f84 to ddf3066 Compare August 30, 2024 12:11
@michaelnebel michaelnebel force-pushed the java/modelgenfieldbased branch 2 times, most recently from 2a94828 to 9cd176a Compare August 30, 2024 14:01
@michaelnebel michaelnebel marked this pull request as ready for review September 2, 2024 06:50
@michaelnebel michaelnebel requested review from a team as code owners September 2, 2024 06:50
@michaelnebel michaelnebel added the no-change-note-required This PR does not need a change note label Sep 2, 2024
@michaelnebel michaelnebel force-pushed the java/modelgenfieldbased branch from 9cd176a to 7bf7df2 Compare September 3, 2024 10:58
hvitved
hvitved previously approved these changes Sep 5, 2024
Copy link
Contributor

@hvitved hvitved left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


private module PropagateContentFlow = ContentDataFlow::Global<PropagateContentFlowConfig>;

private string printStoreAccessPath(PropagateContentFlow::AccessPath ap) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Using concat is better (for the same reason that using string builders is better than doing string concatenations). One would then need to define a predicate string getContent(PropagateContentFlow::AccessPath ap, int i).

)
}

private string printReadAccessPath(PropagateContentFlow::AccessPath ap) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same.

@michaelnebel michaelnebel merged commit a5b4622 into github:main Sep 6, 2024
50 checks passed
@michaelnebel michaelnebel deleted the java/modelgenfieldbased branch September 6, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# Java no-change-note-required This PR does not need a change note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants