graphml reader namespace handling is broken

[count0]

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.

[count0]

Logged In: YES 
user_id=365386
Originator: YES

File Added: graphml.hpp.patch

[count0]

Logged In: YES 
user_id=365386
Originator: YES

File Added: reader_test.cc

[count0]

Logged In: YES 
user_id=365386
Originator: YES

File Added: reader-test.xml

[count0]

Logged In: YES 
user_id=365386
Originator: YES

File Added: read_graphml.rst

[count0]

Logged In: YES 
user_id=365386
Originator: YES

File Added: write_graphml.rst

[Jeremiah Willcock]

I do not see any files anymore. It appears the parse error issue has been fixed; have the other issues?

[Jeremiah Willcock]

These are the files from the old tracker.

[Jeremiah Willcock]

(In [53650]) Applied patches from ticket #900; fixes #900