8339336: Fix build system whitespace to adhere to coding conventions

Reviewed-by: erikj

make/Bundles.gmk

@@ -284,7 +284,7 @@ ifneq ($(filter product-bundles% legacy-bundles, $(MAKECMDGOALS)), )
   ifeq ($(MACOSX_CODESIGN_MODE), hardened)
     # Macosx release build and code signing available.
 
-    ################################################################################
+    ############################################################################
     # JDK bundle
     $(eval $(call SetupCopyFiles, CREATE_JDK_BUNDLE_DIR_SIGNED, \
         SRC := $(JDK_IMAGE_DIR), \
@@ -313,7 +313,7 @@ ifneq ($(filter product-bundles% legacy-bundles, $(MAKECMDGOALS)), )
 
     PRODUCT_TARGETS += $(BUILD_JDK_BUNDLE)
 
-    ################################################################################
+    ############################################################################
     # JRE bundle
     $(eval $(call SetupCopyFiles, CREATE_JRE_BUNDLE_DIR_SIGNED, \
         SRC := $(JRE_IMAGE_DIR), \

make/CompileToolsJdk.gmk

@@ -52,8 +52,7 @@ $(eval $(call SetupJavaCompilation, BUILD_TOOLS_JDK, \
         build/tools/deps \
         build/tools/docs \
         build/tools/jigsaw \
-        build/tools/depend \
-        , \
+        build/tools/depend, \
     BIN := $(BUILDTOOLS_OUTPUTDIR)/jdk_tools_classes, \
     DISABLED_WARNINGS := dangling-doc-comments options, \
     JAVAC_FLAGS := \
@@ -69,14 +68,16 @@ TARGETS += $(BUILD_TOOLS_JDK)
 $(eval $(call SetupCopyFiles, COPY_NIMBUS_TEMPLATES, \
     SRC := $(TOPDIR)/src/java.desktop/share/classes/javax/swing/plaf/nimbus, \
     DEST := $(BUILDTOOLS_OUTPUTDIR)/jdk_tools_classes/build/tools/generatenimbus/resources, \
-    FILES := $(wildcard $(TOPDIR)/src/java.desktop/share/classes/javax/swing/plaf/nimbus/*.template)))
+    FILES := $(wildcard $(TOPDIR)/src/java.desktop/share/classes/javax/swing/plaf/nimbus/*.template), \
+))
 
 TARGETS += $(COPY_NIMBUS_TEMPLATES)
 
 $(eval $(call SetupCopyFiles, COPY_CLDRCONVERTER_PROPERTIES, \
     SRC := $(TOPDIR)/make/jdk/src/classes/build/tools/cldrconverter, \
     DEST := $(BUILDTOOLS_OUTPUTDIR)/jdk_tools_classes/build/tools/cldrconverter, \
-    FILES := $(wildcard $(TOPDIR)/make/jdk/src/classes/build/tools/cldrconverter/*.properties)))
+    FILES := $(wildcard $(TOPDIR)/make/jdk/src/classes/build/tools/cldrconverter/*.properties), \
+))
 
 TARGETS += $(COPY_CLDRCONVERTER_PROPERTIES)
 

make/CopyInterimTZDB.gmk

@@ -30,7 +30,7 @@ include MakeBase.gmk
 
 include CopyFiles.gmk
 
-##########################################################################################
+################################################################################
 
 ### TZDB tool needs files from java.time.zone package
 
@@ -45,8 +45,9 @@ $(eval $(call SetupCopyFiles,COPY_INTERIM_TZDB, \
     SRC := $(TOPDIR)/src/java.base/share/classes/java/time/zone, \
     DEST := $(BUILDTOOLS_OUTPUTDIR)/interim_tzdb_classes/build/tools/tzdb, \
     FILES := ZoneRules.java ZoneOffsetTransition.java ZoneOffsetTransitionRule.java Ser.java, \
-    MACRO := tzdb_copyfiles))
+    MACRO := tzdb_copyfiles, \
+))
 
-##########################################################################################
+################################################################################
 
 all: $(COPY_INTERIM_TZDB)

make/JrtfsJar.gmk

@@ -67,7 +67,8 @@ $(eval $(call SetupJavaCompilation, BUILD_JRTFS, \
 $(eval $(call SetupCopyFiles, COPY_JIMAGE_SERVICE_PROVIDER, \
     SRC := $(TOPDIR)/src/java.base/share/classes, \
     DEST := $(SUPPORT_OUTPUTDIR)/jrtfs_classes, \
-    FILES := META-INF/services/java.nio.file.spi.FileSystemProvider))
+    FILES := META-INF/services/java.nio.file.spi.FileSystemProvider, \
+))
 
 $(eval $(call SetupJarArchive, BUILD_JRTFS_JAR, \
     DEPENDENCIES := $(BUILD_JRTFS) $(COPY_JIMAGE_SERVICE_PROVIDER), \

make/RunTests.gmk

@@ -104,7 +104,8 @@ ifneq ($(wildcard $(JTREG_FAILURE_HANDLER)), )
       -observerDir:$(JTREG_FAILURE_HANDLER) \
       -timeoutHandler:jdk.test.failurehandler.jtreg.GatherProcessInfoTimeoutHandler \
       -observer:jdk.test.failurehandler.jtreg.GatherDiagnosticInfoObserver \
-      -timeoutHandlerTimeout:$(JTREG_FAILURE_HANDLER_TIMEOUT)
+      -timeoutHandlerTimeout:$(JTREG_FAILURE_HANDLER_TIMEOUT) \
+      #
 endif
 
 GTEST_LAUNCHER_DIRS := $(patsubst %/gtestLauncher, %, \

make/SourceRevision.gmk

@@ -72,7 +72,7 @@ ifeq ($(USE_SCM), true)
   MakeFilenameFromRepo = \
       $(strip $(subst .,top, $(subst /,-, $1)))
 
-  ################################################################################
+  ##############################################################################
   # SetupGetRevisionForRepo defines a make rule for creating a file containing
   # the name of the repository and the output of the scm command for that
   # repository.

make/TestImage.gmk

@@ -31,7 +31,7 @@ include MakeBase.gmk
 # Hook to include the corresponding custom file, if present.
 $(eval $(call IncludeCustomExtension, TestImage-pre.gmk))
 
-############################################################################
+################################################################################
 
 BUILD_INFO_PROPERTIES := $(TEST_IMAGE_DIR)/build-info.properties
 

make/ToolsHotspot.gmk

@@ -42,6 +42,6 @@ BUILD_TOOLS_HOTSPOT := $(call SetupJavaCompilationCompileTarget, \
 TOOL_JFR_GEN := $(JAVA_SMALL) -cp $(HOTSPOT_TOOLS_OUTPUTDIR) \
     build.tools.jfr.GenerateJfrFiles
 
-##########################################################################################
+################################################################################
 
 endif # _TOOLS_HOTSPOT_GMK

make/ToolsJdk.gmk

@@ -128,14 +128,14 @@ TOOL_PUBLICSUFFIXLIST = $(JAVA_SMALL) -cp $(BUILDTOOLS_OUTPUTDIR)/jdk_tools_clas
 TOOL_FIXUPPANDOC = $(JAVA_SMALL) -cp $(BUILDTOOLS_OUTPUTDIR)/jdk_tools_classes \
     build.tools.fixuppandoc.Main
 
-##########################################################################################
+################################################################################
 
 # Executable javascript filter for man page generation using pandoc.
 
 PANDOC_TROFF_MANPAGE_FILTER := $(BUILDTOOLS_OUTPUTDIR)/manpages/pandoc-troff-manpage-filter
 PANDOC_HTML_MANPAGE_FILTER := $(BUILDTOOLS_OUTPUTDIR)/manpages/pandoc-html-manpage-filter
 
-##########################################################################################
+################################################################################
 
 # Hook to include the corresponding custom post file, if present.
 $(eval $(call IncludeCustomExtension, ToolsJdk-post.gmk))

make/ZipSecurity.gmk

@@ -29,7 +29,7 @@ include $(SPEC)
 include MakeBase.gmk
 include JavaCompilation.gmk
 
-##########################################################################################
+################################################################################
 #
 # sec-bin.zip is used by builds where the corresponding sources are not available
 #
@@ -60,11 +60,12 @@ $(eval $(call SetupZipArchive,BUILD_SEC_BIN_ZIP, \
         modules/java.security.jgss/sun/security/krb5/internal/util, \
     INCLUDE_FILES := modules/java.security.jgss/sun/security/jgss/spi/GSSContextSpi.class, \
     EXCLUDES := modules/java.security.jgss/sun/security/krb5/internal/tools, \
-    ZIP := $(IMAGES_OUTPUTDIR)/sec-bin.zip))
+    ZIP := $(IMAGES_OUTPUTDIR)/sec-bin.zip, \
+))
 
 TARGETS += $(IMAGES_OUTPUTDIR)/sec-bin.zip
 
-##########################################################################################
+################################################################################
 #
 # Windows specific binary security packages.
 #
@@ -73,7 +74,8 @@ ifeq ($(call isTargetOs, windows), true)
   $(eval $(call SetupZipArchive, BUILD_SEC_WINDOWS_BIN_ZIP, \
       SRC := $(JDK_OUTPUTDIR), \
       INCLUDES := modules/java.security.jgss/sun/security/krb5/internal/tools, \
-      ZIP := $(IMAGES_OUTPUTDIR)/sec-windows-bin.zip))
+      ZIP := $(IMAGES_OUTPUTDIR)/sec-windows-bin.zip, \
+  ))
 
   TARGETS += $(IMAGES_OUTPUTDIR)/sec-windows-bin.zip
 
@@ -90,12 +92,13 @@ ifeq ($(call isTargetOs, windows), true)
           modules_libs/java.security.jgss/w2k_lsa_auth.dll.diz \
           modules_libs/java.security.jgss/w2k_lsa_auth.dll.map \
           modules_libs/java.security.jgss/w2k_lsa_auth.dll.pdb, \
-      ZIP := $(IMAGES_OUTPUTDIR)/$(JGSS_ZIP_NAME)))
+      ZIP := $(IMAGES_OUTPUTDIR)/$(JGSS_ZIP_NAME), \
+  ))
 
   TARGETS += $(IMAGES_OUTPUTDIR)/$(JGSS_ZIP_NAME)
 endif
 
-##########################################################################################
+################################################################################
 
 all: $(TARGETS)
 

make/autoconf/basic.m4

@@ -26,7 +26,7 @@
 m4_include([basic_tools.m4])
 m4_include([basic_windows.m4])
 
-###############################################################################
+################################################################################
 AC_DEFUN_ONCE([BASIC_INIT],
 [
   # Save the original command line. This is passed to us by the wrapper configure script.
@@ -46,7 +46,7 @@ AC_DEFUN_ONCE([BASIC_INIT],
   AC_MSG_NOTICE([Configuration created at $DATE_WHEN_CONFIGURED.])
 ])
 
-###############################################################################
+################################################################################
 # Check that there are no unprocessed overridden variables left.
 # If so, they are an incorrect argument and we will exit with an error.
 AC_DEFUN([BASIC_CHECK_LEFTOVER_OVERRIDDEN],
@@ -58,7 +58,7 @@ AC_DEFUN([BASIC_CHECK_LEFTOVER_OVERRIDDEN],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Setup basic configuration paths, and platform-specific stuff related to PATHs.
 # Make sure to only use tools set up in BASIC_SETUP_FUNDAMENTAL_TOOLS.
 AC_DEFUN_ONCE([BASIC_SETUP_PATHS],
@@ -102,7 +102,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_PATHS],
   AUTOCONF_DIR=$TOPDIR/make/autoconf
 ])
 
-###############################################################################
+################################################################################
 # Setup what kind of build environment type we have (CI or local developer)
 AC_DEFUN_ONCE([BASIC_SETUP_BUILD_ENV],
 [
@@ -141,7 +141,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_BUILD_ENV],
   AC_SUBST(LOCALE_USED)
 ])
 
-###############################################################################
+################################################################################
 # Evaluates platform specific overrides for devkit variables.
 # $1: Name of variable
 AC_DEFUN([BASIC_EVAL_DEVKIT_VARIABLE],
@@ -151,7 +151,7 @@ AC_DEFUN([BASIC_EVAL_DEVKIT_VARIABLE],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Evaluates platform specific overrides for build devkit variables.
 # $1: Name of variable
 AC_DEFUN([BASIC_EVAL_BUILD_DEVKIT_VARIABLE],
@@ -161,7 +161,7 @@ AC_DEFUN([BASIC_EVAL_BUILD_DEVKIT_VARIABLE],
   fi
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN([BASIC_SETUP_XCODE_SYSROOT],
 [
   AC_MSG_CHECKING([for sdk name])
@@ -246,7 +246,7 @@ AC_DEFUN([BASIC_SETUP_XCODE_SYSROOT],
   fi
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN_ONCE([BASIC_SETUP_DEVKIT],
 [
   AC_ARG_WITH([devkit], [AS_HELP_STRING([--with-devkit],
@@ -380,7 +380,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_DEVKIT],
   AC_MSG_RESULT([$EXTRA_PATH])
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN_ONCE([BASIC_SETUP_OUTPUT_DIR],
 [
 
@@ -477,7 +477,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_OUTPUT_DIR],
   AC_CONFIG_FILES([$OUTPUTDIR/Makefile:$AUTOCONF_DIR/Makefile.template])
 ])
 
-###############################################################################
+################################################################################
 # Check if build directory is on local disk. If not possible to determine,
 # we prefer to claim it's local.
 # Argument 1: directory to test
@@ -514,7 +514,7 @@ AC_DEFUN([BASIC_CHECK_DIR_ON_LOCAL_DISK],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Check that source files have basic read permissions set. This might
 # not be the case in cygwin in certain conditions.
 AC_DEFUN_ONCE([BASIC_CHECK_SRC_PERMS],
@@ -529,7 +529,7 @@ AC_DEFUN_ONCE([BASIC_CHECK_SRC_PERMS],
   fi
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN_ONCE([BASIC_TEST_USABILITY_ISSUES],
 [
   AC_MSG_CHECKING([if build directory is on local disk])
@@ -572,7 +572,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_DEFAULT_MAKE_TARGET],
   AC_SUBST(DEFAULT_MAKE_TARGET)
 ])
 
-###############################################################################
+################################################################################
 # Setup the default value for LOG=
 #
 AC_DEFUN_ONCE([BASIC_SETUP_DEFAULT_LOG],
@@ -591,7 +591,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_DEFAULT_LOG],
   AC_SUBST(DEFAULT_LOG)
 ])
 
-###############################################################################
+################################################################################
 # Code to run after AC_OUTPUT
 AC_DEFUN_ONCE([BASIC_POST_CONFIG_OUTPUT],
 [

make/autoconf/basic_tools.m4

@@ -23,12 +23,12 @@
 # questions.
 #
 
-###############################################################################
+################################################################################
 # It is recommended to use exactly this version of pandoc, especially for
 # re-generating checked in html files
 RECOMMENDED_PANDOC_VERSION=2.19.2
 
-###############################################################################
+################################################################################
 # Setup the most fundamental tools, used for setting up build platform and
 # path handling.
 AC_DEFUN_ONCE([BASIC_SETUP_FUNDAMENTAL_TOOLS],
@@ -59,7 +59,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_FUNDAMENTAL_TOOLS],
   UTIL_LOOKUP_PROGS(CMD, cmd.exe, $PATH:/cygdrive/c/windows/system32:/mnt/c/windows/system32:/c/windows/system32)
 ])
 
-###############################################################################
+################################################################################
 # Setup further tools that should be resolved early but after setting up
 # build platform and path handling.
 AC_DEFUN_ONCE([BASIC_SETUP_TOOLS],
@@ -116,7 +116,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_TOOLS],
   RM="$RM -f"
 ])
 
-###############################################################################
+################################################################################
 # Check if we have found a usable version of make
 # $1: the path to a potential make binary (or empty)
 # $2: the description on how we found this
@@ -176,7 +176,7 @@ AC_DEFUN([BASIC_CHECK_MAKE_VERSION],
   fi
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN([BASIC_CHECK_MAKE_OUTPUT_SYNC],
 [
   # Check if make supports the output sync option and if so, setup using it.
@@ -201,7 +201,7 @@ AC_DEFUN([BASIC_CHECK_MAKE_OUTPUT_SYNC],
   AC_SUBST(OUTPUT_SYNC)
 ])
 
-###############################################################################
+################################################################################
 # Goes looking for a usable version of GNU make.
 AC_DEFUN([BASIC_CHECK_GNU_MAKE],
 [
@@ -249,7 +249,7 @@ AC_DEFUN([BASIC_CHECK_GNU_MAKE],
   BASIC_CHECK_MAKE_OUTPUT_SYNC
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN([BASIC_CHECK_FIND_DELETE],
 [
   # Test if find supports -delete
@@ -278,7 +278,7 @@ AC_DEFUN([BASIC_CHECK_FIND_DELETE],
   AC_SUBST(FIND_DELETE)
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN([BASIC_CHECK_TAR],
 [
   # Test which kind of tar was found
@@ -316,7 +316,7 @@ AC_DEFUN([BASIC_CHECK_TAR],
   AC_SUBST(TAR_SUPPORTS_TRANSFORM)
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN([BASIC_CHECK_GREP],
 [
   # Test that grep supports -Fx with a list of pattern which includes null pattern.
@@ -340,7 +340,7 @@ AC_DEFUN([BASIC_CHECK_GREP],
   fi
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN_ONCE([BASIC_SETUP_COMPLEX_TOOLS],
 [
   BASIC_CHECK_GNU_MAKE
@@ -412,7 +412,7 @@ AC_DEFUN_ONCE([BASIC_SETUP_COMPLEX_TOOLS],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Check for support for specific options in bash
 AC_DEFUN_ONCE([BASIC_CHECK_BASH_OPTIONS],
 [

make/autoconf/boot-jdk.m4

@@ -23,7 +23,7 @@
 # questions.
 #
 
-########################################################################
+################################################################################
 # This file handles detection of the Boot JDK. The Boot JDK detection
 # process has been developed as a response to solve a complex real-world
 # problem. Initially, it was simple, but it has grown as platform after
@@ -49,7 +49,7 @@
 # JDK, and if one is found, check if it is acceptable. If not, we print
 # our reasons for rejecting it (useful when debugging non-working
 # configure situations) and continue checking the next one.
-########################################################################
+################################################################################
 
 # Execute the check given as argument, and verify the result
 # If the Boot JDK was previously found, do nothing
@@ -322,7 +322,7 @@ AC_DEFUN([BOOTJDK_SETUP_CLASSPATH],
   AC_SUBST(CLASSPATH)
 ])
 
-###############################################################################
+################################################################################
 #
 # We need a Boot JDK to bootstrap the build.
 #

make/autoconf/bootcycle-spec.gmk.template

@@ -48,4 +48,3 @@ JAVA_FLAGS_BIG := @BOOTCYCLE_JVM_ARGS_BIG@
 # By filtering out those JVM args, the bootcycle JVM will use its default
 # settings for CDS.
 JAVA_FLAGS := $(filter-out -XX:SharedArchiveFile% -Xshare%, $(JAVA_FLAGS))
-

make/autoconf/compare.sh.template

@@ -26,7 +26,7 @@
 # the root of the build directory.
 
 
-##########################################################################################
+################################################################################
 # Substitutions from autoconf
 
 export LEGACY_BUILD_DIR=@OPENJDK_TARGET_OS@-@OPENJDK_TARGET_CPU_LEGACY@

make/autoconf/configure.ac

@@ -23,11 +23,11 @@
 # questions.
 #
 
-###############################################################################
+################################################################################
 #
 # Includes and boilerplate
 #
-###############################################################################
+################################################################################
 
 
 AC_PREREQ([2.69])
@@ -63,14 +63,14 @@ m4_include([platform.m4])
 m4_include([source-dirs.m4])
 m4_include([toolchain.m4])
 
-###############################################################################
+################################################################################
 #
 # Initialization / Boot-strapping
 #
 # The bootstrapping process needs to solve the "chicken or the egg" problem,
 # thus it jumps back and forth, each time gaining something needed later on.
 #
-###############################################################################
+################################################################################
 
 # If we are requested to print additional help, do that and then exit.
 # This must be the very first call.
@@ -128,51 +128,51 @@ PLATFORM_SETUP_OPENJDK_BUILD_OS_VERSION
 BASIC_SETUP_DEFAULT_MAKE_TARGET
 BASIC_SETUP_DEFAULT_LOG
 
-###############################################################################
+################################################################################
 #
 # Determine OpenJDK variants and version numbers.
 #
-###############################################################################
+################################################################################
 
 # We need build & target for this.
 JDKOPT_SETUP_JMOD_OPTIONS
 JDKOPT_SETUP_JLINK_OPTIONS
 JDKVER_SETUP_JDK_VERSION_NUMBERS
 
-###############################################################################
+################################################################################
 #
 # Setup BootJDK, used to bootstrap the build.
 #
-###############################################################################
+################################################################################
 
 BOOTJDK_SETUP_BOOT_JDK
 BOOTJDK_SETUP_BUILD_JDK
 BOOTJDK_SETUP_DOCS_REFERENCE_JDK
 
-###############################################################################
+################################################################################
 #
 # Determine JDK specific build time options.
 #
-###############################################################################
+################################################################################
 
 JDKOPT_SETUP_REPRODUCIBLE_BUILD
 JDKOPT_SETUP_JDK_OPTIONS
 
-###############################################################################
+################################################################################
 #
 # Configure the sources to use. We can add or override individual directories.
 #
-###############################################################################
+################################################################################
 
 SRCDIRS_SETUP_DIRS
 SRCDIRS_SETUP_IMPORT_MODULES
 
-###############################################################################
+################################################################################
 #
 # Setup the toolchain (compilers etc), i.e. tools used to compile and process
 # native code.
 #
-###############################################################################
+################################################################################
 
 # See if we are doing a complete static build or not
 JDKOPT_SETUP_STATIC_BUILD
@@ -227,31 +227,31 @@ JDKOPT_SETUP_LEAK_SANITIZER
 # This needs to go before 'LIB_DETERMINE_DEPENDENCIES'
 JDKOPT_SETUP_FALLBACK_LINKER
 
-###############################################################################
+################################################################################
 #
 # Check dependencies for external and internal libraries.
 #
-###############################################################################
+################################################################################
 
 LIB_DETERMINE_DEPENDENCIES
 LIB_SETUP_LIBRARIES
 
-###############################################################################
+################################################################################
 #
 # Setup hotspot and JVM features (needs toolchain).
 #
-###############################################################################
+################################################################################
 
 JVM_FEATURES_PARSE_OPTIONS
 JVM_FEATURES_SETUP
 
 HOTSPOT_SETUP_MISC
 
-###############################################################################
+################################################################################
 #
 # We need to do some final tweaking, when everything else is done.
 #
-###############################################################################
+################################################################################
 
 LIB_TESTS_ENABLE_DISABLE_FAILURE_HANDLER
 LIB_TESTS_ENABLE_DISABLE_JTREG_TEST_THREAD_FACTORY
@@ -263,12 +263,12 @@ JDKOPT_ENABLE_DISABLE_CDS_ARCHIVE
 JDKOPT_ENABLE_DISABLE_COMPATIBLE_CDS_ALIGNMENT
 JDKOPT_SETUP_MACOSX_SIGNING
 
-###############################################################################
+################################################################################
 #
 # Configure parts of the build that only affect the build performance,
 # not the result.
 #
-###############################################################################
+################################################################################
 
 BPERF_SETUP_BUILD_CORES
 BPERF_SETUP_BUILD_MEMORY
@@ -288,11 +288,11 @@ BPERF_SETUP_PRECOMPILED_HEADERS
 # Setup use of ccache, if available
 BPERF_SETUP_CCACHE
 
-###############################################################################
+################################################################################
 #
 # And now the finish...
 #
-###############################################################################
+################################################################################
 
 # Check for some common pitfalls
 BASIC_TEST_USABILITY_ISSUES

make/autoconf/flags-cflags.m4

@@ -480,7 +480,7 @@ AC_DEFUN([FLAGS_SETUP_CFLAGS_HELPER],
   # Always enable optional macros for VM.
   ALWAYS_CFLAGS_JVM="-D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS"
 
-  ###############################################################################
+  ##############################################################################
 
   # Adjust flags according to debug level.
   # Setup debug/release defines
@@ -514,7 +514,7 @@ AC_DEFUN([FLAGS_SETUP_CFLAGS_HELPER],
     ALWAYS_DEFINES_JVM="$ALWAYS_DEFINES -DNOMINMAX"
   fi
 
-  ###############################################################################
+  ##############################################################################
   #
   #
   # CFLAGS BASIC

make/autoconf/hotspot.m4

@@ -26,7 +26,7 @@
 # All valid JVM variants
 VALID_JVM_VARIANTS="server client minimal core zero custom"
 
-###############################################################################
+################################################################################
 # Check if the specified JVM variant should be built. To be used in shell if
 # constructs, like this:
 # if HOTSPOT_CHECK_JVM_VARIANT(server); then
@@ -38,7 +38,7 @@ VALID_JVM_VARIANTS="server client minimal core zero custom"
 AC_DEFUN([HOTSPOT_CHECK_JVM_VARIANT],
 [ [ [[ " $JVM_VARIANTS " =~ " $1 " ]] ] ])
 
-###############################################################################
+################################################################################
 # Check which variants of the JVM that we want to build. Available variants are:
 #   server: normal interpreter, and a tiered C1/C2 compiler
 #   client: normal interpreter, and C1 (no C2 compiler)
@@ -102,7 +102,7 @@ AC_DEFUN_ONCE([HOTSPOT_SETUP_JVM_VARIANTS],
   AC_SUBST(JVM_VARIANT_MAIN)
 ])
 
-###############################################################################
+################################################################################
 # Misc hotspot setup that does not fit elsewhere.
 #
 AC_DEFUN_ONCE([HOTSPOT_SETUP_MISC],

make/autoconf/jdk-options.m4

@@ -23,7 +23,7 @@
 # questions.
 #
 
-###############################################################################
+################################################################################
 # Set the debug level
 #    release: no debug information, all optimizations, no asserts.
 #    optimized: no debug information, all optimizations, no asserts, HotSpot target is 'optimized'.
@@ -81,7 +81,7 @@ AC_DEFUN_ONCE([JDKOPT_SETUP_DEBUG_LEVEL],
   AC_SUBST(DEBUG_LEVEL)
 ])
 
-###############################################################################
+################################################################################
 #
 # Should we build only OpenJDK even if closed sources are present?
 #
@@ -246,7 +246,7 @@ AC_DEFUN_ONCE([JDKOPT_SETUP_JDK_OPTIONS],
 
 ])
 
-###############################################################################
+################################################################################
 
 AC_DEFUN_ONCE([JDKOPT_SETUP_DEBUG_SYMBOLS],
 [
@@ -410,7 +410,7 @@ AC_DEFUN_ONCE([JDKOPT_SETUP_CODE_COVERAGE],
   AC_SUBST(JCOV_FILTERS)
 ])
 
-###############################################################################
+################################################################################
 #
 # AddressSanitizer
 #
@@ -467,7 +467,7 @@ AC_DEFUN_ONCE([JDKOPT_SETUP_ADDRESS_SANITIZER],
   AC_SUBST(ASAN_ENABLED)
 ])
 
-###############################################################################
+################################################################################
 #
 # LeakSanitizer
 #
@@ -500,7 +500,7 @@ AC_DEFUN_ONCE([JDKOPT_SETUP_LEAK_SANITIZER],
   AC_SUBST(LSAN_ENABLED)
 ])
 
-###############################################################################
+################################################################################
 #
 # UndefinedBehaviorSanitizer
 #

make/autoconf/jdk-version.m4

@@ -23,7 +23,7 @@
 # questions.
 #
 
-###############################################################################
+################################################################################
 #
 # Setup version numbers
 #

make/autoconf/jvm-features.m4

@@ -23,7 +23,7 @@
 # questions.
 #
 
-###############################################################################
+################################################################################
 # Terminology used in this file:
 #
 # Valid features      == All possible features that the JVM knows about.
@@ -36,7 +36,7 @@
 #
 # All valid features are considered available, unless listed as unavailable.
 # All available features will be turned on as default, unless listed in a filter.
-###############################################################################
+################################################################################
 
 # We need these as m4 defines to be able to loop over them using m4 later on.
 
@@ -78,7 +78,7 @@ m4_define(jvm_feature_desc_vm_structs, [export JVM structures to the Serviceabli
 m4_define(jvm_feature_desc_zero, [support building variant 'zero'])
 m4_define(jvm_feature_desc_zgc, [include the Z garbage collector])
 
-###############################################################################
+################################################################################
 # Parse command line options for JVM feature selection. After this function
 # has run $JVM_FEATURES_ENABLED, $JVM_FEATURES_DISABLED and $JVM_FEATURES_VALID
 # can be used.
@@ -199,7 +199,7 @@ AC_DEFUN_ONCE([JVM_FEATURES_PARSE_OPTIONS],
   AC_SUBST(VALID_JVM_FEATURES)
 ])
 
-###############################################################################
+################################################################################
 # Helper function for the JVM_FEATURES_CHECK_* suite.
 # The code in the code block should assign 'false' to the variable AVAILABLE
 # if the feature is not available, and this function will handle everything
@@ -225,7 +225,7 @@ AC_DEFUN([JVM_FEATURES_CHECK_AVAILABILITY],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Check if the feature 'cds' is available on this platform.
 #
 AC_DEFUN_ONCE([JVM_FEATURES_CHECK_CDS],
@@ -241,7 +241,7 @@ AC_DEFUN_ONCE([JVM_FEATURES_CHECK_CDS],
   ])
 ])
 
-###############################################################################
+################################################################################
 # Check if the feature 'dtrace' is available on this platform.
 #
 AC_DEFUN_ONCE([JVM_FEATURES_CHECK_DTRACE],
@@ -270,7 +270,7 @@ AC_DEFUN_ONCE([JVM_FEATURES_CHECK_DTRACE],
   ])
 ])
 
-###############################################################################
+################################################################################
 # Check if the feature 'jvmci' is available on this platform.
 #
 AC_DEFUN_ONCE([JVM_FEATURES_CHECK_JVMCI],
@@ -290,7 +290,7 @@ AC_DEFUN_ONCE([JVM_FEATURES_CHECK_JVMCI],
   ])
 ])
 
-###############################################################################
+################################################################################
 # Check if the feature 'shenandoahgc' is available on this platform.
 #
 AC_DEFUN_ONCE([JVM_FEATURES_CHECK_SHENANDOAHGC],
@@ -309,7 +309,7 @@ AC_DEFUN_ONCE([JVM_FEATURES_CHECK_SHENANDOAHGC],
   ])
 ])
 
-###############################################################################
+################################################################################
 # Check if the feature 'zgc' is available on this platform.
 #
 AC_DEFUN_ONCE([JVM_FEATURES_CHECK_ZGC],
@@ -365,7 +365,7 @@ AC_DEFUN_ONCE([JVM_FEATURES_CHECK_ZGC],
   ])
 ])
 
-###############################################################################
+################################################################################
 # Setup JVM_FEATURES_PLATFORM_UNAVAILABLE and JVM_FEATURES_PLATFORM_FILTER
 # to contain those features that are unavailable, or should be off by default,
 # for this platform, regardless of JVM variant.
@@ -383,7 +383,7 @@ AC_DEFUN_ONCE([JVM_FEATURES_PREPARE_PLATFORM],
 
 ])
 
-###############################################################################
+################################################################################
 # Setup JVM_FEATURES_VARIANT_UNAVAILABLE and JVM_FEATURES_VARIANT_FILTER
 # to contain those features that are unavailable, or should be off by default,
 # for this particular JVM variant.
@@ -431,7 +431,7 @@ AC_DEFUN([JVM_FEATURES_PREPARE_VARIANT],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Calculate the actual set of active JVM features for this JVM variant. Store
 # the result in JVM_FEATURES_ACTIVE.
 #
@@ -479,7 +479,7 @@ AC_DEFUN([JVM_FEATURES_CALCULATE_ACTIVE],
       $JVM_FEATURES_ENABLED, $JVM_FEATURES_DISABLED)
 ])
 
-###############################################################################
+################################################################################
 # Helper function for JVM_FEATURES_VERIFY. Check if the specified JVM
 # feature is active. To be used in shell if constructs, like this:
 # 'if JVM_FEATURES_IS_ACTIVE(jvmti); then'
@@ -489,7 +489,7 @@ AC_DEFUN([JVM_FEATURES_CALCULATE_ACTIVE],
 AC_DEFUN([JVM_FEATURES_IS_ACTIVE],
 [ [ [[ " $JVM_FEATURES_ACTIVE " =~ ' '$1' ' ]] ] ])
 
-###############################################################################
+################################################################################
 # Verify that the resulting set of features is consistent and legal.
 #
 # arg 1: JVM variant
@@ -527,7 +527,7 @@ AC_DEFUN([JVM_FEATURES_VERIFY],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Set up all JVM features for each enabled JVM variant. Requires that
 # JVM_FEATURES_PARSE_OPTIONS has been called.
 #

make/autoconf/lib-tests.m4

@@ -31,7 +31,7 @@
 JTREG_MINIMUM_VERSION=7.4
 GTEST_MINIMUM_VERSION=1.14.0
 
-###############################################################################
+################################################################################
 #
 # Setup and check for gtest framework source files
 #
@@ -74,7 +74,7 @@ AC_DEFUN_ONCE([LIB_TESTS_SETUP_GTEST],
   AC_SUBST(GTEST_FRAMEWORK_SRC)
 ])
 
-###############################################################################
+################################################################################
 #
 # Setup and check the Java Microbenchmark Harness
 #

make/autoconf/platform.m4

@@ -678,7 +678,7 @@ AC_DEFUN([PLATFORM_CHECK_DEPRECATION],
 
 AC_DEFUN_ONCE([PLATFORM_SETUP_OPENJDK_BUILD_OS_VERSION],
 [
-  ###############################################################################
+  ##############################################################################
 
   # Note that this is the build platform OS version!
 
@@ -693,7 +693,7 @@ AC_DEFUN_ONCE([PLATFORM_SETUP_OPENJDK_BUILD_OS_VERSION],
 
 AC_DEFUN_ONCE([PLATFORM_SETUP_OPENJDK_TARGET_BITS],
 [
-  ###############################################################################
+  ##############################################################################
   #
   # Now we check if libjvm.so will use 32 or 64 bit pointers for the C/C++ code.
   # (The JVM can use 32 or 64 bit Java pointers but that decision
@@ -739,7 +739,7 @@ AC_DEFUN_ONCE([PLATFORM_SETUP_OPENJDK_TARGET_BITS],
 
 AC_DEFUN_ONCE([PLATFORM_SETUP_OPENJDK_TARGET_ENDIANNESS],
 [
-  ###############################################################################
+  ##############################################################################
   #
   # Is the target little or big endian?
   #

make/autoconf/spec.gmk.template

@@ -804,11 +804,7 @@ UCRT_DLL_DIR := @UCRT_DLL_DIR@
 ENABLE_PANDOC := @ENABLE_PANDOC@
 PANDOC_MARKDOWN_FLAG := @PANDOC_MARKDOWN_FLAG@
 
-####################################################
-#
 # Libraries
-#
-
 USE_EXTERNAL_LCMS := @USE_EXTERNAL_LCMS@
 LCMS_CFLAGS := @LCMS_CFLAGS@
 LCMS_LIBS := @LCMS_LIBS@
@@ -821,11 +817,7 @@ USE_EXTERNAL_LIBPNG := @USE_EXTERNAL_LIBPNG@
 PNG_LIBS := @PNG_LIBS@
 PNG_CFLAGS := @PNG_CFLAGS@
 
-####################################################
-#
 # Misc
-#
-
 INCLUDE_SA := @INCLUDE_SA@
 INCLUDE_JVMCI := @INCLUDE_JVMCI@
 INCLUDE_COMPILER2 := @INCLUDE_COMPILER2@

make/autoconf/toolchain.m4

@@ -23,14 +23,14 @@
 # questions.
 #
 
-########################################################################
+################################################################################
 # This file is responsible for detecting, verifying and setting up the
 # toolchain, i.e. the compiler, linker and related utilities. It will setup
 # proper paths to the binaries, but it will not setup any flags.
 #
 # The binaries used is determined by the toolchain type, which is the family of
 # compilers and related tools that are used.
-########################################################################
+################################################################################
 
 m4_include([toolchain_microsoft.m4])
 

make/autoconf/util.m4

@@ -25,7 +25,7 @@
 
 m4_include([util_paths.m4])
 
-###############################################################################
+################################################################################
 # Create a function/macro that takes a series of named arguments. The call is
 # similar to AC_DEFUN, but the setup of the function looks like this:
 # UTIL_DEFUN_NAMED([MYFUNC], [FOO *BAR], [$@], [
@@ -100,7 +100,7 @@ AC_DEFUN([UTIL_DEFUN_NAMED],
   ])
 ])
 
-###############################################################################
+################################################################################
 # Assert that a programmatic condition holds. If not, exit with an error message.
 # Check that a shell expression gives return code 0
 #
@@ -121,7 +121,7 @@ AC_DEFUN([UTIL_ASSERT_SHELL_TEST],
 ])
 
 
-###############################################################################
+################################################################################
 # Assert that a programmatic condition holds. If not, exit with an error message.
 # Check that two strings are equal.
 #
@@ -137,7 +137,7 @@ AC_DEFUN([UTIL_ASSERT_STRING_EQUALS],
       $3)
 ])
 
-###############################################################################
+################################################################################
 # Assert that a programmatic condition holds. If not, exit with an error message.
 # Check that two strings not are equal.
 #
@@ -153,7 +153,7 @@ AC_DEFUN([UTIL_ASSERT_STRING_NOT_EQUALS],
       $3)
 ])
 
-###############################################################################
+################################################################################
 # Assert that a programmatic condition holds. If not, exit with an error message.
 # Check that the given expression evaluates to the string 'true'
 #
@@ -165,7 +165,7 @@ AC_DEFUN([UTIL_ASSERT_TRUE],
   UTIL_ASSERT_STRING_EQUALS($1, true, $3)
 ])
 
-###############################################################################
+################################################################################
 # Assert that a programmatic condition holds. If not, exit with an error message.
 # Check that the given expression does not evaluate to the string 'true'
 #
@@ -177,7 +177,7 @@ AC_DEFUN([UTIL_ASSERT_NOT_TRUE],
   UTIL_ASSERT_STRING_NOT_EQUALS($1, true, $3)
 ])
 
-###############################################################################
+################################################################################
 # Check if a list of space-separated words are selected only from a list of
 # space-separated legal words. Typical use is to see if a user-specified
 # set of words is selected from a set of legal words.
@@ -204,7 +204,7 @@ AC_DEFUN([UTIL_GET_NON_MATCHING_VALUES],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Check if a list of space-separated words contains any word(s) from a list of
 # space-separated illegal words. Typical use is to see if a user-specified
 # set of words contains any from a set of illegal words.
@@ -231,7 +231,7 @@ AC_DEFUN([UTIL_GET_MATCHING_VALUES],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Converts an ISO-8601 date/time string to a unix epoch timestamp. If no
 # suitable conversion method was found, an empty string is returned.
 #
@@ -259,7 +259,7 @@ AC_DEFUN([UTIL_GET_EPOCH_TIMESTAMP],
   $1=$timestamp
 ])
 
-###############################################################################
+################################################################################
 # Sort a space-separated list, and remove duplicates.
 #
 # Sets the specified variable to the resulting list.
@@ -273,7 +273,7 @@ AC_DEFUN([UTIL_SORT_LIST],
   $1=${result//$'\n'/ }
 ])
 
-###############################################################################
+################################################################################
 # Test if $1 is a valid argument to $3 (often is $JAVA passed as $3)
 # If so, then append $1 to $2 \
 # Also set JVM_ARG_OK to true/false depending on outcome.
@@ -294,7 +294,7 @@ AC_DEFUN([UTIL_ADD_JVM_ARG_IF_OK],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Register a --with argument but mark it as deprecated
 # $1: The name of the with argument to deprecate, not including --with-
 AC_DEFUN([UTIL_DEPRECATED_ARG_WITH],
@@ -304,7 +304,7 @@ AC_DEFUN([UTIL_DEPRECATED_ARG_WITH],
       [AC_MSG_WARN([Option --with-$1 is deprecated and will be ignored.])])
 ])
 
-###############################################################################
+################################################################################
 # Register a --enable argument but mark it as deprecated
 # $1: The name of the with argument to deprecate, not including --enable-
 AC_DEFUN([UTIL_DEPRECATED_ARG_ENABLE],
@@ -314,7 +314,7 @@ AC_DEFUN([UTIL_DEPRECATED_ARG_ENABLE],
       [AC_MSG_WARN([Option --enable-$1 is deprecated and will be ignored.])])
 ])
 
-###############################################################################
+################################################################################
 # Register an --enable-* argument as an alias for another argument.
 # $1: The name of the enable argument for the new alias, not including --enable-
 # $2: The full name of the argument of which to make this an alias, including
@@ -329,7 +329,7 @@ AC_DEFUN([UTIL_ALIASED_ARG_ENABLE],
   ])
 ])
 
-###############################################################################
+################################################################################
 # Creates a command-line option using the --enable-* pattern. Will return a
 # value of 'true' or 'false' in the RESULT variable, depending on whether the
 # option was enabled or not by the user. The option can not be turned on if it
@@ -471,7 +471,7 @@ UTIL_DEFUN_NAMED([UTIL_ARG_ENABLE],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Helper functions for ARG_WITH, to validate different types of argument
 
 # Dispatcher to call the correct UTIL_CHECK_TYPE_* function depending on the ARG_TYPE
@@ -575,7 +575,7 @@ AC_DEFUN([UTIL_CHECK_TYPE_features],
   ARG_RESULT=$($ECHO $feature_list)
 ])
 
-###############################################################################
+################################################################################
 # Creates a command-line option using the --with-* pattern. Will return a
 # string in the RESULT variable with the option provided by the user, or the
 # empty string if the --with-* option was not given. The option can not be given
@@ -810,7 +810,7 @@ UTIL_DEFUN_NAMED([UTIL_ARG_WITH],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Helper functions for CHECK_VALUE in ARG_WITH.
 AC_DEFUN([UTIL_CHECK_STRING_NON_EMPTY],
 [

make/autoconf/util_paths.m4

@@ -23,7 +23,7 @@
 # questions.
 #
 
-###############################################################################
+################################################################################
 # Appends a string to a path variable, only adding the : when needed.
 AC_DEFUN([UTIL_APPEND_TO_PATH],
 [
@@ -36,7 +36,7 @@ AC_DEFUN([UTIL_APPEND_TO_PATH],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Prepends a string to a path variable, only adding the : when needed.
 AC_DEFUN([UTIL_PREPEND_TO_PATH],
 [
@@ -49,7 +49,7 @@ AC_DEFUN([UTIL_PREPEND_TO_PATH],
   fi
 ])
 
-###############################################################################
+################################################################################
 # This will make sure the given variable points to a full and proper
 # path. This means:
 # 1) There will be no spaces in the path. On unix platforms,
@@ -118,7 +118,7 @@ AC_DEFUN([UTIL_FIXUP_PATH],
   fi
 ])
 
-##############################################################################
+################################################################################
 # Fixup path to be a Windows full long path
 # Note: Only supported with cygwin/msys2 (cygpath tool)
 AC_DEFUN([UTIL_FIXUP_WIN_LONG_PATH],
@@ -136,7 +136,7 @@ AC_DEFUN([UTIL_FIXUP_WIN_LONG_PATH],
 ])
 
 
-###############################################################################
+################################################################################
 # Check if the given file is a unix-style or windows-style executable, that is,
 # if it expects paths in unix-style or windows-style.
 # Returns "windows" or "unix" in $RESULT.
@@ -170,7 +170,7 @@ AC_DEFUN([UTIL_CHECK_WINENV_EXEC_TYPE],
   fi
 ])
 
-###############################################################################
+################################################################################
 # This will make sure the given variable points to a executable
 # with a full and proper path. This means:
 # 1) There will be no spaces in the path. On unix platforms,
@@ -289,7 +289,7 @@ AC_DEFUN([UTIL_FIXUP_EXECUTABLE],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Setup a tool for the given variable. If correctly specified by the user,
 # use that value, otherwise search for the tool using the supplied code snippet.
 # $1: variable to set
@@ -369,7 +369,7 @@ AC_DEFUN([UTIL_SETUP_TOOL],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Locate a tool using proper methods.
 # $1: variable to set
 # $2: executable name (or list of names) to look for
@@ -436,7 +436,7 @@ AC_DEFUN([UTIL_LOOKUP_PROGS],
   ])
 ])
 
-###############################################################################
+################################################################################
 # Call UTIL_SETUP_TOOL with AC_CHECK_TOOLS to locate the tool. This will look
 # first for cross-compilation tools.
 # $1: variable to set
@@ -452,7 +452,7 @@ AC_DEFUN([UTIL_LOOKUP_TOOLCHAIN_PROGS],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Test that variable $1 denoting a program is not empty. If empty, exit with an error.
 # $1: variable to check
 AC_DEFUN([UTIL_CHECK_NONEMPTY],
@@ -462,7 +462,7 @@ AC_DEFUN([UTIL_CHECK_NONEMPTY],
   fi
 ])
 
-###############################################################################
+################################################################################
 # Like UTIL_LOOKUP_PROGS but fails if no tool was found.
 # $1: variable to set
 # $2: executable name (or list of names) to look for
@@ -473,7 +473,7 @@ AC_DEFUN([UTIL_REQUIRE_PROGS],
   UTIL_CHECK_NONEMPTY($1)
 ])
 
-###############################################################################
+################################################################################
 # Like UTIL_LOOKUP_PROGS but fails if no tool was found.
 # $1: variable to set
 # $2: executable name (or list of names) to look for
@@ -485,7 +485,7 @@ AC_DEFUN([UTIL_REQUIRE_TOOLCHAIN_PROGS],
 ])
 
 
-###############################################################################
+################################################################################
 # Like UTIL_SETUP_TOOL but fails if no tool was found.
 # $1: variable to set
 # $2: autoconf macro to call to look for the special tool
@@ -497,7 +497,7 @@ AC_DEFUN([UTIL_REQUIRE_SPECIAL],
   # unix tools. No further processing needed.
 ])
 
-###############################################################################
+################################################################################
 # Add FIXPATH prefix to variable. Normally this is done by UTIL_LOOKUP_PROGS
 # or UTIL_FIXUP_EXECUTABLE, but in some circumstances this has to be done
 # explicitly, such as when the command in question does not exist yet.
@@ -510,7 +510,7 @@ AC_DEFUN([UTIL_ADD_FIXPATH],
   fi
 ])
 
-###############################################################################
+################################################################################
 AC_DEFUN([UTIL_REMOVE_SYMBOLIC_LINKS],
 [
   if test "x$OPENJDK_BUILD_OS" != xwindows; then

make/common/CopyFiles.gmk

@@ -23,7 +23,7 @@
 # questions.
 #
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including CopyFiles.gmk)
 endif
 

make/common/Execute.gmk

@@ -23,7 +23,7 @@
 # questions.
 #
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including Execute.gmk)
 endif
 

make/common/FileUtils.gmk

@@ -23,7 +23,7 @@
 # questions.
 #
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including FileUtils.gmk)
 endif
 

make/common/JarArchive.gmk

@@ -26,7 +26,7 @@
 ifndef _JAR_ARCHIVE_GMK
 _JAR_ARCHIVE_GMK := 1
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including JarArchive.gmk)
 endif
 
@@ -40,26 +40,26 @@ FALSE_FIND_PATTERN:=-name FILE_NAME_THAT_DOESNT_EXIST
 # and the targets generated are listed in a variable by that name.
 #
 # Remaining parameters are named arguments. These include:
-#   DEPENDENCIES:=List of dependencies for the jar target. If left empty,
+#   DEPENDENCIES List of dependencies for the jar target. If left empty,
 #       dependencies are calculated automatically from the source files found.
 #       For this to work, the source files must exist when the makefile is
 #       parsed.
-#   SRCS:=List of directories in where to find files to add to archive
-#   BIN:=Directory where to store build control files
-#   SUFFIXES:=File suffixes to include in jar
-#   INCLUDES:=List of directories/packages in SRCS that should be included
-#   EXCLUDES:=List of directories/packages in SRCS that should be excluded
-#   EXCLUDE_FILES:=List of files in SRCS that should be excluded
-#   EXTRA_FILES:=List of files in SRCS that should be included regardless of suffix match.
-#   JAR:=Jar file to create
-#   MANIFEST:=Optional manifest file template.
-#   JARMAIN:=Optional main class to add to manifest
-#   JARINDEX:=true means generate the index in the jar file.
-#   SKIP_METAINF:=Set to prevent contents of an META-INF directory to be automatically
+#   SRCS List of directories in where to find files to add to archive
+#   BIN Directory where to store build control files
+#   SUFFIXES File suffixes to include in jar
+#   INCLUDES List of directories/packages in SRCS that should be included
+#   EXCLUDES List of directories/packages in SRCS that should be excluded
+#   EXCLUDE_FILES List of files in SRCS that should be excluded
+#   EXTRA_FILES List of files in SRCS that should be included regardless of suffix match.
+#   JAR Jar file to create
+#   MANIFEST Optional manifest file template.
+#   JARMAIN Optional main class to add to manifest
+#   JARINDEX true means generate the index in the jar file.
+#   SKIP_METAINF Set to prevent contents of an META-INF directory to be automatically
 #       added to the archive.
-#   EXTRA_MANIFEST_ATTR:=Extra attribute to add to manifest.
+#   EXTRA_MANIFEST_ATTR Extra attribute to add to manifest.
 #   CHECK_COMPRESS_JAR Check the COMPRESS_JAR variable
-#   JAR_CMD:=Optionally override the jar command to use when creating the archive.
+#   JAR_CMD Optionally override the jar command to use when creating the archive.
 SetupJarArchive = $(NamedParamsMacroTemplate)
 define SetupJarArchiveBody
 
@@ -72,7 +72,7 @@ define SetupJarArchiveBody
   $1_DELETES_FILE := $$($1_BIN)/_the.$$($1_JARNAME)_deletes
   $$(call SetIfEmpty, $1_JAR_CMD, $$(JAR))
 
-  ifeq (,$$($1_SUFFIXES))
+  ifeq ($$($1_SUFFIXES), )
     # No suffix was set, default to classes.
     $1_SUFFIXES := .class
   endif
@@ -81,7 +81,7 @@ define SetupJarArchiveBody
   # On windows, a lot of includes/excludes risk making the command line too long, so
   # writing the grep patterns to files.
   # Grep returns 1 if nothing is matched. Do not fail the build for this.
-  ifneq (,$$($1_INCLUDES))
+  ifneq ($$($1_INCLUDES), )
     $1_GREP_INCLUDE_PATTERNS := $$(call EscapeDollar, \
         $$(foreach src,$$($1_SRCS), $$(addprefix $$(src)/,$$($1_INCLUDES))))
     # If there are a lot of include patterns, output to file to shorten command lines
@@ -96,7 +96,7 @@ define SetupJarArchiveBody
           || test "$$$$?" = "1" )
     endif
   endif
-  ifneq (,$$($1_EXCLUDES)$$($1_EXCLUDE_FILES))
+  ifneq ($$($1_EXCLUDES)$$($1_EXCLUDE_FILES), )
     $1_GREP_EXCLUDE_PATTERNS := $$(call EscapeDollar, \
         $$(foreach src, $$($1_SRCS), $$(addprefix $$(src)/, \
         $$($1_EXCLUDES) $$($1_EXCLUDE_FILES))))
@@ -114,7 +114,7 @@ define SetupJarArchiveBody
   endif
 
   # Check if this jar needs to have its index generated.
-  ifneq (,$$($1_JARINDEX))
+  ifneq ($$($1_JARINDEX), )
     $1_JARINDEX = (cd $$(dir $$@) && $$($1_JAR_CMD) -i $$(notdir $$@))
   else
     $1_JARINDEX = true
@@ -129,16 +129,16 @@ define SetupJarArchiveBody
     $$(call FillFindCache, $$($1_FIND_LIST))
     $1_DEPENDENCIES := $$(filter $$(addprefix %, $$($1_SUFFIXES)), \
         $$(call FindFiles, $$($1_SRCS)))
-    ifneq (,$$($1_GREP_INCLUDE_PATTERNS))
+    ifneq ($$($1_GREP_INCLUDE_PATTERNS), )
       $1_DEPENDENCIES := $$(filter $$(addsuffix %, $$($1_GREP_INCLUDE_PATTERNS)), $$($1_DEPENDENCIES))
     endif
-    ifneq (,$$($1_GREP_EXCLUDE_PATTERNS))
+    ifneq ($$($1_GREP_EXCLUDE_PATTERNS), )
       $1_DEPENDENCIES := $$(filter-out $$(addsuffix %, $$($1_GREP_EXCLUDE_PATTERNS)), $$($1_DEPENDENCIES))
     endif
     # Look for EXTRA_FILES in all SRCS dirs and as absolute paths.
     $1_DEPENDENCIES += $$(wildcard $$(foreach src, $$($1_SRCS), \
         $$(addprefix $$(src)/, $$($1_EXTRA_FILES))) $$($1_EXTRA_FILES))
-    ifeq (,$$($1_SKIP_METAINF))
+    ifeq ($$($1_SKIP_METAINF), )
       $1_DEPENDENCIES += $$(call FindFiles, $$(wildcard $$(addsuffix /META-INF, $$($1_SRCS))))
     endif
   endif
@@ -180,7 +180,7 @@ define SetupJarArchiveBody
 
   # The capture metainf macro finds all files below the META-INF directory that are newer than the jar-file.
   # Find returns non zero if the META-INF dir does not exist, ignore this.
-  ifeq (,$$($1_SKIP_METAINF))
+  ifeq ($$($1_SKIP_METAINF), )
     $1_CAPTURE_METAINF = $$(foreach src, $$($1_SRCS), \
         ( ( $(FIND) $$(src)/META-INF -type f -a -newer $$@ 2> /dev/null || true ) \
             | $(SED) 's|$$(src)/|-C $$(src) |g' >> \
@@ -209,7 +209,7 @@ define SetupJarArchiveBody
       $$($1_CAPTURE_EXTRA_FILES)
 
   # Find returns non zero if the META-INF dir does not exist, ignore this.
-  ifeq (,$$($1_SKIP_METAINF))
+  ifeq ($$($1_SKIP_METAINF), )
     $1_SCAPTURE_METAINF = $$(foreach src, $$($1_SRCS), \
         ( ( $(FIND) $$(src)/META-INF -type f 2> /dev/null || true ) \
             | $(SED) 's|$$(src)/|-C $$(src) |g' >> \
@@ -228,7 +228,7 @@ define SetupJarArchiveBody
   ifeq ($$(BOOT_JDK_JAR_SUPPORTS_DATE), true)
     $1_JAR_OPTIONS += --date $(SOURCE_DATE_ISO_8601)
   endif
-  ifneq (,$$($1_CHECK_COMPRESS_JAR))
+  ifneq ($$($1_CHECK_COMPRESS_JAR), )
     ifneq ($(COMPRESS_JARS), true)
       $1_JAR_OPTIONS += --no-compress
     endif

make/common/JavaCompilation.gmk

@@ -26,7 +26,7 @@
 ifndef _JAVA_COMPILATION_GMK
 _JAVA_COMPILATION_GMK := 1
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including JavaCompilation.gmk)
 endif
 
@@ -138,39 +138,39 @@ endef
 # The target for public API digest is returned in $1_API_TARGET.
 #
 # Remaining parameters are named arguments. These include:
-#   SMALL_JAVA:=set to false to run javac as a "big" java app
-#   COMPILER:=bootjdk or interim, the latter is default
-#   TARGET_RELEASE:=javac flags to set the targeted jdk release (-source/-target or --release)
+#   SMALL_JAVA set to false to run javac as a "big" java app
+#   COMPILER bootjdk or interim, the latter is default
+#   TARGET_RELEASE javac flags to set the targeted jdk release (-source/-target or --release)
 #   Defaults to $(TARGET_RELEASE_NEWJDK).
-#   JAVAC_FLAGS:=javac flags to append to the default ones.
-#   JAVA_FLAGS:=flags to be appended to the java launching the compiler
-#   DISABLED_WARNINGS:=list of Xlint warnings that should be disabled
-#   SRC:=one or more directories to search for sources. The order of the source roots
+#   JAVAC_FLAGS javac flags to append to the default ones.
+#   JAVA_FLAGS flags to be appended to the java launching the compiler
+#   DISABLED_WARNINGS list of Xlint warnings that should be disabled
+#   SRC one or more directories to search for sources. The order of the source roots
 #        is significant. The first found file of a certain name has priority.
-#   BIN:=store classes here
-#   MODULE:=Name of module being compiled. If set, classes are put in BIN/MODULE.
-#   CLASSPATH:=a list of additional entries to set as classpath to javac
-#   INCLUDES:=myapp.foo means will only compile java files in myapp.foo or any of its sub-packages.
-#   EXCLUDES:=myapp.foo means will do not compile java files in myapp.foo or any of its sub-packages.
-#   COPY:=.prp means copy all prp files to the corresponding package in BIN.
-#   COPY_FILES:=myapp/foo/setting.txt means copy this file over to the package myapp/foo
-#   CLEAN:=.properties means copy and clean all properties file to the corresponding package in BIN.
-#   CLEAN_FILES:=myapp/foo/setting.txt means clean this file over to the package myapp/foo
-#   SRCZIP:=Create a src.zip based on the found sources and copied files.
-#   INCLUDE_FILES:="com/sun/SolarisFoobar.java" means only compile this file!
-#   EXCLUDE_FILES:="com/sun/SolarisFoobar.java" means do not compile this particular file!
+#   BIN store classes here
+#   MODULE Name of module being compiled. If set, classes are put in BIN/MODULE.
+#   CLASSPATH a list of additional entries to set as classpath to javac
+#   INCLUDES myapp.foo means will only compile java files in myapp.foo or any of its sub-packages.
+#   EXCLUDES myapp.foo means will do not compile java files in myapp.foo or any of its sub-packages.
+#   COPY .prp means copy all prp files to the corresponding package in BIN.
+#   COPY_FILES myapp/foo/setting.txt means copy this file over to the package myapp/foo
+#   CLEAN .properties means copy and clean all properties file to the corresponding package in BIN.
+#   CLEAN_FILES myapp/foo/setting.txt means clean this file over to the package myapp/foo
+#   SRCZIP Create a src.zip based on the found sources and copied files.
+#   INCLUDE_FILES "com/sun/SolarisFoobar.java" means only compile this file!
+#   EXCLUDE_FILES "com/sun/SolarisFoobar.java" means do not compile this particular file!
 #       "SolarisFoobar.java" means do not compile SolarisFoobar, wherever it is found.
-#   EXTRA_FILES:=List of extra source files to include in compilation. Can be used to
+#   EXTRA_FILES List of extra source files to include in compilation. Can be used to
 #       specify files that need to be generated by other rules first.
-#   HEADERS:=path to directory where all generated c-headers are written.
-#   DEPENDS:=Extra dependency
-#   KEEP_DUPS:=Do not remove duplicate file names from different source roots.
-#   FAIL_NO_SRC:=Set to false to not fail the build if no source files are found,
+#   HEADERS path to directory where all generated c-headers are written.
+#   DEPENDS Extra dependency
+#   KEEP_DUPS Do not remove duplicate file names from different source roots.
+#   FAIL_NO_SRC Set to false to not fail the build if no source files are found,
 #        default is true.
-#   CREATE_API_DIGEST:=Set to true to use a javac plugin to generate a public API
+#   CREATE_API_DIGEST Set to true to use a javac plugin to generate a public API
 #        hash which can be used for down stream dependencies to only rebuild
 #        when the API changes.
-#   KEEP_ALL_TRANSLATIONS:=Set to true to skip translation filtering
+#   KEEP_ALL_TRANSLATIONS Set to true to skip translation filtering
 SetupJavaCompilation = $(NamedParamsMacroTemplate)
 define SetupJavaCompilationBody
 
@@ -371,24 +371,24 @@ define SetupJavaCompilationBody
     # All files below META-INF are always copied.
     $1_ALL_COPIES := $$(filter $$(addsuffix /META-INF%, $$($1_SRC)), $$($1_ALL_SRCS))
     # Find all files to be copied from source to bin.
-    ifneq (,$$($1_COPY)$$($1_COPY_FILES))
+    ifneq ($$($1_COPY)$$($1_COPY_FILES), )
       # Search for all files to be copied.
       $1_ALL_COPIES += $$(filter $$(addprefix %, $$($1_COPY)), $$($1_ALL_SRCS))
       # Copy these explicitly
       $1_ALL_COPIES += $$($1_COPY_FILES)
     endif
     # Copy must also respect filters.
-    ifneq (,$$($1_INCLUDE_PATTERN))
+    ifneq ($$($1_INCLUDE_PATTERN), )
       $1_ALL_COPIES := $$(filter $$($1_INCLUDE_PATTERN), $$($1_ALL_COPIES))
     endif
-    ifneq (,$$($1_EXCLUDE_PATTERN))
+    ifneq ($$($1_EXCLUDE_PATTERN), )
       $1_ALL_COPIES := $$(filter-out $$($1_EXCLUDE_PATTERN), $$($1_ALL_COPIES))
     endif
     # Filter out any excluded translations
     ifneq ($$($1_KEEP_ALL_TRANSLATIONS), true)
       $1_ALL_COPIES := $$(call FilterExcludedTranslations, $$($1_ALL_COPIES), .properties)
     endif
-    ifneq (,$$($1_ALL_COPIES))
+    ifneq ($$($1_ALL_COPIES), )
       # Yep, there are files to be copied!
       $1_ALL_COPY_TARGETS :=
           $$(foreach i, $$($1_ALL_COPIES), $$(eval $$(call add_file_to_copy,$1,$$i)))
@@ -396,23 +396,23 @@ define SetupJavaCompilationBody
     endif
 
     # Find all property files to be copied and cleaned from source to bin.
-    ifneq (,$$($1_CLEAN)$$($1_CLEAN_FILES))
+    ifneq ($$($1_CLEAN)$$($1_CLEAN_FILES), )
       # Search for all files to be copied.
       $1_ALL_CLEANS := $$(filter $$(addprefix %, $$($1_CLEAN)), $$($1_ALL_SRCS))
       # Clean these explicitly
       $1_ALL_CLEANS += $$($1_CLEAN_FILES)
       # Copy and clean must also respect filters.
-      ifneq (,$$($1_INCLUDE_PATTERN))
+      ifneq ($$($1_INCLUDE_PATTERN), )
         $1_ALL_CLEANS := $$(filter $$($1_INCLUDE_PATTERN), $$($1_ALL_CLEANS))
       endif
-      ifneq (,$$($1_EXCLUDE_PATTERN))
+      ifneq ($$($1_EXCLUDE_PATTERN), )
         $1_ALL_CLEANS := $$(filter-out $$($1_EXCLUDE_PATTERN), $$($1_ALL_CLEANS))
       endif
       # Filter out any excluded translations
       ifneq ($$($1_KEEP_ALL_TRANSLATIONS), true)
         $1_ALL_CLEANS := $$(call FilterExcludedTranslations, $$($1_ALL_CLEANS), .properties)
       endif
-      ifneq (,$$($1_ALL_CLEANS))
+      ifneq ($$($1_ALL_CLEANS), )
         # Yep, there are files to be copied and cleaned!
         $1_ALL_COPY_CLEAN_TARGETS :=
             $$(foreach i, $$($1_ALL_CLEANS), $$(eval $$(call add_file_to_clean,$1,$$i)))
@@ -431,7 +431,7 @@ define SetupJavaCompilationBody
 
     # Put headers in a temp dir to filter out those that actually
     # changed before copying them to the real header dir.
-    ifneq (,$$($1_HEADERS))
+    ifneq ($$($1_HEADERS), )
       $1_HEADERS_ARG := -h $$($1_HEADERS).$1.tmp
 
       $$($1_HEADERS)/_the.$1_headers: $$($1_COMPILE_TARGET)
@@ -503,11 +503,11 @@ define SetupJavaCompilationBody
         $$($1_HEADER_TARGETS)
 
     # Check if a jar file was specified, then setup the rules for the jar.
-    ifneq (,$$($1_JAR))
+    ifneq ($$($1_JAR), )
       # If no suffixes was explicitly set for this jar file.
       # Use class and the cleaned/copied properties file suffixes as the default
       # for the types of files to be put into the jar.
-      ifeq (,$$($1_SUFFIXES))
+      ifeq ($$($1_SUFFIXES), )
         $1_SUFFIXES := .class $$($1_CLEAN) $$($1_COPY)
       endif
 
@@ -531,7 +531,7 @@ define SetupJavaCompilationBody
     endif
 
     # Check if a srczip was specified, then setup the rules for the srczip.
-    ifneq (,$$($1_SRCZIP))
+    ifneq ($$($1_SRCZIP), )
       $$(eval $$(call SetupZipArchive, ZIP_ARCHIVE_$1, \
           SRC := $$($1_SRC), \
           ZIP := $$($1_SRCZIP), \

make/common/MakeBase.gmk

@@ -23,12 +23,12 @@
 # questions.
 #
 
-################################################################
+################################################################################
 # MakeBase provides the core functionality needed and used by all makefiles. It
 # should be included by all makefiles. MakeBase provides essential
 # functionality for named parameter functions, variable dependency, tool
 # execution, logging and fixpath functionality.
-################################################################
+################################################################################
 
 ifndef _MAKEBASE_GMK
 _MAKEBASE_GMK := 1

make/common/MakeIO.gmk

@@ -23,7 +23,7 @@
 # questions.
 #
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including MakeIO.gmk)
 endif
 
@@ -82,7 +82,7 @@ else # HAS_FILE_FUNCTION = false
   # Param 2 - File to print to
   # Param 3 - Set to true to append to file instead of overwriting
   define ListPathsSafely
-    ifneq (,$$(word 30001,$$($$(strip $1))))
+    ifneq ($$(word 30001, $$($$(strip $1))), )
       $$(error Cannot list safely more than 30000 paths. $1 has $$(words $$($$(strip $1))) paths!)
     endif
     $$(call MakeDir, $$(dir $2))

make/common/ProcessMarkdown.gmk

@@ -22,7 +22,7 @@
 # questions.
 #
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including ProcessMarkdown.gmk)
 endif
 

make/common/TestFilesCompilation.gmk

@@ -26,7 +26,7 @@
 ifndef _TEST_FILES_COMPILATION_GMK
 _TEST_FILES_COMPILATION_GMK := 1
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including TestFilesCompilation.gmk)
 endif
 

make/common/TextFileProcessing.gmk

@@ -23,7 +23,7 @@
 # questions.
 #
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including TextFileProcessing.gmk)
 endif
 
@@ -104,10 +104,10 @@ define SetupTextFileProcessingBody
     $1_EXCLUDE_FILES := $$(foreach i, $$($1_SOURCE_DIRS), $$(addprefix $$i/, $$($1_EXCLUDE_FILES)))
     $1_INCLUDE_FILES := $$(foreach i, $$($1_SOURCE_DIRS), $$(addprefix $$i/, $$($1_INCLUDE_FILES)))
     $1_SOURCE_FILES := $$(filter-out $$($1_EXCLUDE_FILES), $$($1_SOURCE_FILES))
-    ifneq (,$$(strip $$($1_INCLUDE_FILES)))
+    ifneq ($$(strip $$($1_INCLUDE_FILES)), )
       $1_SOURCE_FILES := $$(filter $$($1_INCLUDE_FILES), $$($1_SOURCE_FILES))
     endif
-    ifeq (,$$($1_SOURCE_FILES))
+    ifeq ($$($1_SOURCE_FILES), )
       $$(info No sources found for $1 when looking inside the dirs $$($1_SRC))
     endif
   endif

make/common/Utils.gmk

@@ -23,7 +23,7 @@
 # questions.
 #
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including Utils.gmk)
 endif
 

make/common/ZipArchive.gmk

@@ -29,7 +29,7 @@ _ZIP_ARCHIVE_GMK := 1
 # Depends on build tools for MakeZipReproducible
 include ../ToolsJdk.gmk
 
-ifeq (,$(_MAKEBASE_GMK))
+ifeq ($(_MAKEBASE_GMK), )
   $(error You must include MakeBase.gmk prior to including ZipArchive.gmk)
 endif
 

make/devkit/Makefile

@@ -81,7 +81,7 @@ $(info target_platforms $(target_platforms))
 
 all compile : $(platforms)
 
-ifeq (,$(SKIP_ME))
+ifeq ($(SKIP_ME), )
   $(foreach p,$(filter-out $(me),$(platforms)),$(eval $(p) : $$(me)))
 endif
 

make/devkit/Tools.gmk

@@ -23,7 +23,7 @@
 # questions.
 #
 
-##########################################################################################
+################################################################################
 #
 # Workhorse makefile for creating ONE cross compiler
 # Needs either to be from BUILD -> BUILD OR have
@@ -92,7 +92,7 @@ else
   $(error Unknown base OS $(BASE_OS))
 endif
 
-##########################################################################################
+################################################################################
 # Define external dependencies
 
 # Latest that could be made to work.
@@ -223,7 +223,7 @@ RPM_LIST := \
     systemtap-sdt-devel \
     #
 
-##########################################################################################
+################################################################################
 # Define common directories and files
 
 # Ensure we have 32-bit libs also for x64. We enable mixed-mode.
@@ -258,7 +258,7 @@ download-rpms:
 	      wget -r -np -nd $(patsubst %, -A "*%*.rpm", $(RPM_LIST)) $(BASE_URL)
         endif
 
-##########################################################################################
+################################################################################
 # Unpack source packages
 
 # Generate downloading + unpacking of sources.
@@ -285,7 +285,7 @@ endef
 # Download and unpack all source packages
 $(foreach p,GCC BINUTILS CCACHE MPFR GMP MPC GDB,$(eval $(call Download,$(p))))
 
-##########################################################################################
+################################################################################
 # Unpack RPMS
 
 RPM_ARCHS := $(ARCH) noarch
@@ -324,7 +324,7 @@ endef
 
 $(foreach p,$(RPM_FILE_LIST),$(eval $(call unrpm,$(p))))
 
-##########################################################################################
+################################################################################
 
 # Note: MUST create a <sysroot>/usr/lib even if not really needed.
 # gcc will use a path relative to it to resolve lib64. (x86_64).
@@ -344,7 +344,7 @@ $(libs) : $(rpms)
 	@mkdir -p $(SYSROOT)/usr/lib
 	@touch $@
 
-##########################################################################################
+################################################################################
 # Create links for ffi header files so that they become visible by default when using the
 # devkit.
 ifeq ($(ARCH), x86_64)
@@ -357,12 +357,12 @@ ifeq ($(ARCH), x86_64)
   SYSROOT_LINKS += $(SYSROOT)/usr/include/ffi.h $(SYSROOT)/usr/include/ffitarget.h
 endif
 
-##########################################################################################
+################################################################################
 
 # Define marker files for each source package to be compiled
 $(foreach t,binutils mpfr gmp mpc gcc ccache gdb,$(eval $(t) = $(TARGETDIR)/$($(t)_ver).done))
 
-##########################################################################################
+################################################################################
 
 # Default base config
 CONFIG = --target=$(TARGET) \
@@ -390,7 +390,7 @@ endif
 
 TOOLS ?= $(call declare_tools,_FOR_TARGET,$(TARGET)-)
 
-##########################################################################################
+################################################################################
 
 # Create a TARGET bfd + libiberty only.
 # Configure one or two times depending on mulitlib arch.
@@ -424,7 +424,7 @@ $(bfdmakes) : CONFIG = --target=$(TARGET) \
 
 $(bfdmakes) : TOOLS = $(call declare_tools,_FOR_TARGET,$(TARGET)-) $(call declare_tools,,$(TARGET)-)
 
-##########################################################################################
+################################################################################
 
 $(gcc) \
     $(binutils) \
@@ -591,7 +591,7 @@ else
 	@echo 'done'
 endif
 
-##########################################################################################
+################################################################################
 # very straightforward. just build a ccache. it is only for host.
 $(BUILDDIR)/$(ccache_ver)/Makefile \
     : $(CCACHE_CFG)
@@ -606,7 +606,7 @@ $(BUILDDIR)/$(ccache_ver)/Makefile \
 
 gccpatch = $(TARGETDIR)/gcc-patched
 
-##########################################################################################
+################################################################################
 # For some reason cpp is not created as a target-compiler
 ifeq ($(HOST),$(TARGET))
   $(gccpatch) : $(gcc) link_libs
@@ -621,7 +621,7 @@ ifeq ($(HOST),$(TARGET))
 	@touch $@
 	@echo 'done'
 
-  ##########################################################################################
+  ##############################################################################
   # Ugly at best. Seems that when we compile host->host compiler, that are NOT
   # the BUILD compiler, the result will not try searching for libs in package root.
   # "Solve" this by create links from the target libdirs to where they are.
@@ -641,7 +641,7 @@ else
 	@echo 'done'
 endif
 
-##########################################################################################
+################################################################################
 # Build in two steps.
 # make <default>
 # make install.
@@ -656,7 +656,7 @@ $(TARGETDIR)/%.done : $(BUILDDIR)/%/Makefile
 	@touch $@
 	@echo 'done'
 
-##########################################################################################
+################################################################################
 
 $(PREFIX)/devkit.info:
 	@echo 'Creating devkit.info in the root of the kit'
@@ -670,7 +670,7 @@ $(PREFIX)/devkit.info:
 	echo 'DEVKIT_SYSROOT="$$DEVKIT_ROOT/$(TARGET)/sysroot"' >> $@
 	echo 'DEVKIT_EXTRA_PATH="$$DEVKIT_ROOT/bin"' >> $@
 
-##########################################################################################
+################################################################################
 # Copy these makefiles into the root of the kit
 $(PREFIX)/Makefile: ./Makefile
 	rm -rf $@
@@ -686,7 +686,7 @@ $(PREFIX)/Tars.gmk: ./Tars.gmk
 
 THESE_MAKEFILES := $(PREFIX)/Makefile $(PREFIX)/Tools.gmk $(PREFIX)/Tars.gmk
 
-##########################################################################################
+################################################################################
 
 ifeq ($(TARGET), $(HOST))
   # To build with dtrace support, the build needs access to the dtrace executable from the
@@ -706,7 +706,7 @@ ifeq ($(TARGET), $(HOST))
       ld.gold nm objcopy objdump ranlib readelf size strings strip)
 endif
 
-##########################################################################################
+################################################################################
 
 bfdlib : $(bfdlib)
 binutils : $(binutils)

make/hotspot/CopyToExplodedJdk.gmk

@@ -37,7 +37,8 @@ ifeq ($(call isTargetOs, windows), true)
   $(eval $(call SetupCopyFiles, COPY_LIBS_LIB, \
       SRC := $(SUPPORT_OUTPUTDIR)/modules_libs/java.base, \
       DEST := $(JDK_OUTPUTDIR)/lib, \
-      FILES := $(filter %.lib, $(LIB_TARGETS))))
+      FILES := $(filter %.lib, $(LIB_TARGETS)), \
+  ))
 
   TARGETS += $(COPY_LIBS_BIN) $(COPY_LIBS_LIB)
 else

make/ide/visualstudio/hotspot/CreateVSProject.gmk

@@ -80,7 +80,7 @@ ifeq ($(call isTargetOs, windows), true)
     -ignorePath zero \
       #
 
-  ################################################################################
+  ##############################################################################
   # Build the ProjectCreator java tool.
 
   TOOLS_OUTPUTDIR := $(MAKESUPPORT_OUTPUTDIR)/ide/visualstudio

make/ide/vscode/hotspot/CreateVSCodeProject.gmk

@@ -47,7 +47,7 @@ GetIndexerFragment = \
 ################################################################################
 # Show indexer-specific notes if they exist, otherwise do nothing
 ################################################################################
-ifneq (,$(wildcard $(call GetIndexerFragment,notes)))
+ifneq ($(wildcard $(call GetIndexerFragment,notes)), )
   ShowIndexerNotes = $(CAT) $(call GetIndexerFragment,notes)
 else
   ShowIndexerNotes =

make/modules/java.base/gensrc/GensrcModuleLoaderMap.gmk

@@ -42,7 +42,7 @@ NATIVE_ACCESS_MODULES_LIST := $(call SubstComma, $(NATIVE_ACCESS_MODULES))
 VARDEPS_VALUE := $(BOOT_MODULES_LIST) $(PLATFORM_MODULES_LIST) $(NATIVE_ACCESS_MODULES_LIST)
 VARDEPS_FILE := $(call DependOnVariable, VARDEPS_VALUE)
 
-############################################################################
+################################################################################
 
 $(SUPPORT_OUTPUTDIR)/gensrc/java.base/jdk/internal/module/ModuleLoaderMap.java: \
     $(TOPDIR)/src/java.base/share/classes/jdk/internal/module/ModuleLoaderMap.java \

make/modules/java.desktop/lib/ClientLibraries.gmk

@@ -101,7 +101,8 @@ TARGETS += $(BUILD_LIBLCMS)
 ifeq ($(USE_EXTERNAL_LIBJPEG), true)
   LIBJAVAJPEG_INCLUDE_FILES := \
       imageioJPEG.c \
-      jpegdecoder.c
+      jpegdecoder.c \
+      #
   # If we're using an external library, we can't include our own SRC path
   # as includes, instead the system headers should be used.
   LIBJAVAJPEG_HEADERS_FROM_SRC := false

make/test/JtregNativeHotspot.gmk

@@ -59,77 +59,93 @@ VM_TESTBASE_DIR := $(TOPDIR)/test/hotspot/jtreg/vmTestbase
 VM_SHARE_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/vm/share \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jni
+    -I$(VM_TESTBASE_DIR)/nsk/share/jni \
+    #
 
 NSK_MONITORING_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jni
+    -I$(VM_TESTBASE_DIR)/nsk/share/jni \
+    #
 
 NSK_JDI_INCLUDES := \
-    -I$(VM_TESTBASE_DIR)/nsk/share/native
+    -I$(VM_TESTBASE_DIR)/nsk/share/native \
+    #
 
 NSK_SHARE_JDI_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jdi
+    -I$(VM_TESTBASE_DIR)/nsk/share/jdi \
+    #
 
 NSK_SHARE_JNI_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jni
+    -I$(VM_TESTBASE_DIR)/nsk/share/jni \
+    #
 
 RUNTIME_DEFMETH_INCLUDES := \
     -I$(TOPDIR)/test/hotspot/jtreg/vmTestbase/nsk/share/jni \
     -I$(TOPDIR)/test/hotspot/jtreg/vmTestbase/nsk/share/native \
-    -I$(TOPDIR)/test/hotspot/jtreg/vmTestbase/nsk/share/jvmti
+    -I$(TOPDIR)/test/hotspot/jtreg/vmTestbase/nsk/share/jvmti \
+    #
 
 NSK_SHARE_LOCKS_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/locks
+    -I$(VM_TESTBASE_DIR)/nsk/share/locks \
+    #
 
 MLVM_JVMTI_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
     -I$(VM_TESTBASE_DIR)/nsk/share/jni \
     -I$(VM_TESTBASE_DIR)/nsk/share/jvmti \
     -I$(VM_TESTBASE_DIR)/nsk/share/jvmti/agent_common \
-    -I$(VM_TESTBASE_DIR)/vm/mlvm/share
+    -I$(VM_TESTBASE_DIR)/vm/mlvm/share \
+    #
 
 MLVM_STRESS_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jni
+    -I$(VM_TESTBASE_DIR)/nsk/share/jni \
+    #
 
 NSK_GC_LOCK_JNI_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/gc/lock/jni \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jni
+    -I$(VM_TESTBASE_DIR)/nsk/share/jni \
+    #
 
 NSK_GC_LOCK_REF_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/gc/lock/jniref \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jni
+    -I$(VM_TESTBASE_DIR)/nsk/share/jni \
+    #
 
 NSK_STRACE_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jni
+    -I$(VM_TESTBASE_DIR)/nsk/share/jni \
+    #
 
 NSK_STRESS_JNI_INCLUDES := \
-    -I$(VM_TESTBASE_DIR)/nsk/stress/jni
+    -I$(VM_TESTBASE_DIR)/nsk/stress/jni \
+    #
 
 NSK_JVMTI_AGENT_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/jni \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
     -I$(VM_TESTBASE_DIR)/nsk/share/jvmti \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jvmti/agent_common
+    -I$(VM_TESTBASE_DIR)/nsk/share/jvmti/agent_common \
+    #
 
 NSK_JVMTI_AOD_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/jni \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
     -I$(VM_TESTBASE_DIR)/nsk/share/aod \
     -I$(VM_TESTBASE_DIR)/nsk/share/jvmti \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jvmti/aod
+    -I$(VM_TESTBASE_DIR)/nsk/share/jvmti/aod \
+    #
 
 NSK_AOD_INCLUDES := \
     -I$(VM_TESTBASE_DIR)/nsk/share/aod \
     -I$(VM_TESTBASE_DIR)/nsk/share/native \
-    -I$(VM_TESTBASE_DIR)/nsk/share/jni
+    -I$(VM_TESTBASE_DIR)/nsk/share/jni \
+    #
 
 NO_FRAMEPOINTER_CFLAGS :=
 ifeq ($(call isTargetOs, linux), true)

make/test/JtregNativeJdk.gmk

@@ -53,7 +53,8 @@ BUILD_JDK_JTREG_IMAGE_DIR := $(TEST_IMAGE_DIR)/jdk/jtreg
 BUILD_JDK_JTREG_EXECUTABLES_CFLAGS_exeJliLaunchTest := \
     -I$(TOPDIR)/src/java.base/share/native/libjli \
     -I$(TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS_TYPE)/native/libjli \
-    -I$(TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/native/libjli
+    -I$(TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/native/libjli \
+    #
 
 BUILD_JDK_JTREG_EXECUTABLES_JDK_LIBS_exeJliLaunchTest := java.base:libjli
 BUILD_JDK_JTREG_EXECUTABLES_JDK_LIBS_exeCallerAccessTest := java.base:libjvm