You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by jp...@apache.org on 2012/05/22 06:14:31 UTC

git commit: Fix the Apple/clang-318.0.45 build

Updated Branches:
  refs/heads/master b57523bdb -> 38d8b6a14


Fix the Apple/clang-318.0.45 build

This was the error I was seeing:

HttpSM.cc:91:13: error: variable 'bound' is not needed and will not be emitted
      [-Werror,-Wunneeded-internal-declaration]
      static char bound[] = "RANGE_SEPARATOR";

Fixed with a sprinkle of const.


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/38d8b6a1
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/38d8b6a1
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/38d8b6a1

Branch: refs/heads/master
Commit: 38d8b6a14a7f9356a30be972177ae133ae6d9fce
Parents: b57523b
Author: James Peach <jp...@apache.org>
Authored: Mon May 21 21:13:07 2012 -0700
Committer: James Peach <jp...@apache.org>
Committed: Mon May 21 21:13:15 2012 -0700

----------------------------------------------------------------------
 proxy/http/HttpSM.cc |   11 +++++------
 1 files changed, 5 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/38d8b6a1/proxy/http/HttpSM.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 2622fa4..af64010 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -88,12 +88,11 @@ static uint32_t val[MAX_SCATTER_LEN];
 static uint16_t to[MAX_SCATTER_LEN];
 static int scat_count = 0;
 
-static char bound[] = "RANGE_SEPARATOR";
-//
-static char cont_type[] = "Content-type: ";
-static char cont_range[] = "Content-range: bytes ";
-static int sub_header_size = sizeof(cont_type) - 1 + 2 + sizeof(cont_range) - 1 + 4;
-static int boundary_size = 2 + sizeof(bound) - 1 + 2;
+static const char bound[] = "RANGE_SEPARATOR";
+static char const cont_type[] = "Content-type: ";
+static char const cont_range[] = "Content-range: bytes ";
+static const int sub_header_size = sizeof(cont_type) - 1 + 2 + sizeof(cont_range) - 1 + 4;
+static const int boundary_size = 2 + sizeof(bound) - 1 + 2;
 
 /**
  * Takes two milestones and returns the difference.