<div dir="ltr"><div>Hi Robert,</div><div><br></div><div>can you fix this too when you get back to OSG dev please?</div><div><br></div><div>Thank you a bunch</div><div>Nick<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 9, 2018 at 1:58 PM Trajce Nikolov NICK <<a href="mailto:trajce.nikolov.nick@gmail.com">trajce.nikolov.nick@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi again Robert,<div><br></div><div>fast hint: The nodes to be parsed are root->osm->node->tag and their properties contains these non asci codes. But probably fastest is to write recursive parser</div><div><br></div><div>Thanks again</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 9, 2018 at 1:54 PM Trajce Nikolov NICK <<a href="mailto:trajce.nikolov.nick@gmail.com" target="_blank">trajce.nikolov.nick@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks so much Robert !!! Attached is the file<div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 9, 2018 at 1:50 PM Robert Osfield <<a href="mailto:robert.osfield@gmail.com" target="_blank">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>
On Mon, 9 Jul 2018 at 12:26, Trajce Nikolov NICK<br>
<<a href="mailto:trajce.nikolov.nick@gmail.com" target="_blank">trajce.nikolov.nick@gmail.com</a>> wrote:<br>
> >  .... are changes for changes sake rather than likely to make any<br>
> functional difference<br>
><br>
> 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? :-)<br>
<br>
I might be quicker just to provide a link to or attach an xml file<br>
that causes problem with the present parser, and then I can look at<br>
what happens when the present code encounters it, and how your changes<br>
affect things.<br>
<br>
As a general note, PRs/commits should be focused on addressing one<br>
issue at a time, so bundling a fix for a specific bug along with other<br>
cosmetic or other changes just confuses what is functional and what is<br>
not.  It's not uncommon to have to go back to various revisions in<br>
code to see where regressions may have occurred so having commits that<br>
just wrap up a small set of associated changes is really helpful in<br>
identifying what has been changed and why.<br>
<br>
Robert.<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="m_-5190290356843732737m_-3195940819083448362gmail_signature" data-smartmail="gmail_signature">trajce nikolov nick<br></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="m_-5190290356843732737gmail_signature" data-smartmail="gmail_signature">trajce nikolov nick<br></div>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">trajce nikolov nick<br></div>