Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
爽朗的琪琪
jadx
提交
3782aa7d
J
jadx
项目概览
爽朗的琪琪
/
jadx
与 Fork 源项目一致
从无法访问的项目Fork
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
J
jadx
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
前往新版Gitcode,体验更适合开发者的 AI 搜索 >>
提交
3782aa7d
编写于
4月 07, 2015
作者:
S
Skylot
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
core: fix wildcard type in iterable loop
上级
d5740c1b
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
65 addition
and
7 deletion
+65
-7
jadx-core/src/main/java/jadx/core/dex/visitors/regions/LoopRegionVisitor.java
...ava/jadx/core/dex/visitors/regions/LoopRegionVisitor.java
+18
-7
jadx-core/src/test/java/jadx/tests/integration/generics/TestGenerics6.java
...t/java/jadx/tests/integration/generics/TestGenerics6.java
+47
-0
未找到文件。
jadx-core/src/main/java/jadx/core/dex/visitors/regions/LoopRegionVisitor.java
浏览文件 @
3782aa7d
...
...
@@ -16,7 +16,6 @@ import jadx.core.dex.instructions.args.LiteralArg;
import
jadx.core.dex.instructions.args.RegisterArg
;
import
jadx.core.dex.instructions.args.SSAVar
;
import
jadx.core.dex.nodes.BlockNode
;
import
jadx.core.dex.nodes.DexNode
;
import
jadx.core.dex.nodes.IBlock
;
import
jadx.core.dex.nodes.IRegion
;
import
jadx.core.dex.nodes.InsnNode
;
...
...
@@ -232,14 +231,23 @@ public class LoopRegionVisitor extends AbstractVisitor implements IRegionVisitor
}
List
<
InsnNode
>
toSkip
=
new
LinkedList
<
InsnNode
>();
RegisterArg
iterVar
=
nextCall
.
getResult
();
if
(
iterVar
==
null
)
{
return
false
;
}
if
(
nextCall
.
contains
(
AFlag
.
WRAPPED
))
{
InsnArg
wrapArg
=
BlockUtils
.
searchWrappedInsnParent
(
mth
,
nextCall
);
if
(
wrapArg
!=
null
&&
wrapArg
.
getParentInsn
()
!=
null
)
{
InsnNode
parentInsn
=
wrapArg
.
getParentInsn
();
if
(
parentInsn
.
getType
()
!=
InsnType
.
CHECK_CAST
)
{
if
(!
fixIterableType
(
mth
,
iterableArg
,
iterVar
))
{
return
false
;
}
parentInsn
.
replaceArg
(
wrapArg
,
iterVar
);
}
else
{
iterVar
=
parentInsn
.
getResult
();
if
(
iterVar
==
null
||
!
fixIterableType
(
mth
,
iterableArg
,
iterVar
))
{
return
false
;
}
InsnArg
castArg
=
BlockUtils
.
searchWrappedInsnParent
(
mth
,
parentInsn
);
if
(
castArg
!=
null
&&
castArg
.
getParentInsn
()
!=
null
)
{
castArg
.
getParentInsn
().
replaceArg
(
castArg
,
iterVar
);
...
...
@@ -255,9 +263,6 @@ public class LoopRegionVisitor extends AbstractVisitor implements IRegionVisitor
}
else
{
toSkip
.
add
(
nextCall
);
}
if
(
iterVar
==
null
||
!
fixIterableType
(
mth
.
dex
(),
iterableArg
,
iterVar
))
{
return
false
;
}
assignInsn
.
add
(
AFlag
.
SKIP
);
for
(
InsnNode
insnNode
:
toSkip
)
{
...
...
@@ -267,7 +272,7 @@ public class LoopRegionVisitor extends AbstractVisitor implements IRegionVisitor
return
true
;
}
private
static
boolean
fixIterableType
(
DexNode
dex
,
InsnArg
iterableArg
,
RegisterArg
iterVar
)
{
private
static
boolean
fixIterableType
(
MethodNode
mth
,
InsnArg
iterableArg
,
RegisterArg
iterVar
)
{
ArgType
iterableType
=
iterableArg
.
getType
();
ArgType
varType
=
iterVar
.
getType
();
if
(
iterableType
.
isGeneric
())
{
...
...
@@ -283,10 +288,16 @@ public class LoopRegionVisitor extends AbstractVisitor implements IRegionVisitor
iterVar
.
setType
(
gType
);
return
true
;
}
if
(
ArgType
.
isInstanceOf
(
dex
,
gType
,
varType
))
{
if
(
ArgType
.
isInstanceOf
(
mth
.
dex
(),
gType
,
varType
))
{
return
true
;
}
ArgType
wildcardType
=
gType
.
getWildcardType
();
if
(
wildcardType
!=
null
&&
gType
.
getWildcardBounds
()
==
1
&&
ArgType
.
isInstanceOf
(
mth
.
dex
(),
wildcardType
,
varType
))
{
return
true
;
}
LOG
.
warn
(
"Generic type differs:
{} and {}"
,
gType
,
varType
);
LOG
.
warn
(
"Generic type differs:
'{}' and '{}' in {}"
,
gType
,
varType
,
mth
);
return
false
;
}
if
(!
iterableArg
.
isRegister
())
{
...
...
jadx-core/src/test/java/jadx/tests/integration/generics/TestGenerics6.java
0 → 100644
浏览文件 @
3782aa7d
package
jadx.tests.integration.generics
;
import
jadx.core.dex.nodes.ClassNode
;
import
jadx.tests.api.IntegrationTest
;
import
java.util.Collection
;
import
org.junit.Test
;
import
static
jadx
.
tests
.
api
.
utils
.
JadxMatchers
.
containsOne
;
import
static
org
.
junit
.
Assert
.
assertThat
;
public
class
TestGenerics6
extends
IntegrationTest
{
public
static
class
TestCls
{
public
void
test1
(
Collection
<?
extends
A
>
as
)
{
for
(
A
a
:
as
)
{
a
.
f
();
}
}
public
void
test2
(
Collection
<?
extends
A
>
is
)
{
for
(
I
i
:
is
)
{
i
.
f
();
}
}
private
interface
I
{
void
f
();
}
private
class
A
implements
I
{
public
void
f
()
{
}
}
}
@Test
public
void
test
()
{
ClassNode
cls
=
getClassNode
(
TestCls
.
class
);
String
code
=
cls
.
getCode
().
toString
();
assertThat
(
code
,
containsOne
(
"for (A a : as) {"
));
// TODO: fix iterable arg type (unexpected cast to A in bytecode)
// assertThat(code, containsOne("for (I i : is) {"));
}
}
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录