8000 Java: Replace MethodAccess, LValue, RValue with more intuitive names. Introduce NewClassExpr. by smowton · Pull Request #14575 · github/codeql · GitHub
[go: up one dir, main page]
Skip to content

Java: Replace MethodAccess, LValue, RValue with more intuitive names. Introduce NewClassExpr. #14575

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.

Sign up for GitHub

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

Merged
merged 16 commits into from
Oct 24, 2023
Merged

Java: Replace MethodAccess, LValue, RValue with more intuitive names. Introduce NewClassExpr. #14575

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
a10731c
Java: introduce more-intuitive names for ClassInstanceExpr, L/RValue …
smowton Oct 24, 2023
f552a15
Mass-rename MethodAccess -> MethodCall
smowton Oct 24, 2023
59a49ee
Add aliases for public, importable renamed classes and predicates.
smowton Oct 24, 2023
ac38d4c
Mass rename L/RValue -> VarWrite/Read
smowton Oct 24, 2023
09e83d1
Fix isEnclosingMethodAccess wrapper
smowton Oct 24, 2023
e8c9708
Autoformat
smowton Oct 24, 2023
3627eb2
Add missing qldoc
smowton Oct 24, 2023
efb63aa
Add change note
smowton Oct 24, 2023
e3edea2
Apply simple suggestions from code review
smowton Oct 24, 2023
ede1758
Amend NewClassExpr description
smowton Oct 24, 2023
011666b
Fix description and improve predicate name of VarWrite.
smowton Oct 24, 2023
06238dd
Improve reflective class names
smowton Oct 24, 2023
b849a66
Update test expectations
smowton Oct 24, 2023
4205f1b
Temporarily un-deprecate MethodAccess to decouple from private tests
smowton Oct 24, 2023
92d3d9d
Update integration test expectations
smowton Oct 24, 2023
30610c9
Temporarily de-deprecate SuperMethodAccess to accommodate private tests
smowton Oct 24, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add change note
  • Loading branch information
@smowton
smowton authored Oct 24, 2023
commit efb63aada3b4ebb123621b0308b11b07546ef485
5 changes: 5 additions & 0 deletions java/ql/lib/change-notes/2023-10-24-java-renames.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
category: minorAnalysis
---
* Java classes `MethodAccess`, `LValue` and `RValue` were renamed to `MethodCall`, `VarWrite` and `VarRead` respectively, along with related predicates and class names. The old names remain usable for the time being but are deprecated and should be replaced.
* New class `NewClassExpr` was added to represent specifically an explicit `new ClassName(...)` invocation, in contrast to `ClassInstanceExpr` which also includes expressions that implicitly instantiate classes, such as defining a lambda or taking a method reference.
0