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

--- Comment #4 from alex <[email protected]> 2010-09-06 01:44:05 EDT ---
I do not know if it is allowed to comment on resolved issues (if not, please
excuse my ignorance), but I would like to point out that while I understand
that you want both implementations to work the same, I think it would be easier
to return a blank cell in the following situation:

CellValue cellValue = evaluator.evaluate(cell);
switch(cellValue.getCellType()) {
...
}

This will throw a nullpointer exception if evaluator returns "null" for blank
cells. It is of course entirely feasible to check for "== null" before doing
the switch, but isn't that partly the point of having a "CELL_TYPE_BLANK"?

-- 
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