Hi Noel,

On Tuesday, 2007-06-26 13:33:33 +0100, Noel Power wrote:

> ok, I also use 
> ScRange::ParseRows
> ScRange::ParseColumns  &

Currently it doesn't matter because ScVbaRange::Columns() and
ScVbaRange::Rows() don't test the return value anyway.
ScRange::ParseRows() and ScRange::ParseCols() don't follow the other
implementation semantics, that should be adapted. The SCA_VALID is never
set, and they only set SCA_VALID_COL (even for ParseRows(), which seems
to be a copy&paste error), but should set SCA_VALID_COL2 respectively
SCA_VALIDD_ROW2 as well if a valid reference was recognized.

> ScRange::Format

That one sets the corresponding SCA_VALID_* bits internally if passed
a SCA_VALID, so should be fine.

  Eike

-- 
 OOo/SO Calc core developer. Number formatter stricken i18n transpositionizer.
 OpenOffice.org Engineering at Sun: http://blogs.sun.com/GullFOSS
 Please don't send personal mail to this [EMAIL PROTECTED] account, which I use 
for
 mailing lists only and don't read from outside Sun. Use [EMAIL PROTECTED] 
Thanks.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to