RFR 7902143: Add an ANC filter that takes method list as input
Alexey Fedorchenko
alexey.fedorchenko at oracle.com
Thu Apr 5 18:36:22 UTC 2018
Hello!
Thanks for the patch! I will review the content of the changes.
Please, look at these items:
- the changes for the build.xml looks like a diff to your local copy and not to the latest repo state:
http://hg.openjdk.java.net/code-tools/jcov/file/391790f13cfd/build/build.xml#l165 <http://hg.openjdk.java.net/code-tools/jcov/file/391790f13cfd/build/build.xml#l165>
(The target was set to 1.7 long time ago)
- please, send a patch not a diff (I will be able to push it from your name)
- do you plan to provide any execution mechanism for the /test/unit ?
Thank you.
—Alexey
> On Apr 4, 2018, at 3:44 PM, Alexandre (Shura) Iline <alexandre.iline at oracle.com> wrote:
>
>
>
>> On Apr 4, 2018, at 1:25 PM, Alexandre (Shura) Iline <alexandre.iline at oracle.com> wrote:
>>
>> Hi.
>>
>> Please take a look on this new ANC filter:
>> http://cr.openjdk.java.net/~shurailine/7902143/webrev.00/
>>
>> Please notice that beyond actually adding the functionality I am suggesting to
>> 1. allow to use Java 7 features
>> 2. change RepGen behavior in regards to errors in ANC filters. It should not just print a message - it should fail.
>>
>> Shura
>
More information about the jcov-dev
mailing list