jujojujo2003 / AnDCpp

An Android DC++ client based on libjdcpp
5 stars 6 forks source link

out of memory error. #1

Closed madhavanmalolan closed 11 years ago

madhavanmalolan commented 11 years ago

Stack Trace:

11-01 14:54:35.981: E/AndroidRuntime(26855): FATAL EXCEPTION: Thread-3328 11-01 14:54:35.981: E/AndroidRuntime(26855): java.lang.OutOfMemoryError: [memory exhausted] 11-01 14:54:35.981: E/AndroidRuntime(26855): at dalvik.system.NativeStart.main(Native Method)

This happens when downloading LARGE sharer's filelist, like "HumanValues-Main"

phinfinity commented 11 years ago

Ummm we can't really help it right now. We are using a really inefficient xml parser which uses twice as much memory as the file list. I could try reducing this by half, but reducing it even more will be really hard. On Nov 1, 2013 3:01 PM, "madhavan93" notifications@github.com wrote:

Stack Trace:

11-01 14:54:35.981: E/AndroidRuntime(26855): FATAL EXCEPTION: Thread-3328 11-01 14:54:35.981: E/AndroidRuntime(26855): java.lang.OutOfMemoryError: [memory exhausted] 11-01 14:54:35.981: E/AndroidRuntime(26855): at dalvik.system.NativeStart.main(Native Method)

This happens when downloading LARGE sharer's filelist, like "HumanValues-Main"

— Reply to this email directly or view it on GitHubhttps://github.com/jujojujo2003/AnDCpp/issues/1 .

jujojujo2003 commented 11 years ago

Anish r u using a DOM XML parser? Why didnt u use a streaming parser?

On Fri, Nov 1, 2013 at 3:05 PM, phinfinity notifications@github.com wrote:

Ummm we can't really help it right now. We are using a really inefficient xml parser which uses twice as much memory as the file list. I could try reducing this by half, but reducing it even more will be really hard. On Nov 1, 2013 3:01 PM, "madhavan93" notifications@github.com wrote:

Stack Trace:

11-01 14:54:35.981: E/AndroidRuntime(26855): FATAL EXCEPTION: Thread-3328 11-01 14:54:35.981: E/AndroidRuntime(26855): java.lang.OutOfMemoryError: [memory exhausted] 11-01 14:54:35.981: E/AndroidRuntime(26855): at dalvik.system.NativeStart.main(Native Method)

This happens when downloading LARGE sharer's filelist, like "HumanValues-Main"

— Reply to this email directly or view it on GitHub< https://github.com/jujojujo2003/AnDCpp/issues/1> .

— Reply to this email directly or view it on GitHubhttps://github.com/jujojujo2003/AnDCpp/issues/1#issuecomment-27555182 .

phinfinity commented 11 years ago

yes, because I was lazy :P , That is what I mentioned above. using SAX parser can bring down memory usage by half. It's a bit of effort though not straightforward trivial

phinfinity commented 11 years ago

This is a bug with libjdcpp , closing this and opening issue there.