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/05/20 01:07:33 UTC

DO NOT REPLY [Bug 9237] - Encoding spec in lower case (DTD/XML) not recognized

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=9237>.
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=9237

Encoding spec in lower case (DTD/XML) not recognized

jberry@criticalpath.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED



------- Additional Comments From jberry@criticalpath.com  2002-05-19 23:07 -------
David,

I believe this bug is due to broken implementations of upperCase and 
lowerCase in the Macintosh transcoder routines. I've checked in revised 
copies of these, and am listing the diffs below.

Could you apply the diffs to your version and confirm that this fixes your 
problem? Then update this bug report?

Thanks!

  diff -u -r1.2 -r1.3
  --- MacOSUnicodeConverter.cpp	9 Apr 2002 15:44:00 -0000	1.2
  +++ MacOSUnicodeConverter.cpp	19 May 2002 22:57:56 -0000	1.3
  @@ -55,7 +55,7 @@
    */
   
   /*
  - *	$Id: MacOSUnicodeConverter.cpp,v 1.2 2002/04/09 15:44:00 
knoaman Exp $
  + *	$Id: MacOSUnicodeConverter.cpp,v 1.3 2002/05/19 22:57:56 jberry 
Exp $
    */
   
   
  @@ -468,20 +468,16 @@
   #if defined(XML_METROWERKS)
   	// Use this if there's a reasonable c library available.
   	// Metrowerks does this reasonably
  -	wchar_t * p = (wchar_t*) toUpperCase;
   	wchar_t c;
  -	
  -	while ((c = *p) != 0)
  +	for (XMLCh* p = (XMLCh*)toUpperCase; ((c = *p) != 0); )
   		*p++ = std::towupper(c);
   #elif defined(XML_MACOSX) || true
   	// This might work, assuming we're on an ascii compiler.
   	// We'll use this under ProjectBuilder for now.
   	// Note that this only handles the ascii portion of the
   	// string, leaving all other characters in original case.
  -	wchar_t * p = (wchar_t*)toUpperCase;
  -	wchar_t c;
  -	
  -	while ((c = *p) != 0)
  +	XMLCh c;
  +	for (XMLCh* p = (XMLCh*)toUpperCase; ((c = *p) != 0); )
   	{
   		if (c >= 'a' && c <= 'z')
   			c += 'A' - 'a';
  @@ -492,26 +488,23 @@
   #endif
   }
   
  +
   void MacOSUnicodeConverter::lowerCase(XMLCh* const toLowerCase) 
const
   {
   	//	��� TODO: Support CFString for this conversion
   #if defined(XML_METROWERKS)
   	// Use this if there's a reasonable c library available.
   	// Metrowerks does this reasonably
  -	wchar_t * p = (wchar_t*) toLowerCase;
   	wchar_t c;
  -	
  -	while ((c = *p) != 0)
  +	for (XMLCh* p = (XMLCh*)toLowerCase; ((c = *p) != 0); )
   		*p++ = std::towlower(c);
   #elif defined(XML_MACOSX) || true
   	// This might work, assuming we're on an ascii compiler.
   	// We'll use this under ProjectBuilder for now.
   	// Note that this only handles the ascii portion of the
   	// string, leaving all other characters in original case.
  -	wchar_t * p = (wchar_t*)toLowerCase;
  -	wchar_t c;
  -	
  -	while ((c = *p) != 0)
  +	XMLCh c;
  +	for (XMLCh* p = (XMLCh*)toLowerCase; ((c = *p) != 0); )
   	{
   		if (c >= 'A' && c <= 'Z')
   			c += 'a' - 'A';

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