[Kmymoney-devel] CSV Plugin Bug/Puzzle

[Allan]

On 22/02/12 01:14, Allan wrote:
> This is somewhat vague as it lacks line numbers, because it only occurs
> in my updated version.
>
> For reasons unconnected with my recent mods., I imported an old
> investment file which I had made, and when I imported it, a date error
> was flagged, which usually means that the chosen date format was
> incorrect, or that the wrong column was chosen. It indicated that the
> date was empty and that the format was m/d/y. I checked back and in fact
> I had set it to d/m/y. Also, the actual date was one of those where it
> matched any format. Looking more closely now, I noticed that when the
> file was first displayed, two identical lines were shown, and in fact
> there was only a single line in the file.
>
> Looking at the readFile() routine in investprocessing.cpp, in about the
> middle, there is a 'for' loop to display the lines. Debugging, I found
> that the for loop was being entered twice, although the readFile was
> being called just once at this time. The row count is cleared near to
> the beginning of readFile() and incremented when the line has actually
> been displayed. A debug statement close to the row count clearing was
> triggered just once, a few lines later, another debug was triggered
> twice, as was the 'for' loop.
>
> I can see no logical reason reason why this should occur? Perhaps a
> corrupted stack?
>
> I haven't been able to reproduce the first problem, with the date. The
> second problem, with the row count is easily solved by moving the clear
> instruction closer to the for loop.
>
> I don't like problems that can be fixed but not explained. What do the
> experts think?
>
> Would it be best to add this to the reviewboard, rather than as a bug?
>
> Allan

Well, that was interesting.  The problem was occurring in 
investprocessing.cpp but not in the similar area of csvdialog.cpp.  Or 
so I thought.  However, the same basic issue was present there, but 
being hidden.

The ReadFile() routine was being beset by connects, which I've disabled 
in that area.

Allan