-
-
Notifications
You must be signed in to change notification settings - Fork 65
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
Apk to jimple #888
base: develop
Are you sure you want to change the base?
Apk to jimple #888
Conversation
# Conflicts: # sootup.core/src/main/java/sootup/core/inputlocation/FileType.java
# Conflicts: # sootup.java.bytecode/src/main/java/sootup/java/bytecode/inputlocation/PathBasedAnalysisInputLocation.java
# Conflicts: # sootup.core/src/main/java/sootup/core/jimple/basic/LocalGenerator.java # sootup.core/src/main/java/sootup/core/model/HasPosition.java # sootup.examples/src/test/java/sootup/examples/bodyInterceptor/BodyInterceptor.java # sootup.java.bytecode/src/test/java/sootup/java/bytecode/interceptors/CastAndReturnInlinerTest.java # sootup.java.bytecode/src/test/java/sootup/java/bytecode/interceptors/LocalSplitterTest.java # sootup.java.bytecode/src/test/java/sootup/java/bytecode/interceptors/typeresolving/AugEvalFunctionTest.java # sootup.java.bytecode/src/test/java/sootup/java/bytecode/interceptors/typeresolving/TypeResolverTest.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/JimpleSemanticsChecker.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/LocalSplitter.java
# Conflicts: # sootup.java.bytecode/src/test/java/sootup/java/bytecode/interceptors/typeresolving/BytecodeHierarchyTest.java # sootup.java.bytecode/src/test/java/sootup/java/bytecode/interceptors/typeresolving/TypeAssignerTest.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/AugEvalFunction.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/BytecodeHierarchy.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/PrimitiveHierarchy.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/TypePromotionVisitor.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/TypeResolver.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/types/TopType.java
# Conflicts: # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/AugEvalFunction.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/BytecodeHierarchy.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/PrimitiveHierarchy.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/TypePromotionVisitor.java # sootup.java.core/src/main/java/sootup/java/core/interceptors/typeresolving/types/TopType.java
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #888 +/- ##
=============================================
- Coverage 70.18% 67.58% -2.60%
- Complexity 4129 4131 +2
=============================================
Files 315 320 +5
Lines 15588 16198 +610
Branches 2672 2875 +203
=============================================
+ Hits 10940 10948 +8
- Misses 3761 4356 +595
- Partials 887 894 +7 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for your big contribution :) I have a little feedback for you..
@Nullable | ||
@Override | ||
public SourceType getSourceType() { | ||
return SourceType.Application; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in theory we have it configurable everywhere.. but I see no usecase to set it to Library - so i guess we can keep it that way.
import sootup.core.model.Body; | ||
|
||
/* Flow -insensitive def-use analysis */ | ||
public class DexDefUseAnalysis { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it dex specific or would it make sense to move it to the analysis module?
sootup.java.core/src/main/java/sootup/java/core/interceptors/Dex/DexNumberTranformer.java
Outdated
Show resolved
Hide resolved
sootup.java.core/src/main/java/sootup/java/core/interceptors/Dex/DexTransformer.java
Outdated
Show resolved
Hide resolved
if (baseDef instanceof JAssignStmt) { | ||
JAssignStmt stmt = (JAssignStmt) baseDef; | ||
Value r = stmt.getRightOp(); | ||
if (r instanceof JFieldRef) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ValueVisitor would make sense here to replace the ifelse tree
1. Removed Tagged Instruction 2. Changed RTE to ISException 3. Resolved some comments
No description provided.