diff options
| author | tiansiming <tiansiming@xiaomi.com> | 2017-10-13 10:57:25 +0800 |
|---|---|---|
| committer | tiansiming <tiansiming@xiaomi.com> | 2017-10-15 15:08:35 +0800 |
| commit | 5330b5a78c7380a1b7c7b717aed2d240cd97530e (patch) | |
| tree | 7990faf0d880dbb5bd23becffa876588014ce70d /tools/aapt2/java/JavaClassGenerator.cpp | |
| parent | d0983285857f9107b9306a57f3b500672d76c7d9 (diff) | |
Remove "result2" in AccountManagerService
I think result2 here is to filter out other value returned from getAccountRemovalAllowed.
As the annotation shown in getAccountRemovalAllowed in AbstractAccountAuthenticator,
the result will only contain KEY_INTENT or KEY_BOOLEAN_RESULT or
KEY_ERROR_CODE and KEY_ERROR_MESSAGE. KEY_INTENT has been judged at the beginning of onResult,
KEY_ERROR_CODE and KEY_ERROR_MESSAGE seem extraneous when back to AccountManager,
so i believe that it's ok to use result directly here other than get one more Bundle object.
(tip: if result2 is nessesary, its name doesn't looks formal)
Bug:N/A
Test:N/A
Change-Id: I4e352baa1e74ac680398c328db0c4bc873ba528b
Signed-off-by: tiansiming <tiansiming@xiaomi.com>
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator.cpp')
0 files changed, 0 insertions, 0 deletions