RFR (XS): JDK-8058534: Remove HeapRegion::_orig_end

Thomas Schatzl thomas.schatzl at oracle.com
Fri Sep 19 07:28:03 UTC 2014


Hi,

On Fri, 2014-09-19 at 09:20 +0200, Bengt Rutisson wrote:
> Hi Joe,
> 
> Latest webrev looks good to me.
> 
> I can sponsor this and push it through JPRT. I'll fix Kim's comment 
> about the missing space before I push. Nice catch, Kim!

  can you fix the indentation of the error message too before pushing?
I.e. align the arguments to the opening bracket?

Thanks,
  Thomas

> 
> Bengt
> 
> On 9/19/14 1:25 AM, Kim Barrett wrote:
> > On Sep 18, 2014, at 6:43 PM, Joseph Provino <joseph.provino at oracle.com> wrote:
> >> Kim, thanks.  I made that change you suggested.
> >>
> >> Webrev:  http://cr.openjdk.java.net/~jprovino/8058534/webrev.01/
> > Sorry, I missed some problems with the assert message added in HeapRegion::initialize()
> > gc_implementation/g1/heapRegion.cpp, lines 356-9.
> >
> > There’s a pair of strings to break a source line:
> >     " should match exactly”
> >     "bottom plus …”
> > missing a space at the end of the first or beginning of second, else output contains “… exactlybottom …”.
> >
> > String literals immediately followed by identifiers, with no intervening spaces, will run afoul of C++11 user-defined literals.  Although gcc and clang provide CLAs to control C++11 features, my understanding is that MS compiler is mostly rolling C++11 features in as implemented in new versions, generally without such controls.  As a result, I would expect such usage to fail to compile on some (possibly still in the future) versions.  Suggest putting spaces between string literals and macro identifiers.
> >
> >
> 





More information about the hotspot-gc-dev mailing list