On 2020-03-05 01:11, Alexandre (Shura) Iline wrote:
Hi.
In addition to the code change, it also makes sense to update documentation.
Definitely. Thanks for doing that!
Can you please take another look? Makefile changes as the same as in webrev.00,
this version only have changes in doc/testing.(md|html) files.
New web rev: http://cr.openjdk.java.net/~shurailine/JDK-8240241/webrev.01/
<http://cr.openjdk.java.net/~shurailine/JDK-8240241/webrev.01/>
LGTM.
/Magnus
Shura
On Mar 3, 2020, at 12:56 PM, Erik Joelsson <[email protected]> wrote:
Looks good.
/Erik
On 2020-03-03 12:45, Alexandre (Shura) Iline wrote:
Hi.
Can you take a look on this change introducing make support for JCov
DiffCoverage command.
DiffCoverage allows to generate a textual report showing coverage for only
changed source code,
This is an example of how the functionality can be used:
————————————————————————————————————————————————————————————————————
$ make jcov-test TEST=open/test/langtools/jdk/javadoc/doclet
TEST_OPTS_JCOV_DIFF_CHANGESET=58193
$ cat build/linux-x64/test-results/jcov-output/diff_coverage_report
jdk/javadoc/internal/doclets/formats/html/markup/HtmlStyle.java: HtmlStyle.void
<clinit>()
+ 106 | notes,
jdk/javadoc/internal/doclets/formats/html/SerializedFormWriterImpl.java:
SerializedFormWriterImpl.jdk.javadoc.internal.doclets.toolkit.Content
getSerialUIDInfoHeader()
+ 187 | return new HtmlTree(HtmlTag.DL).setStyle(HtmlStyle.nameValue);
jdk/javadoc/internal/doclets/formats/html/HtmlDocletWriter.java:
HtmlDocletWriter.void
<init>(jdk.javadoc.internal.doclets.formats.html.HtmlConfiguration,jdk.javadoc.internal.d
oclets.toolkit.util.DocPath)
345 | * @param htmlTree the documentation tree to which the tags will
be added
347 | protected void addTagsInfo(Element e, Content htmlTree) {
+ 351 | Content dl = new
HtmlTree(HtmlTag.DL).setStyle(HtmlStyle.notes);
+ 360 | htmlTree.add(dl);
jdk/javadoc/internal/doclets/formats/html/HtmlSerialFieldWriter.java:
HtmlSerialFieldWriter.void
addMemberTags(javax.lang.model.element.VariableElement,jdk.javadoc.internal.doc
lets.toolkit.Content)
+ 203 | Content dlTags = new
HtmlTree(HtmlTag.DL).setStyle(HtmlStyle.notes);
jdk/javadoc/internal/doclets/formats/html/ClassWriterImpl.java:
ClassWriterImpl.void addParamInfo(jdk.javadoc.internal.doclets.toolkit.Content)
+ 354 |
classInfoTree.add(HtmlTree.DL(paramInfo).setStyle(HtmlStyle.notes));
jdk/javadoc/internal/doclets/formats/html/HtmlSerialMethodWriter.java:
HtmlSerialMethodWriter.void
addMemberTags(javax.lang.model.element.ExecutableElement,jdk.javadoc.internal
.doclets.toolkit.Content)
+ 162 | Content dlTags = new
HtmlTree(HtmlTag.DL).setStyle(HtmlStyle.notes);
...
lines: 428 new; 40 covered; 4 not covered; 43 not code; 341 no information
————————————————————————————————————————————————————————————————————
Webrev: http://cr.openjdk.java.net/~shurailine/JDK-8240241/webrev.00/
Shura.
