Re: [dev] CHINA001 (was: Re: [dev] List of 2730 uncallable methods in DEV300_m10)

2008-05-12 Thread Kohei Yoshida

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)

2008-05-11 Thread Caolan McNamara
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)

2008-05-09 Thread Kohei Yoshida
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]