On Mon, 7 Apr 2025 08:45:13 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:
> Allow the user to set "--with-jcov-modules=java.desktop", instead of > "--with-jcov-filters=--include-modules java.desktop", since this is one of > the common use cases for the filtering. Marked as reviewed by erikj (Reviewer). make/Coverage.gmk line 42: > 40: JCOV_TEMP := $(JCOV_SUPPORT_DIR)/temp > 41: > 42: ifneq ($(JCOV_MODULES), ) Is this redundant conditional done on purpose? ------------- PR Review: https://git.openjdk.org/jdk/pull/24481#pullrequestreview-2751203268 PR Review Comment: https://git.openjdk.org/jdk/pull/24481#discussion_r2033908828