[Libreoffice] [PATCH] Commented out some unused functions in calc

Alfonso Eusebio alfonso_eusebio at yahoo.co.uk
Sat Jan 29 07:10:36 PST 2011


Hi,

I've been going through the list of identified unused functions in calc and 
commenting out those that don't seem to be used anywhere.
Attached is the patch (LGPLv3+/MPL) that comments out some 10 functions and a 
text file with those investigated so far, and the results.

I chose to comment out, rather than delete, to allow for some extra testing once 
it's committed. Commented functions can be easily identified later on and 
removed.
Please let me know if direct removal would be preferred.

After committing my changes locally I've built module "sc" and done a "make 
dev-install". Everything seems to be working fine, but the test wasn't 
exhaustive.
The changes have been rebased with current master.

There are a few functions that belong to some "addin" mechanism now obsolete. 
I'm not sure if it can be removed completely, but it looks like a good candidate 
(details in attached text file).

Thanks & regards,
Alfonso


      
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Commented-out-some-unused-functions-in-calc.patch
Type: text/x-patch
Size: 12545 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110129/a23f91eb/attachment-0001.bin>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: review_of_unused_functions_in_calc_1.txt
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110129/a23f91eb/attachment-0001.txt>


More information about the LibreOffice mailing list