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