RFR: 8215788: Clarify JarInputStream Manifest access [v2]
Lance Andersen
lancea at openjdk.org
Wed Sep 14 01:42:21 UTC 2022
On Wed, 14 Sep 2022 00:39:37 GMT, Weijun Wang <weijun at openjdk.org> wrote:
> On lines 36 and 37, there are two "Manifest". The first uses `linkplain` so it's using normal font style, the 2nd uses `code` so it's fixed-width. I would like them to be the same. In fact, I would not use `linkplain` at all. I only use it when the text is not a Java identifier. For example, `{@linkplain SecurityManager the Security Manager}`. However in this PR, for all the places where you use it, the text is either a class name or a method name. I would just use `link`.
Ok, thank you switched to use `link` from `linkplain` per your suggestion
> Not sure. Maybe just `is read` is OK.
Updated as "...will be thrown when the entry is read."
-------------
PR: https://git.openjdk.org/jdk/pull/10045
More information about the security-dev
mailing list