This is an automated email from the ASF dual-hosted git repository.

gancho pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 6565396  HdrHeap default size unit test fix.
6565396 is described below

commit 65653961072d9e6d56570e5a744b7499ee6b1b22
Author: Gancho Tenev <[email protected]>
AuthorDate: Mon Feb 25 15:30:46 2019 -0800

    HdrHeap default size unit test fix.
    
    After refactoring "HdrHeap refresh" in PR #4953 unit test need
    to be updated as well. Changing HDR_HEAP_DEFAULT_SIZE to
    HdrHeap::DEFAULT_SIZE
---
 doc/developer-guide/core-architecture/heap.en.rst | 4 ++--
 proxy/hdrs/unit_tests/test_Hdrs.cc                | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/doc/developer-guide/core-architecture/heap.en.rst 
b/doc/developer-guide/core-architecture/heap.en.rst
index e03b5ad..31bee10 100644
--- a/doc/developer-guide/core-architecture/heap.en.rst
+++ b/doc/developer-guide/core-architecture/heap.en.rst
@@ -132,10 +132,10 @@ Classes
 
 .. function:: HdrHeap * new_HdrHeap(int n)
 
-   Create and return a new instance of :class:`HdrHeap`. If :arg:`n` is less 
than ``HDR_HEAP_DEFAULT_SIZE``
+   Create and return a new instance of :class:`HdrHeap`. If :arg:`n` is less 
than ``HdrHeap::DEFAULT_SIZE``
    it is increased to that value.
 
-   If the allocated size is ``HDR_HEAP_DEFAULT_SIZE`` (or smaller and upsized 
to that value) then
+   If the allocated size is ``HdrHeap::DEFAULT_SIZE`` (or smaller and upsized 
to that value) then
    the instance is allocated from a thread local pool via 
:code:`hdrHeapAllocator`. If larger it
    is allocated from global memory via :code:`ats_malloc`.
 
diff --git a/proxy/hdrs/unit_tests/test_Hdrs.cc 
b/proxy/hdrs/unit_tests/test_Hdrs.cc
index a43503a..790418c 100644
--- a/proxy/hdrs/unit_tests/test_Hdrs.cc
+++ b/proxy/hdrs/unit_tests/test_Hdrs.cc
@@ -68,7 +68,7 @@ TEST_CASE("HdrTest", "[proxy][hdrtest]")
 
   for (auto const &test : tests) {
     HTTPHdr req_hdr;
-    HdrHeap *heap = new_HdrHeap(HDR_HEAP_DEFAULT_SIZE + 64); // extra to 
prevent proxy allocation.
+    HdrHeap *heap = new_HdrHeap(HdrHeap::DEFAULT_SIZE + 64); // extra to 
prevent proxy allocation.
 
     req_hdr.create(HTTP_TYPE_REQUEST, heap);
 

Reply via email to