From d0c283254bdad8d303422140a059190eeadc2cc7 Mon Sep 17 00:00:00 2001 From: "Philipp A. Hartmann" Date: Thu, 30 Oct 2014 18:21:27 +0100 Subject: [PATCH] Stack<>: add _ suffix to member ownAllocator_ For consistency, all member variables should follow the same pattern. --- include/rapidjson/internal/stack.h | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/include/rapidjson/internal/stack.h b/include/rapidjson/internal/stack.h index fbf05b4d5..24ad4bc9a 100644 --- a/include/rapidjson/internal/stack.h +++ b/include/rapidjson/internal/stack.h @@ -35,23 +35,23 @@ class Stack { public: // Optimization note: Do not allocate memory for stack_ in constructor. // Do it lazily when first Push() -> Expand() -> Resize(). - Stack(Allocator* allocator, size_t stackCapacity) : allocator_(allocator), ownAllocator(0), stack_(0), stackTop_(0), stackEnd_(0), initialCapacity_(stackCapacity) { + Stack(Allocator* allocator, size_t stackCapacity) : allocator_(allocator), ownAllocator_(0), stack_(0), stackTop_(0), stackEnd_(0), initialCapacity_(stackCapacity) { RAPIDJSON_ASSERT(stackCapacity > 0); if (!allocator_) - ownAllocator = allocator_ = RAPIDJSON_NEW(Allocator()); + ownAllocator_ = allocator_ = RAPIDJSON_NEW(Allocator()); } #if RAPIDJSON_HAS_CXX11_RVALUE_REFS Stack(Stack&& rhs) : allocator_(rhs.allocator_), - ownAllocator(rhs.ownAllocator), + ownAllocator_(rhs.ownAllocator_), stack_(rhs.stack_), stackTop_(rhs.stackTop_), stackEnd_(rhs.stackEnd_), initialCapacity_(rhs.initialCapacity_) { rhs.allocator_ = 0; - rhs.ownAllocator = 0; + rhs.ownAllocator_ = 0; rhs.stack_ = 0; rhs.stackTop_ = 0; rhs.stackEnd_ = 0; @@ -70,14 +70,14 @@ class Stack { Destroy(); allocator_ = rhs.allocator_; - ownAllocator = rhs.ownAllocator; + ownAllocator_ = rhs.ownAllocator_; stack_ = rhs.stack_; stackTop_ = rhs.stackTop_; stackEnd_ = rhs.stackEnd_; initialCapacity_ = rhs.initialCapacity_; rhs.allocator_ = 0; - rhs.ownAllocator = 0; + rhs.ownAllocator_ = 0; rhs.stack_ = 0; rhs.stackTop_ = 0; rhs.stackEnd_ = 0; @@ -162,7 +162,7 @@ class Stack { void Destroy() { Allocator::Free(stack_); - RAPIDJSON_DELETE(ownAllocator); // Only delete if it is owned by the stack + RAPIDJSON_DELETE(ownAllocator_); // Only delete if it is owned by the stack } // Prohibit copy constructor & assignment operator. @@ -170,7 +170,7 @@ class Stack { Stack& operator=(const Stack&); Allocator* allocator_; - Allocator* ownAllocator; + Allocator* ownAllocator_; char *stack_; char *stackTop_; char *stackEnd_;