RFR JDK-8240241: Add support for JCov DiffCoverage to make files
    Alexandre (Shura) Iline 
    alexandre.iline at oracle.com
       
    Thu Mar  5 00:11:00 UTC 2020
    
    
  
Hi.
In addition to the code change, it also makes sense to update documentation.
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/>
Shura
> On Mar 3, 2020, at 12:56 PM, Erik Joelsson <erik.joelsson at oracle.com> 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.
>> 
    
    
More information about the build-dev
mailing list