You are viewing a plain text version of this content. The canonical link for it is here.
Posted to c-dev@xerces.apache.org by bu...@apache.org on 2002/11/13 18:56:39 UTC

DO NOT REPLY [Bug 14260] - MacOSUnicodeConverter::upperCase() passes wrong arguments to Carbon function

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=14260>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=14260

MacOSUnicodeConverter::upperCase() passes wrong arguments to Carbon function

jberry@criticalpath.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From jberry@criticalpath.com  2002-11-13 17:56 -------
David,

I checked in a fix for this last night. Partial diffs are below. Please verify. Thanks for your 
bug report. -jdb

  @@ -463,11 +463,12 @@
   #if TARGET_API_MAC_CARBON
   
      // If we're targeting carbon, use the CFString conversion to uppercase
  +   int len = XMLString::stringLen(toUpperCase);
      CFMutableStringRef cfString = 
CFStringCreateMutableWithExternalCharactersNoCopy(
  -        NULL,
  +        kCFAllocatorDefault,
           (UniChar*)toUpperCase,
  -        XMLString::stringLen(toUpperCase),
  -        0,
  +        len,		// length
  +        len,		// capacity
           kCFAllocatorNull);
      CFStringUppercase(cfString, NULL);
      CFRelease(cfString);
  @@ -491,11 +492,12 @@
   #if TARGET_API_MAC_CARBON
   
      // If we're targeting carbon, use the CFString conversion to uppercase
  +   int len = XMLString::stringLen(toLowerCase);
      CFMutableStringRef cfString = 
CFStringCreateMutableWithExternalCharactersNoCopy(
  -        NULL,
  +        kCFAllocatorDefault,
           (UniChar*)toLowerCase,
  -        XMLString::stringLen(toLowerCase),
  -        0,
  +        len,		// length
  +        len,		// capacity
           kCFAllocatorNull);
      CFStringLowercase(cfString, NULL);
      CFRelease(cfString);

---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-c-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-c-dev-help@xml.apache.org