Skip to content

Commit baae8d0

Browse files
committed
Shared: Address model generator review comments.
1 parent 0b39c5b commit baae8d0

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

shared/mad/codeql/mad/modelgenerator/internal/ModelGeneratorImpl.qll

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -673,23 +673,21 @@ module MakeModelGenerator<
673673
}
674674

675675
private string getReversedHash(PropagateContentFlow::AccessPath ap) {
676-
result = concat(int i | | getHashAtIndex(ap, i), "" order by i desc)
676+
result = strictconcat(int i | | getHashAtIndex(ap, i), "." order by i desc)
677677
}
678678

679679
private string getHash(PropagateContentFlow::AccessPath ap) {
680-
result = concat(int i | | getHashAtIndex(ap, i), "" order by i)
680+
result = strictconcat(int i | | getHashAtIndex(ap, i), "." order by i)
681681
}
682682

683683
/**
684-
* Gets all access paths that contains the synthetic fields
684+
* Gets all access paths that contain the synthetic fields
685685
* from `ap` in reverse order (if `ap` contains at least one synthetic field).
686686
* These are the possible candidates for synthetic path continuations.
687687
*/
688688
private PropagateContentFlow::AccessPath getSyntheticPathCandidate(
689689
PropagateContentFlow::AccessPath ap
690690
) {
691-
hasSyntheticContent(ap) and
692-
hasSyntheticContent(result) and
693691
getHash(ap) = getReversedHash(result)
694692
}
695693

0 commit comments

Comments
 (0)