Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Syntax feels a bit awkward #1

Closed
chrisnicola opened this issue Oct 17, 2010 · 2 comments
Closed

Syntax feels a bit awkward #1

chrisnicola opened this issue Oct 17, 2010 · 2 comments

Comments

@chrisnicola
Copy link

What is the purpose of the '#' and '@' syntax? Would it not be better to use a dot syntax like .text and .attr and especially .name? Does it serve some other purpose?
@chrisnicola
Copy link
Author

Here is what I'm thinking:
http://github.com/lucisferre/node-xml2js/commit/2f2048bf638fa19c3807acab752a485499ba8434

@Leonidas-from-XIV
Copy link
Owner

Your link gives me a 404, sorry. Apparently you already deleted your repo (I took over maintenance some time ago and had no time to investigate all issues). If you want to get in touch about this issue, please reopen this.

knusul pushed a commit to knusul/node-xml2js that referenced this issue Jan 21, 2016
Handle errors a bit more gracefully and added more tests Leonidas-fro…
4f8ff73 
…m-XIV#1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Leonidas-from-XIV @chrisnicola