Yes it is the only (non-test) source i could find in hotspot, but i want to change it in all openjdk sources i can find it. I thought i really must discuss it part by part in the mailing-lists. Actually i am working on the issue to save against regression on this. Thanks Alexandr for this input. So there will be something that integrates into jtreg for this too.
-- Sebastian On 09/30/2015 01:11 AM, Christian Thalinger wrote: > Thanks for volunteering to look into such old bugs! > > Is this the only occurrence of that pattern in the hotspot repository? If > you want to fix this in hotspot only without the jdk changes described in the > RFE (I just changed the type from Bug to Enhancement) then we have to file a > new RFE because we can’t close the original one. > >> On Sep 27, 2015, at 6:58 AM, Sebastian Sickelmann >> <sebastian.sickelm...@gmx.de> wrote: >> >> Hello, >> >> my name is Sebastian Sickelmann and i signed the OCA. >> >> Actually I am searching through the JBS for low hanging fruits. >> Right now i am looking through the openjdk-sources and try to evaluate >> if i can make something about JDK-5108778. >> >> As I am not an author, I am actually not able to host webrevs on >> cr.openjdk.java.net. >> >> Is there someone who would support at hosting the hotspot-part of >> JDK-5108778 on cr.openjdk.java.net for reviewing? >> >> I placed the hotspot part in my dropbox at: >> https://dl.dropboxusercontent.com/u/43692695/oss-patches/openjdk/jdk-5108778/hotspot_0/webrev/index.html >> or as zip: >> https://dl.dropboxusercontent.com/u/43692695/oss-patches/openjdk/jdk-5108778/hotspot_0/webrev.zip >> >> I executed all jtreg tests in hotspot/test and get no more errors than >> before the change. >> >> -- Sebastian >> >
