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

Support JDK 23 #1441

Merged
merged 5 commits into from
Sep 19, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion .github/workflows/continuous-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
- os: ubuntu-latest
java: 21
- os: ubuntu-latest
java: 22
liblit marked this conversation as resolved.
Show resolved Hide resolved
java: 23
fail-fast: false
runs-on: ${{ matrix.os }}
steps:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,12 @@
* GetMethodContextSelector} should be placed in be placed in front of {@link
* JavaLangClassContextInterpreter} and {@link JavaLangClassContextSelector} .
*
* <p>TODO Do the same for {@link Class#getField(String)} and {@link
liblit marked this conversation as resolved.
Show resolved Hide resolved
* Class#getDeclaredField(String)}.
*
* @author Michael Heilmann
* @see com.ibm.wala.analysis.reflection.GetMethodContextInterpreter
* @see com.ibm.wala.analysis.reflection.GetMethodContextSelector TODO Do the same for {@link
* Class#getField(String)} and {@link Class#getDeclaredField(String)}.
* @see com.ibm.wala.analysis.reflection.GetMethodContextSelector
*/
public class GetMethodContext implements Context {
/** The type abstraction. */
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,6 @@
* <li>it stores summary edges at each callee instead of at each call site.
* </ul>
*
* <p>
liblit marked this conversation as resolved.
Show resolved Hide resolved
*
* @param <T> type of node in the supergraph
* @param <P> type of a procedure (like a box in an RSM)
* @param <F> type of factoids propagated when solving this problem
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -226,10 +226,6 @@ protected boolean addTarget(int pc, CGNode tNode) {
}
}

/**
liblit marked this conversation as resolved.
Show resolved Hide resolved
* @see
* com.ibm.wala.ipa.callgraph.impl.BasicCallGraph.NodeImpl#removeNodeAndEdges(com.ibm.wala.ipa.callgraph.CGNode)
*/
public void removeTarget(CGNode target) {
allTargets.remove(getCallGraph().getNumber(target));
for (IntIterator it = targets.safeIterateIndices(); it.hasNext(); ) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,8 @@ private void parse() throws InvalidClassFileException {
if (magic != MAGIC) {
throw new InvalidClassFileException(offset, "bad magic number: " + magic);
}
// Support class files up through JDK 22 (version 66)
if (majorVersion < 45 || majorVersion > 66) {
// Support class files up through JDK 23 (version 67)
if (majorVersion < 45 || majorVersion > 67) {
throw new InvalidClassFileException(
offset, "unknown class file version: " + majorVersion + '.' + minorVersion);
}
Expand Down
Loading