https://bz.apache.org/bugzilla/show_bug.cgi?id=58818

--- Comment #2 from Javen O'Neal <[email protected]> ---
Created attachment 33508
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=33508&action=edit
fix EscherAggregate.createAggregate

It seems like the the break statement in createAggregate [2] leaves some
records unread. In attachment 33418, the NoteRecord has object id 3, and is
skipped by the break statement. Removing the break statement fixes the
described error (HSSFSheet.getCellComments) but fails a unit test [1]. 

[1] src/testcases/org/apache/poi/hssf/model/TestDrawingAggregate.java
[2]
>        // any NoteRecords that follow the drawing block must be aggregated 
> and and saved in the tailRec collection
>        while (loc < records.size()) {
>            if (sid(records, loc) == NoteRecord.sid) {
>                NoteRecord r = (NoteRecord) records.get(loc);
>                agg.addTailRecord(r);
>            }
>            else {
>                break;
>            }
>            loc++;
>        }

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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to