[Expat-bugs] [ expat-Bugs-928113 ] XML_SetUnknownEncodingHandler() return values are mixed.

[SourceForge.net]

Bugs item #928113, was opened at 2004-04-02 04:05
Message generated for change (Comment added) made by kwaclaw
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=110127&aid=928113&group_id=10127

Category: Documentation
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Maxim Petrovich Dementiev (maximdementiev)
Assigned to: Fred L. Drake, Jr. (fdrake)
Summary: XML_SetUnknownEncodingHandler() return values are mixed.

Initial Comment:
In documentation (reference.html) to expat-1.95.7 
wrote:

"If the handler knows how to deal with an encoding 
with the given name, it should fill in the info data 
structure and return XML_STATUS_ERROR. Otherwise it 
should return XML_STATUS_OK."

In expat.h:

enum XML_Status {
  XML_STATUS_ERROR = 0, // false
#define XML_STATUS_ERROR XML_STATUS_ERROR
  XML_STATUS_OK = 1 // not zero - ture
#define XML_STATUS_OK XML_STATUS_OK
};

In xmlparse.c:

    if (unknownEncodingHandler(...)) {
      // ... handler knows how to deal ...
      ... return XML_ERROR_NONE;
    }
   ...
  return XML_ERROR_UNKNOWN_ENCODING;

So right might be:

"If the handler knows how to deal with an encoding ... 
return XML_STATUS_OK (not zero - ture). Otherwise it 
should return XML_STATUS_ERROR (zero - false)."


----------------------------------------------------------------------

>Comment By: Karl Waclawek (kwaclaw)
Date: 2004-04-30 11:41

Message:
Logged In: YES 
user_id=290026

Fixed in reference.html rev. 1.57.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=110127&aid=928113&group_id=10127