classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] Stupid spec constant values


From: Stuart Ballard
Subject: Re: [cp-patches] Stupid spec constant values
Date: Wed, 5 Oct 2005 12:10:42 -0400

(forgot to cc the list again)

On 10/5/05, Mark Wielaard <address@hidden> wrote:
> I think the "right thing to do" here is just ignoring this field.

I disagree, but you're the maintainer so I guess you get the final word.

> A constantly changing constant static field value is useless anyway.

Yes, but it's also harmless. And it's my opinion that false positives
in the error report *are* harmful in a small way.

If I can't convince you to check in the patch then I'll try to think
of a way to get japitools to ignore these. I'd rather not, though.

> And
> adding RCS keywords in our own source code only makes the problem worse
> since people have to be careful that they don't get expanded when moving
> the source around between repositories.

I carefully wrote the patch so that this wouldn't be an issue.

RCSID = "$" + "Id: ...

Stuart.
--
http://sab39.dev.netreach.com/


--
http://sab39.dev.netreach.com/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]