2007-11-17 Alp Toker <alp@atoker.com>
authoralp@webkit.org <alp@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 17 Nov 2007 23:14:40 +0000 (23:14 +0000)
committeralp@webkit.org <alp@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 17 Nov 2007 23:14:40 +0000 (23:14 +0000)
        Reviewed by Eric.

        http://bugs.webkit.org/show_bug.cgi?id=16032
        JS minidom is not portable

        Use a plain UTF-8 string instead of a CFString.

        Print to stdout, not stderr like CFShow() would have done, since that
        behaviour seems unintentional.

        * API/minidom.c:
        (main):

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

JavaScriptCore/API/minidom.c
JavaScriptCore/ChangeLog

index 94160785234a0fa88b2bdd9b95117089fe912c39..0702fd018df3b444ee4398a8080374003b87d4e4 100644 (file)
@@ -1,6 +1,7 @@
 // -*- mode: c++; c-basic-offset: 4 -*-
 /*
  * Copyright (C) 2006 Apple Computer, Inc.  All rights reserved.
+ * Copyright (C) 2007 Alp Toker <alp@atoker.com>
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -25,6 +26,8 @@
  */
 
 #include "JSNode.h"
+#include <stdio.h>
+#include <stdlib.h>
 #include <wtf/Assertions.h>
 #include <wtf/UnusedParam.h>
 
@@ -56,9 +59,11 @@ int main(int argc, char* argv[])
     else {
         printf("FAIL: Test script threw exception:\n");
         JSStringRef exceptionIString = JSValueToStringCopy(context, exception, NULL);
-        CFStringRef exceptionCF = JSStringCopyCFString(kCFAllocatorDefault, exceptionIString);
-        CFShow(exceptionCF);
-        CFRelease(exceptionCF);
+        size_t exceptionUTF8Size = JSStringGetMaximumUTF8CStringSize(exceptionIString);
+        char* exceptionUTF8 = (char*)malloc(exceptionUTF8Size);
+        JSStringGetUTF8CString(exceptionIString, exceptionUTF8, exceptionUTF8Size);
+        printf("%s\n", exceptionUTF8);
+        free(exceptionUTF8);
         JSStringRelease(exceptionIString);
     }
     JSStringRelease(script);
index 4b1403f04e82296db26827e2aa2d338aa45b2fb4..67bed178fd47beaadd346376cbef4e745864fa0e 100644 (file)
@@ -1,3 +1,18 @@
+2007-11-17  Alp Toker  <alp@atoker.com>
+
+        Reviewed by Eric.
+
+        http://bugs.webkit.org/show_bug.cgi?id=16032
+        JS minidom is not portable
+
+        Use a plain UTF-8 string instead of a CFString.
+
+        Print to stdout, not stderr like CFShow() would have done, since that
+        behaviour seems unintentional.
+
+        * API/minidom.c:
+        (main):
+
 2007-11-17  Steve Falkenburg  <sfalken@apple.com>
 
         Windows build fix.