Hi Kohei,

On Thursday, 2008-05-08 00:22:10 -0400, Kohei Yoshida wrote:

> > If I recall correctly, the module with the most unused methods that
> > doesn't have anything in the pipeline to remove them is sc, so there's
> > where the lowest hanging fruit should be.
> 
> Since no one has raised hands, let me tackle that.

Thanks a lot!

Just a note though: please don't blindly remove everything that appears
to be unused, e.g. while ScCompressedArray::GetPrevValue() is currently
unused it is the counterpart of GetNextValue() and IMHO should be kept
for completeness of implementation. It is also some inline template
code, so shouldn't hurt either. Removing FillDataArray() on the other
hand is fine because it was used in just one special scenario.

Thanks
  Eike

-- 
 OOo/SO Calc core developer. Number formatter stricken i18n transpositionizer.
 SunSign   0x87F8D412 : 2F58 5236 DB02 F335 8304  7D6C 65C9 F9B5 87F8 D412
 OpenOffice.org Engineering at Sun: http://blogs.sun.com/GullFOSS
 Please don't send personal mail to the [EMAIL PROTECTED] account, which I use 
for
 mailing lists only and don't read from outside Sun. Use [EMAIL PROTECTED] 
Thanks.

Attachment: pgpBgs4zeBuRZ.pgp
Description: PGP signature

Reply via email to