RFR: JDK-8223309: test failing due to self-assign-overloaded

Erik Joelsson erik.joelsson at oracle.com
Fri May 3 15:02:10 UTC 2019


Hello,

Change looks good, but I would prefer a line break. We try to keep the 
lines in the makefiles/build system at least close to 80 chars when 
possible. No need for new webrev if you make this change.

/Erik

On 2019-05-03 07:44, Johan Vos wrote:
> Using MacOS X 10.14.4 and Xcode 10.2.1 I could not build jdk/jdk anymore:
> /Users/johan/openjdk/jdk/test/hotspot/gtest/classfile/test_symbolTable.cpp:62:6:
> error: explicitly assigning value of variable of type 'TempNewSymbol' to
> itself [-Werror,-Wself-assign-overloaded]
>    s1 = s1; // self assignment
>    ~~ ^ ~~
> 1 error generated.
>
> I added an entry to the list of disabled warnings, and that fixed the
> problem.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8223309
>
> Webrev: http://cr.openjdk.java.net/~jvos/8223309/webrev.00/
>
> - Johan



More information about the build-dev mailing list