Import of notes, generates force close

There's an OutOfMemoryError in the stack trace and IIRC messages can be out of order in adb's output so it could be that.

Would you mind trying with 100 notes?

How big are your notes (average and biggest)?

Hi!

Note size: 800 bytes - 3,6 kbyte

1. I tried with 5x50 notes, and it works, after each run i cleaned tomdroid
with an uninstall.

2. Added notes in a row (starting with the smallest 800byte - 1 kbyte): 100
+ 50, it works
    +1 note it force closes (all notes together ~140 kb)

3. Added notes in a row (starting with the biggest 3,6 kbyte - 1 kbyte):
150, it works
    +1 it force closes (all notes together ~224kb)

So it does not depend on the size of the notes only on the amount, the limit
is 150 notes at least on my configuration (motorola milestone, 2.0.1).

Best regards Bernd

--
MfG Bernd
:x

On Tue, Mar 9, 2010 at 02:58, Olivier Bilodeau <email address hidden>wrote:

> There's an OutOfMemoryError in the stack trace and IIRC messages can be
> out of order in adb's output so it could be that.
>
> Would you mind trying with 100 notes?
>
> How big are your notes (average and biggest)?
>
> ** Changed in: tomdroid
> Importance: Undecided => Medium
>
> ** Changed in: tomdroid
> Assignee: (unassigned) => Olivier Bilodeau (plaxx)
>
> --
> Import of notes, generates force close
> https://bugs.launchpad.net/bugs/534213
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Ok, so it really looks like 150 is the magic number, I'll try to reproduce on my phone and let you know.

Thanks for the precise report!

Hi! It seems I had the same or a very similar bug when tried your app in my HTC Desire
(Importing notes - force close - OutOfMemoryError in log.)

There are about 460 notes in my collection, biggest ones are about 30Kb.
Total size is about 2Mb.

Hi,

as reported from one user in https://bugs.launchpad.net/tomdroid/+bug/669482 , my problem is also resolved with the solution in
http://bazaar.launchpad.net/~syslock/tomdroid/note-editing/revision/233 .

Great find Thomas!

I've just committed the fix into trunk.