RFR: 8058470 [jconsole] VM Summary Tab is blank for JDK9's jconsole.

[Erik Joelsson]

Hello,

The specification for the properties file format says that a comment is 
a line that has either a ! or # as the first non whitespace character. 
Greping around in the source shows we have several instances if comments 
tarting a few spaces in. I don't think we are using ! for comments 
anywhere though. Using this expression seems to do the trick for me:

's/^[         ]*#.*/#/g'

Note that the contents of the [] is one space and a literal tab as \t is 
not portable to all versions of sed that we use.

The cleaning mechanism is hard to make really safe without semantic 
parsing. Have you considered using the CompileProperties option instead? 
For most of the resource bundles in the jdk, we convert the properties 
into java src. As I understand it, resource bundles work seamlessly as 
classes or properties files. I assume this is done for runtime 
performance. You could also choose to just copy the properties files 
instead of cleaning them, especially if there are no or few comments in 
them.

/Erik

On 2015-03-04 20:21, Staffan Larsen wrote:
> The problem is that the makefiles do "cleanup" of the resource files, accidentally deleting half of some strings. In this case
>
> GC_INFO=Name = ''{0}'', Collections = {1,choice,-1#Unavailable|0#{1,number,integer}}, Total time spent = {2}
>
> becomes
>
> GC_INFO=Name \= ''{0}'', Collections \= {1,choice,-1#
>
> The below diff fixes the problem. I added an extra ^ before the # so that only # at the beginning of the line are treated as comments. I don’t know if this has other implications, though? Should # anywhere on the line be treated as the beginning of a comment? In that case we need to handle the jconsole resource file specially.
>
> Thanks,
> /Staffan
>
>
> diff --git a/make/common/JavaCompilation.gmk b/make/common/JavaCompilation.gmk
> --- a/make/common/JavaCompilation.gmk
> +++ b/make/common/JavaCompilation.gmk
> @@ -393,7 +393,7 @@
>          $(MKDIR) -p $$(@D)
>          export LC_ALL=C ; ( $(CAT) $$< && $(ECHO) "" ) \
>              | $(SED) -e 's/\([^\\]\):/\1\\:/g' -e 's/\([^\\]\)=/\1\\=/g' \
> - -e 's/\([^\\]\)!/\1\\!/g' -e 's/#.*/#/g' \
> + -e 's/\([^\\]\)!/\1\\!/g' -e 's/^#.*/#/g' \
>              | $(SED) -f "$(SRC_ROOT)/make/common/support/unicode2x.sed" \
>              | $(SED) -e '/^#/d' -e '/^$$$$/d' \
>                  -e :a -e '/\\$$$$/N; s/\\\n//; ta' \