opensubscriber
   Find in this group all groups
 
Unknown more information…

l : lucene-dev@jakarta.apache.org 2 March 2005 • 1:12PM -0500

Re: patch - DEFAULT_ vars in IndexWriter non-final and DEFAULT for useCompoundFile
by Erik Hatcher

REPLY TO AUTHOR
 
REPLY TO GROUP




Kevin,

Why bother arguing such petty stuff?  getters/setters, as awkward and
verbose as they may be, is how its done in Java land.  There is no
benefit to making it non-final over using getters/setters.  The LOC
argument is silly too - we're not counting LOC or anything.  Let's keep
focused on Lucene's core purpose and improvement.

Erik

On Mar 1, 2005, at 10:19 PM, Kevin A. Burton wrote:

> Doug Cutting wrote:
>
>>
>> This isn't a coding convention, but rather software engineering.  If
>> we wish to be able to back-compatibly modify Lucene's implementation
>> at a later date, its usually easiest to have access through methods
>> rather than fields, since we can intecept reads and writes to the
>> field.
>>
> Standard accessor/mutator stuff... but how often is it used?  Rarely
> if ever... The point being there are a lot of benefits to just make it
> non-final.  I sent an email to commons-dev about this.   I should
> probably just blog about  it so I can give you a permalink :)
>
> Kevin
>
> --
>
> Use Rojo (RSS/Atom aggregator).  Visit http://rojo.com. Ask me for an
> invite!  Also see irc.freenode.net #rojo if you want to chat.
>
> Rojo is Hiring! - http://www.rojonetworks.com/JobsAtRojo.html
>
> If you're interested in RSS, Weblogs, Social Networking, etc... then
> you should work for Rojo!  If you recommend someone and we hire them
> you'll get a free iPod!
>    Kevin A. Burton, Location - San Francisco, CA
>       AIM/YIM - sfburtonator,  Web - http://peerfear.org/
> GPG fingerprint: 5FB2 F3E2 760E 70A8 6174 D393 E84D 8D04 99F1 4412
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: lucene-dev-unsubscribe@jaka...
> For additional commands, e-mail: lucene-dev-help@jaka...


---------------------------------------------------------------------
To unsubscribe, e-mail: lucene-dev-unsubscribe@jaka...
For additional commands, e-mail: lucene-dev-help@jaka...

Bookmark with:

Delicious   Digg   reddit   Facebook   StumbleUpon

Related Messages

opensubscriber is not affiliated with the authors of this message nor responsible for its content.