TinyXML2 parser
Moderators: CEGUI MVP, CEGUI Team
TinyXML2 parser
Hi.
For a long time I use TinyXML2Parser for XML parsing in CEGUI. If you are interested in it, I can share the code in a form that is convenient for you. This parser uses updated and actively developed version of tinyxml library, that can be found here:
https://github.com/leethomason/tinyxml2
I would appreciate if you consider it useful and add it into CEGUI. For now, after each CEGUI update I have to add all source and project files by hand.
For a long time I use TinyXML2Parser for XML parsing in CEGUI. If you are interested in it, I can share the code in a form that is convenient for you. This parser uses updated and actively developed version of tinyxml library, that can be found here:
https://github.com/leethomason/tinyxml2
I would appreciate if you consider it useful and add it into CEGUI. For now, after each CEGUI update I have to add all source and project files by hand.
Re: TinyXML2 parser
@niello: Adding TinyXML2 support is indeed something I've planned and if you've done it that's great!
If u can contribute it to cegui in the form of a PR (pull request) that would b highly appreciated!
If u can contribute it to cegui in the form of a PR (pull request) that would b highly appreciated!
Re: TinyXML2 parser
Ok, I will update my code to conform to other parsers and then I will do pull request. Good chance to learn how to do it at all ![Smile :)](./images/smilies/icon_smile.gif)
![Smile :)](./images/smilies/icon_smile.gif)
Re: TinyXML2 parser
@niello: If u have no experience with VCS-s, I'm not sure by the time you've learned how to do it your face will be like "
". Good luck! ![Laughing :lol:](./images/smilies/icon_lol.gif)
![Smile :)](./images/smilies/icon_smile.gif)
![Laughing :lol:](./images/smilies/icon_lol.gif)
Re: TinyXML2 parser
I have experience with Perforce, SVN and Git, but since first two have no PR concept and I used Git only as a single developer, I never touched this part of the system.
Re: TinyXML2 parser
Done
The most strange thing is that I must fork and clone the whole CEGUI just to propose an addition of a couple of files.
![Smile :)](./images/smilies/icon_smile.gif)
Re: TinyXML2 parser
I waive my copyright on this work and you can use it without restrictions. You may need to change parser description string and add the parser into a build system. If I can do something to improve this commit, please tell me. I have no experience of contributing into open-source projects so I may do smth wrong.
Re: TinyXML2 parser
@niello: Lol I got news for u: you've waived your rights the moment u created the pull request
Once it gets into cegui it has the cegui license (MIT license). Of course, you're more than welcome to add your name and a short description of your project to "doc/doxygen/authors.dox" in the cegui repository, under "Main Contributors".
![Laughing :lol:](./images/smilies/icon_lol.gif)
Re: TinyXML2 parser
A say it just to note that I understand what I do
I added myself into a contributor list, but I didn't see anybody wrote a brief description of his/her project. Do you mean a description of contribution?
![Smile :)](./images/smilies/icon_smile.gif)
Re: TinyXML2 parser
niello wrote:A say it just to note that I understand what I doI added myself into a contributor list, but I didn't see anybody wrote a brief description of his/her project. Do you mean a description of contribution?
By "project" I meant the PR, not your private project where u use it, so it's actually the same as "contribution", but I think you've already got that
![Smile :)](./images/smilies/icon_smile.gif)
Re: TinyXML2 parser
Nevermind, it is a language barrier. Sometimes it makes me specify really obvious things)
Second PR is ready for your review.
Second PR is ready for your review.
Re: TinyXML2 parser
niello wrote:DoneThe most strange thing is that I must fork and clone the whole CEGUI just to propose an addition of a couple of files.
I think that with today storage's price the pros of cloning the repository outweighs the cons. The whole cegui repo is 190M uncompressed - it's negligible.
Re: TinyXML2 parser
@niello: Also note that there's no need to fork for every PR, even if u wish to open several PR-s to the same repo b4 the previous ones have been merged. What u should do is, instead of having your commits have the same branch name as the branch u want to merge them to, u use a feature branch - i.e. a new branch just for the feature / issue u deal with, e.g. "tinyxml2". Then u make the PR e.g. from "niello11/CEGUI" branch "tinyxml2" to "cegui/CEGUI" branch "v0-8". Actually this is the recommended way to do PR-s always, and is more convenient for both u and us.
Re: TinyXML2 parser
Thanks a lot for the explanation. I will try it the next time I have something useful to share.
Re: TinyXML2 parser
Do you still need tinyxml2 4.0.1 release PR for cegui-dependencies? I see the lib is already in repo.
Return to “Bug Reports, Suggestions, Feature Requests”
Who is online
Users browsing this forum: No registered users and 7 guests