Re: RFR: 8314321: Remove unused field jdk.internal.util.xml.impl.Attrs.mAttrIdx

2023-08-16 Thread Brian Burkhalter
On Tue, 15 Aug 2023 20:13:18 GMT, Andrey Turbanov wrote: > A field `char mAttrIdx` in the `jdk.internal.util.xml.impl.Attrs` class is > unused and can be removed. Marked as reviewed by bpb (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/15298#pullrequestreview-1580942072

Re: RFR: 8314321: Remove unused field jdk.internal.util.xml.impl.Attrs.mAttrIdx

2023-08-16 Thread Vyom Tewari
On Tue, 15 Aug 2023 20:13:18 GMT, Andrey Turbanov wrote: > A field `char mAttrIdx` in the `jdk.internal.util.xml.impl.Attrs` class is > unused and can be removed. Looks OK to me. - Marked as reviewed by vtewari (Committer). PR Review: https://git.openjdk.org/jdk/pull/15298#pullre

Re: RFR: 8314321: Remove unused field jdk.internal.util.xml.impl.Attrs.mAttrIdx

2023-08-16 Thread Alan Bateman
On Tue, 15 Aug 2023 20:13:18 GMT, Andrey Turbanov wrote: > A field `char mAttrIdx` in the `jdk.internal.util.xml.impl.Attrs` class is > unused and can be removed. Marked as reviewed by alanb (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/15298#pullrequestreview-15799266

RFR: 8314321: Remove unused field jdk.internal.util.xml.impl.Attrs.mAttrIdx

2023-08-15 Thread Andrey Turbanov
A field `char mAttrIdx` in the `jdk.internal.util.xml.impl.Attrs` class is unused and can be removed. - Commit messages: - [PATCH] Remove unused field jdk.internal.util.xml.impl.Attrs.mAttrIdx Changes: https://git.openjdk.org/jdk/pull/15298/files Webrev: https://webrevs.openjdk.or