https://issues.apache.org/bugzilla/show_bug.cgi?id=47245

--- Comment #17 from Triqui <pedro.t.gar...@gmail.com> ---
Well, I stated my opinion before (comment 5). If no RowRecords are present, POI
creates them, but if some are present and some not, then this exception says it
won't do it. I can't see the logic behind this. So, to me, it looks like the
best solution is to comment out that exception.

On the other hand, since I've been trying to find out what is wrong with those
files, I've noticed some problems with the row number and the row index.

You are right. Adding -1 is wrong.
But the file has 149 RowRecords (from 0 to 148). They are stored in a TreeMap
with a 0-based KeySet. When I debug it it has the following KeySet [0 ... 148].
Some keys may be missing if a row is empty. But, please note that there is no
row 149, which is the one this cell belongs to:
[LABELSST]
    .row    = 0x0095
    .col    = 0x0002
    .xfindex= 0x0045
  .sstIndex = 0x0053
[/LABELSST]

Again, I think this is a problem with the file and somewhat unrelated to the
bug. The real question is what to do when a row records is missing? If no row
records exists it's ok, POI creates them. But if there are some row records but
not all of them, then it's a problem. Is it really a problem?
I think POI can create the missing row records and carry on, but I would like
it if someone with more knowledge would comment on this.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Reply via email to