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





--- Comment #4 from lorenzo benvenuti <[EMAIL PROTECTED]>  2008-11-27 23:24:24 
PST ---
Patrick,

actually I'm not sure that the "state" of the XSSFCellStyle is fully determined
by the cellXf property. I just took a look at the copy() method and I thought
the style representation was fully delegated to the cellXf bean while the other
properties were just cellStyle "control" variables already set in the
costructor when the current style (the one that is going to clone the source
style) is created.
Please notice I might be wrong, probably the author of the XSSFCellStyle class
would be helpful here.
Regarding the methods calling itself indefinitely, as you may have guessed, I
used the overloaded method to mirror the HSSFCellStyle class implementation of
the cloneStyleFrom method.
At last: I think the "return" clauses I added are necessary for avoiding the
exception to be always thrown; actually I don't like using a "return" in the
middle of a class (in this case I could have changed the "if" condition) but
this was the less invasive change.

Hope this helps!

Lorenzo


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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