lint:TypographyQuotes false negatives: more than one escaped apostrophe are ignored [293397291]

Fixed

Bug

Status Update

No update yet.

Description

pa...@gmail.com

created issue #1

Jul 27, 2023 02:05PM

DESCRIBE THE ISSUE IN DETAIL: as title.

Related:

https://issuetracker.google.com/issues/269323652

STEPS TO REPRODUCE:
1.

https://github.com/TWiStErRob/repros/tree/main/lint/TypographyQuotes-misses-multiples
2. `gradlew lint`
3. Observe `build\reports\lint-results-debug.html`

Expected: many findings, for each \'
Actual: only one finding

------------------

Studio Build: Giraffe
Version of Gradle Plugin: 8.1.0
Version of Gradle: 8.2.1
Version of Java: 17
OS: Windows

Comments

to...@google.com <to...@google.com> Jul 27, 2023 06:04PM

Assigned to an...@google.com.

ya...@google.com <ya...@google.com> Aug 16, 2023 07:26PM

Status: New

ph...@google.com <ph...@google.com> Sep 19, 2023 09:39PM

Assigned to ph...@google.com.

ph...@google.com <ph...@google.com> #2Sep 22, 2023 06:06PM

Marked as fixed.

Are there any updates here?

an...@google.com <an...@google.com> #3Sep 28, 2023 09:05PM

I looked at it and I'm planning to remove the deduplicating lookup the callstack refers to. It should be no longer needed. How was the 80ms in comment#1 calculated? If it's straightforward I'll calculate it again before my change goes in.

Issue 293397291