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.

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

Prints entire XML doc in show method #28

Open
ssfrr opened this issue Jul 19, 2015 · 3 comments
Open

Prints entire XML doc in show method #28

ssfrr opened this issue Jul 19, 2015 · 3 comments

Comments

@ssfrr
Copy link

ssfrr commented Jul 19, 2015

If you call parse_file from the REPL and neglect to put a semicolon after, LightXML will print out the whole document. It may be better to follow the example of the Array methods and only do a partial print if there's more data than will fit on one screen.

@tkelman
Copy link
Contributor

tkelman commented Jul 19, 2015

Not a bad idea, I guess. I'm not sure exactly which functions you'll need to implement to do this, maybe something with showcompact? I'd review a PR if you or anyone else can come up with something that works.

@twavv
Copy link

twavv commented Dec 8, 2019

This is pretty important. I just tried to open some open street map data using LightXML and it crashed HARD (the file is like 200MB).

@xgdgsc
Copy link

xgdgsc commented Jul 24, 2023

What about saving filesize somewhere and if larger than 1MB just print a warning?

EDIT: Found XML.jl has some sane defaults.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants