Follow up on 8003259

Coleen Phillimore coleen.phillimore at oracle.com
Tue Nov 20 14:28:33 PST 2012


Hi, I finally have a 4.7 compiler and saw these myself.   I will take 
care of this.
thanks,
Coleen

On 11/19/2012 10:46 AM, Gilles Duboscq wrote:
> Hello,
>
> It looks like a few more "this->" are needed to fully fix CR 8003259.
> This is needed to build a debug/fastdebug version with GCC 4.7.2.
>
> Regards,
> Gilles
>
> diff -r 01684f7fee1b src/share/vm/memory/binaryTreeDictionary.cpp
> --- a/src/share/vm/memory/binaryTreeDictionary.cpp ven. nov. 16 09:36:41
> 2012 -0800
> +++ b/src/share/vm/memory/binaryTreeDictionary.cpp lun. nov. 19 16:35:30
> 2012 +0100
> @@ -290,7 +290,7 @@
>     assert(chunk->list() == this, "list should be set for chunk");
>     assert(tail() != NULL, "The tree list is embedded in the first chunk");
>     // which means that the list can never be empty.
> -  assert(!verify_chunk_in_free_list(chunk), "Double entry");
> +  assert(!this->verify_chunk_in_free_list(chunk), "Double entry");
>     assert(head() == NULL || head()->prev() == NULL, "list invariant");
>     assert(tail() == NULL || tail()->next() == NULL, "list invariant");
>
> @@ -300,7 +300,7 @@
>
>     assert(!tail() || size() == tail()->size(), "Wrong sized chunk in list");
>     FreeList_t<Chunk_t>::increment_count();
> -  debug_only(increment_returned_bytes_by(chunk->size()*sizeof(HeapWord));)
> +
>   debug_only(this->increment_returned_bytes_by(chunk->size()*sizeof(HeapWord));)
>     assert(head() == NULL || head()->prev() == NULL, "list invariant");
>     assert(tail() == NULL || tail()->next() == NULL, "list invariant");
>   }
> @@ -314,7 +314,7 @@
>     assert(chunk->list() == this, "list should be set for chunk");
>     assert(head() != NULL, "The tree list is embedded in the first chunk");
>     assert(chunk != NULL, "returning NULL chunk");
> -  assert(!verify_chunk_in_free_list(chunk), "Double entry");
> +  assert(!this->verify_chunk_in_free_list(chunk), "Double entry");
>     assert(head() == NULL || head()->prev() == NULL, "list invariant");
>     assert(tail() == NULL || tail()->next() == NULL, "list invariant");
>
> @@ -328,7 +328,7 @@
>     head()->link_after(chunk);
>     assert(!head() || size() == head()->size(), "Wrong sized chunk in list");
>     FreeList_t<Chunk_t>::increment_count();
> -  debug_only(increment_returned_bytes_by(chunk->size()*sizeof(HeapWord));)
> +
>   debug_only(this->increment_returned_bytes_by(chunk->size()*sizeof(HeapWord));)
>     assert(head() == NULL || head()->prev() == NULL, "list invariant");
>     assert(tail() == NULL || tail()->next() == NULL, "list invariant");
>   }



More information about the hotspot-dev mailing list