diff options
| author | 2020-06-30 11:41:23 +0900 | |
|---|---|---|
| committer | 2020-06-30 18:44:51 +0900 | |
| commit | 24d9cad56383fdc6c2c61526c44d2ffd726b8e7b (patch) | |
| tree | 3918643a2507097c0c1444be601516369bdfa8fe /tools/warn/java_warn_patterns.py | |
| parent | 0b4fccb66dc33e6cc28c0e638ee834fbf8f53c50 (diff) | |
Handle the case when non-optional props have the same value
foo=true
foo=true
foo?=false
Consider the above case: Then the duplication of foo is allowed because
they have the same value (true). However, there was a bug that the
optional assirgnment foo?=false is left unmodified.
This fixes the bug by commenting such optional assignments.
Exempt-From-Owner-Approval: fixes a broken build
Bug: 117892318
Bug: 158735147
Test: atest test_post_process_props
Test: m out/target/product/vsoc_x86/vendor/build.prop for cf_x86_auto
Exempt-From-Owner-Approval: cherry-pick from master
Merged-In: Iba9b61d9779d93e86d9bead2286f945f8d51ab1d
(cherry picked from commit 9a32636759822bfb04071c9e28b101e9714ce305)
Change-Id: Iba9b61d9779d93e86d9bead2286f945f8d51ab1d
Diffstat (limited to 'tools/warn/java_warn_patterns.py')
0 files changed, 0 insertions, 0 deletions