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 2010/05/19 02:27:31 UTC

svn commit: r945972 - in /trafficserver/traffic/trunk: libinktomi++/Compatability.h proxy/mgmt2/preparse/StoreReadConfig.cc proxy/mgmt2/utils/WebMgmtUtils.cc

Author: jplevyak
Date: Wed May 19 00:27:31 2010
New Revision: 945972

URL: http://svn.apache.org/viewvc?rev=945972&view=rev
Log:
TS-358: remove more cruft: ink_sscan_longlong, bcopy, in_addr_t

Modified:
    trafficserver/traffic/trunk/libinktomi++/Compatability.h
    trafficserver/traffic/trunk/proxy/mgmt2/preparse/StoreReadConfig.cc
    trafficserver/traffic/trunk/proxy/mgmt2/utils/WebMgmtUtils.cc

Modified: trafficserver/traffic/trunk/libinktomi++/Compatability.h
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/libinktomi%2B%2B/Compatability.h?rev=945972&r1=945971&r2=945972&view=diff
==============================================================================
--- trafficserver/traffic/trunk/libinktomi++/Compatability.h (original)
+++ trafficserver/traffic/trunk/libinktomi++/Compatability.h Wed May 19 00:27:31 2010
@@ -27,6 +27,9 @@
 #include <sys/types.h>
 #include <sys/uio.h>
 #include <unistd.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <strings.h>
 
 #include "ink_port.h"
 #include "ink_resource.h"
@@ -63,42 +66,12 @@ template<class T> T max(const T a, const
 #define MAP_SHARED_MAP_NORESERVE (MAP_SHARED | MAP_NORESERVE)
 #endif
 
-#if (HOST_OS == linux)
-#ifndef __x86_64
-typedef unsigned long long uint64_t;
-#endif
-#endif
-
-#if ((HOST_OS == freebsd) || (HOST_OS == darwin))
+#if (HOST_OS == darwin)
 typedef uint32_t in_addr_t;
 #endif
 
 #define NEED_HRTIME
 
-#include <stdio.h>
-
-#if (HOST_OS == freebsd)
-static inline int
-ink_sscan_longlong(const char *str, ink64 * value)
-{
-  return sscanf(str, "%qd", value);
-}
-#else
-static inline int
-ink_sscan_longlong(const char *str, ink64 * value)
-{
-  // coverity[secure_coding]
-  return sscanf(str, "%lld", value);
-}
-#endif
-
-#if (HOST_OS == linux)
-#define BCOPY_TYPE (char *)
-#else
-#define BCOPY_TYPE (void *)
-extern "C" void bcopy(const void *s1, void *s2, size_t n);
-#endif
-
 #define ink_pread      pread
 #define ink_pwrite     pwrite
 #define ink_open       open

Modified: trafficserver/traffic/trunk/proxy/mgmt2/preparse/StoreReadConfig.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/proxy/mgmt2/preparse/StoreReadConfig.cc?rev=945972&r1=945971&r2=945972&view=diff
==============================================================================
--- trafficserver/traffic/trunk/proxy/mgmt2/preparse/StoreReadConfig.cc (original)
+++ trafficserver/traffic/trunk/proxy/mgmt2/preparse/StoreReadConfig.cc Wed May 19 00:27:31 2010
@@ -59,7 +59,7 @@ parseStorageFile(int fd)
     while (*e && !isdigit(*e))
       e++;
     if (e && *e) {
-      if (1 != ink_sscan_longlong(e, &size)) {
+      if (1 != sscanf(e, "%lld", &size)) {
         err = "error parsing size";
         goto Lfail;
       }

Modified: trafficserver/traffic/trunk/proxy/mgmt2/utils/WebMgmtUtils.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/proxy/mgmt2/utils/WebMgmtUtils.cc?rev=945972&r1=945971&r2=945972&view=diff
==============================================================================
--- trafficserver/traffic/trunk/proxy/mgmt2/utils/WebMgmtUtils.cc (original)
+++ trafficserver/traffic/trunk/proxy/mgmt2/utils/WebMgmtUtils.cc Wed May 19 00:27:31 2010
@@ -76,21 +76,21 @@ varSetFromStr(const char *varName, const
 
   switch (varDataType) {
   case RECD_INT:
-    if (ink_sscan_longlong(value, &data.rec_int) == 1) {
+    if (sscanf(value, "%lld", &data.rec_int) == 1) {
       RecSetRecordInt((char *) varName, data.rec_int);
     } else {
       found = false;
     }
     break;
   case RECD_LLONG:
-    if (ink_sscan_longlong(value, &data.rec_llong) == 1) {
+    if (sscanf(value, "%lld", &data.rec_llong) == 1) {
       RecSetRecordLLong((char *) varName, data.rec_llong);
     } else {
       found = false;
     }
     break;
   case RECD_COUNTER:
-    if (ink_sscan_longlong(value, &data.rec_counter) == 1) {
+    if (sscanf(value, "%lld", &data.rec_counter) == 1) {
       RecSetRecordCounter((char *) varName, data.rec_counter);
     } else {
       found = false;