Reviewed by Maciej, Alexey, and Eric.
authorstaikos <staikos@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 May 2006 04:15:21 +0000 (04:15 +0000)
committerstaikos <staikos@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 May 2006 04:15:21 +0000 (04:15 +0000)
 - Use /**/ in .c files to compile with non-C99 and non-GCC compilers.
 - Change include to <wtf/HashTraits.h> from "HashTraits.h" to avoid -I
 - Use correct parentheses and correct mask for utf-32 support.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@14457 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/testkjs.cpp
JavaScriptCore/pcre/pcre_compile.c
JavaScriptCore/pcre/pcre_exec.c
JavaScriptCore/pcre/pcre_get.c
JavaScriptCore/wtf/UnusedParam.h
JavaScriptCore/wtf/unicode/qt4/UnicodeQt4.h

index b45d4fe..0a811e5 100644 (file)
@@ -1,3 +1,19 @@
+2006-05-17  George Staikos <staikos@kde.org>
+
+        Reviewed by Maciej, Alexey, and Eric.
+
+        * pcre/pcre_compile.c:
+        * pcre/pcre_get.c:
+        * pcre/pcre_exec.c:
+        * wtf/UnusedParam.h:
+        Use /**/ in .c files to compile with non-C99 and non-GCC compilers.
+
+        * kjs/testkjs.cpp:
+        Change include to <wtf/HashTraits.h> from "HashTraits.h" to avoid -I
+
+        * wtf/unicode/qt4/UnicodeQt4.h:
+        Use correct parentheses and correct mask for utf-32 support.
+
 2006-05-17  Alexey Proskuryakov  <ap@nypop.com>
 
         Reviewed by Darin.
index e6fbe07..0de94a2 100644 (file)
@@ -25,7 +25,7 @@
 #include "config.h"
 #include "collector.h"
 
-#include "HashTraits.h"
+#include <wtf/HashTraits.h>
 #include "JSLock.h"
 #include "object.h"
 
index 7ad7b89..c30b23c 100644 (file)
@@ -46,7 +46,7 @@ supporting internal functions that are not used by other modules. */
 #include "pcre_internal.h"
 
 
-// WARNING: These macros evaluate their parameters more than once.
+/* WARNING: These macros evaluate their parameters more than once. */
 #if PCRE_UTF16
 #define CTYPES(cd, x) ((x) <= 255 ? (cd)->ctypes[(x)] : 0)
 #define DIGITAB(x) ((x) <= 255 ? digitab[(x)] : 0)
@@ -2928,7 +2928,7 @@ for (;; ptr++)
           while (*ptr++ != '>');
           namelen = ptr - name - 1;
 
-          // FIXME: This won't work for UTF-16.
+          /* FIXME: This won't work for UTF-16. */
           for (i = 0; i < cd->names_found; i++)
             {
             int crc = memcmp(name, slot+2, namelen);
index 061013a..aac2a34 100644 (file)
@@ -1951,7 +1951,7 @@ for (;;)
       length = 1;
       GETUTF8CHARLEN(fc, ecode, length);
       {
-      int utf16Length; // don't initialize on this line as workaround for Win32 compile problem
+      int utf16Length; /* don't initialize on this line as workaround for Win32 compile problem */
       utf16Length = fc > 0xFFFF ? 2 : 1;
       if (min * utf16Length > md->end_subject - eptr) RRETURN(MATCH_NOMATCH);
       ecode += length;
index 95a4152..bddcdfd 100644 (file)
@@ -65,7 +65,7 @@ Returns:      the number of the named parentheses, or a negative number
 int
 pcre_get_stringnumber(const pcre *code, const pcre_char *stringname)
 {
-// FIXME: This doesn't work for UTF-16 because the name table has 8-bit characters in it!
+/* FIXME: This doesn't work for UTF-16 because the name table has 8-bit characters in it! */
 #if !PCRE_UTF16
 int rc;
 int entrysize;
index 968184c..ff4b9d9 100644 (file)
@@ -1,4 +1,4 @@
-// -*- mode: c++; c-basic-offset: 4 -*-
+/* -*- mode: c++; c-basic-offset: 4 -*- */
 /*
  *  Copyright (C) 2006 Apple Computer, Inc.
  *
@@ -22,9 +22,9 @@
 #ifndef KXMLCORE_UNUSED_PARAM
 #define KXMLCORE_UNUSED_PARAM
 
-// don't use this for C++, it should only be used in plain C files or
-// ObjC methods, where leaving off the parameter name is not allowed.
+/* don't use this for C++, it should only be used in plain C files or
+   ObjC methods, where leaving off the parameter name is not allowed. */
 
 #define UNUSED_PARAM(x) (void)x
 
-#endif // KXMLCORE_UNUSED_PARM
+#endif
index 140e58f..c51f662 100644 (file)
@@ -28,7 +28,7 @@
 
 #include "../UnicodeCategory.h"
 
-namespace KXMLCore {
+namespace WTF {
   namespace Unicode {
     inline int toLower(uint16_t* str, int strLength, uint16_t*& destIfNeeded)
     {
@@ -52,20 +52,20 @@ namespace KXMLCore {
 
     inline bool isFormatChar(int32_t c)
     {
-      return (c & 0xFFFF == 0) && (QChar(c).category() == QChar::Other_Format);
+      return (c & 0xffff0000) == 0 && QChar((unsigned short)c).category() == QChar::Other_Format;
     }
 
     inline bool isSeparatorSpace(int32_t c)
     {
-      return (c & 0xFFFF == 0) && (QChar(c).category() == QChar::Separator_Space);
+      return (c & 0xffff0000) == 0 && QChar((unsigned short)c).category() == QChar::Separator_Space;
     }
 
     inline CharCategory category(int32_t c)
     {
       // FIXME: implement support for non-BMP code points
-      if (c & 0xFFFF != 0)
+      if ((c & 0xffff0000) != 0)
         return NoCategory;
-        
+
       switch (QChar((unsigned short)c).category()) {
         case QChar::Mark_NonSpacing:
           return Mark_NonSpacing;