[boost] [Boost-bugs] [ boost-Bugs-1702866 ] graphml reader namespace handling is broken

From: SourceForge.net (noreply_at_[hidden])
Date: 2007-04-18 07:32:51

Bugs item #1702866, was opened at 2007-04-18 08:32
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=107586&aid=1702866&group_id=7586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: graph
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Count Zero (count0)
Assigned to: Jeremy Siek (jsiek)
Summary: graphml reader namespace handling is broken

Initial Comment:
I believe the modifications done in

http://boost.cvs.sourceforge.net/boost/boost/libs/graph/src/graphml.cpp?r1=1.1&r2=1.2

break xml namespace processing, since it will strip any namespace information from all tags (everything before "|"), not only those which,belong to the graphml namespace. So, tags belonging to other namespaces, such as "foo:node", will be wrongly parsed as graphml tags, eg. "node"...

I'm sending attached patches which revert to the old behavior and have two further modifications:

- Better expat error handling. It now informs the user where in the file a problem occured
- Reordering of tag processing for possible speed improvement ("node" is more frequent than "graph", so it should be tested first, etc)

I'm also sending slightly improved documentation and test case.

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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=107586&aid=1702866&group_id=7586

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Boost-bugs mailing list
Boost-bugs_at_[hidden]
https://lists.sourceforge.net/lists/listinfo/boost-bugs