Re: [dev] CHINA001
Kohei Yoshida wrote: I will then follow this convention and comment out all unused methods with "//UNUSED2008-05". I put the date there so that we will know how long that code has been commented out (to perhaps help us remove it later on after a certain period of time). That sounds good. Niklas - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [dev] CHINA001
Caolan McNamara wrote: Its connected to the binfilter stuff, comments in the non-binfilter code with "CHINA001" should basically indicate that the code was removed from the non-binfilter code because it was believed only needed to support other-code that is now solely in the binfilter, i.e. the legacy 5.2 binfilter filters and other similar things. The CHINA001 lines are from "dialogdiet01", where dialog code was moved into separate libraries (issue 24916). Old binfilter code in sc was only partially removed so far, this causes some of the entries in your list. Niklas - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [dev] CHINA001 (was: Re: [dev] List of 2730 uncallable methods in DEV300_m10)
On Sun, 2008-05-11 at 10:29 +0100, Caolan McNamara wrote: > On Fri, 2008-05-09 at 17:54 -0400, Kohei Yoshida wrote: > > Hi Niklas, > > > > On Fri, 2008-05-09 at 19:17 +0200, Niklas Nebel wrote: > > > example: CHINA001 > > > > In the interest of removing the unused code, I'd like know what those > > CHINA001 labels are for. Is it okay to perhaps review those commented > > out lines and see if we can remove them permanently? > > Its connected to the binfilter stuff, comments in the non-binfilter code > with "CHINA001" should basically indicate that the code was removed from > the non-binfilter code because it was believed only needed to support > other-code that is now solely in the binfilter, i.e. the legacy 5.2 > binfilter filters and other similar things. Thanks Caolan for the information. I was always wondering what that meant. I will then follow this convention and comment out all unused methods with "//UNUSED2008-05". I put the date there so that we will know how long that code has been commented out (to perhaps help us remove it later on after a certain period of time). Kohei -- Kohei Yoshida - OpenOffice.org Engineer - Novell, Inc. <[EMAIL PROTECTED]> - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [dev] CHINA001 (was: Re: [dev] List of 2730 uncallable methods in DEV300_m10)
On Fri, 2008-05-09 at 17:54 -0400, Kohei Yoshida wrote: > Hi Niklas, > > On Fri, 2008-05-09 at 19:17 +0200, Niklas Nebel wrote: > > example: CHINA001 > > In the interest of removing the unused code, I'd like know what those > CHINA001 labels are for. Is it okay to perhaps review those commented > out lines and see if we can remove them permanently? Its connected to the binfilter stuff, comments in the non-binfilter code with "CHINA001" should basically indicate that the code was removed from the non-binfilter code because it was believed only needed to support other-code that is now solely in the binfilter, i.e. the legacy 5.2 binfilter filters and other similar things. C. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[dev] CHINA001 (was: Re: [dev] List of 2730 uncallable methods in DEV300_m10)
Hi Niklas, On Fri, 2008-05-09 at 19:17 +0200, Niklas Nebel wrote: > example: CHINA001 In the interest of removing the unused code, I'd like know what those CHINA001 labels are for. Is it okay to perhaps review those commented out lines and see if we can remove them permanently? Kohei -- Kohei Yoshida - OpenOffice.org Engineer - Novell, Inc. <[EMAIL PROTECTED]> - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]