Skip to content

支持minSdkVersion=14 - #83

Merged
shifujun merged 2 commits into
Tencent:devfrom
PhantomQi:pr_20190805_1320_2
Aug 21, 2019
Merged

支持minSdkVersion=14#83
shifujun merged 2 commits into
Tencent:devfrom
PhantomQi:pr_20190805_1320_2

Conversation

@PhantomQi

Copy link
Copy Markdown
Contributor

No description provided.

@PhantomQi

Copy link
Copy Markdown
Contributor Author

@shifujun 抽空看下是否OK了?

@shifujun

Copy link
Copy Markdown
Collaborator

@PhantomQi 前面我问了一个问题你一直没回复,我以为你在忙呢。

@PhantomQi

Copy link
Copy Markdown
Contributor Author

@PhantomQi 前面我问了一个问题你一直没回复,我以为你在忙呢。

我以为我回复的“done”你能收到通知

@shifujun

Copy link
Copy Markdown
Collaborator

这个pr,我说的是
image

这个问题你没回复。

@PhantomQi

Copy link
Copy Markdown
Contributor Author

这个pr,我说的是
image

这个问题你没回复。

我看下

@PhantomQi

Copy link
Copy Markdown
Contributor Author

这个pr,我说的是
image

这个问题你没回复。

Added in API level 15
TransactionTooLargeException

在14上确实没这个类

@shifujun

Copy link
Copy Markdown
Collaborator

API 14 没有TransactionTooLargeException这个类,那它也不会抛出这个异常啊。所以不需要专门在14上catch它,也就不需要通过字符串匹配它。

@PhantomQi

Copy link
Copy Markdown
Contributor Author

API 14 没有TransactionTooLargeException这个类,那它也不会抛出这个异常啊。所以不需要专门在14上catch它,也就不需要通过字符串匹配它。

  1. 14没有TransactionTooLargeException,那么就不能写他的catch,之前在线上遇到过类似问题,写了一个低api不存在的异常的catch,反而导致了异常
  2. 原逻辑对TransactionTooLargeException做了特殊处理,既然不能单独写catch,且TransactionTooLargeException继承自RemoteException,就只能在RemoteException的catch中还原其特殊处理
    看起来比较别扭,但这是迫不得已的兼容

@shifujun shifujun left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

看起来没问题了

@shifujun
shifujun changed the base branch from master to dev August 21, 2019 03:36
@shifujun
shifujun merged commit 4e6ffd8 into Tencent:dev Aug 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants