<div dir="ltr">hi Robert,<div><br></div><div>

<span style="background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">>  .... are changes for changes sake rather than likely to make any</span><br style="background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><span style="background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">functional difference</span>

<br></div><div><br></div><div>I knew you will have comments ;-). It works just fine with these changes but yes, you are right - quick and somewhat dirty. So can you look at it or you want to do a ping-pong code review with my changes by your advice? :-)</div><div><br></div><div>Please let me know</div><div><br></div><div>Nick</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 9, 2018 at 1:09 PM Robert Osfield <<a href="mailto:robert.osfield@gmail.com">robert.osfield@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Nick,<br>
<br>
I had a quick look at your changes and it seems to be like most of<br>
them are changes for changes sake rather than likely to make any<br>
functional difference.  For non ascii char support I think what you'd<br>
actually want to do is change the XmllNode::Input::string _buffer<br>
member var to a std::vector<int> _buffer, and have the<br>
Input::readAllDataIntoBuffer() method be adapted to read the file<br>
accepting only chars greater than 0 rather than limiting to the ascii<br>
0 to 255 range.<br>
<br>
Robert.<br>
On Mon, 9 Jul 2018 at 11:41, Trajce Nikolov NICK<br>
<<a href="mailto:trajce.nikolov.nick@gmail.com" target="_blank">trajce.nikolov.nick@gmail.com</a>> wrote:<br>
><br>
> Hi Robert,<br>
><br>
> I am working with some localized XML files (with chars out of 0-255 range) and at present the XML Node parsing is not suitable to manage it. And I think the support for is important - at my case it is OpenStreetMap with street names containing these chars.<br>
><br>
> Attached is the modified source (based on the master) that fixes ity, if you have time and will to review. I can do PR if you want<br>
><br>
> Thanks and cheers!<br>
><br>
> Nick<br>
><br>
> p.s. welcome back<br>
><br>
> --<br>
> trajce nikolov nick<br>
> _______________________________________________<br>
> osg-users mailing list<br>
> <a href="mailto:osg-users@lists.openscenegraph.org" target="_blank">osg-users@lists.openscenegraph.org</a><br>
> <a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
_______________________________________________<br>
osg-users mailing list<br>
<a href="mailto:osg-users@lists.openscenegraph.org" target="_blank">osg-users@lists.openscenegraph.org</a><br>
<a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">trajce nikolov nick<br></div>