RFR: 8266666: Implementation for snippets [v3]

Pavel Rappo prappo at openjdk.java.net
Tue Jul 27 16:01:32 UTC 2021


On Mon, 26 Jul 2021 17:05:45 GMT, Hannes Wallnöfer <hannesw at openjdk.org> wrote:

>> Pavel Rappo has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Improve comments
>>   
>>   This commit adds trivial commentary to some of the Java declarations. This commit also adds the "This is NOT part of any supported API..." disclaimer.
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/taglets/snippet/parser/Parser.java line 275:
> 
>> 273:                         throw new ParseException("Unexpected attributes", t.lineSourceOffset + t.markupLineOffset + t.nameLineOffset);
>> 274:                     }
>> 275:                     actions.add(new Bookmark(region.value(), text.subText(t.start(), t.end() - 1)));
> 
> This method is a bit hard on the eyes and runs quite wide. Maybe add a few more line breaks on those very long lines?

So just to be clear, Hannes. Do you suggest to structure the rules like this?


pre.snippet {
    background-color: #e8e8e8;
    padding: 10px;
    margin: 12px 0;
    overflow: auto;
    white-space: pre;
}

pre.snippet .italic {
    font-style: italic;
}

pre.snippet .bold {
    font-weight: bold;
}

pre.snippet .highlighted {
    background-color: sandybrown;
    border-radius: 10%;
}

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

PR: https://git.openjdk.java.net/jdk/pull/4795


More information about the compiler-dev mailing list