lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael McCandless <luc...@mikemccandless.com>
Subject Re: [jira] Updated: (LUCENE-1987) Remove rest of analysis deprecations (Token, CharacterCache)
Date Mon, 19 Oct 2009 20:24:24 GMT
On Mon, Oct 19, 2009 at 4:00 PM, Yonik Seeley
<yonik@lucidimagination.com> wrote:
> On Mon, Oct 19, 2009 at 3:45 PM, Mark Miller <markrmiller@gmail.com> wrote:
>> but there is some old source code here and
>> there that really bugs me
>
> Is it Doug's
>
>  if (foo)
>     bar()
>  else {
>    baz();
>  }
>
> or is it my single line
>
>  if (a==null) return 0;
>
> ;-)

Or my always doing this up until a while ago:

  if (foo)
    something;

but then suddenly [trying to] switch to the correct:

  if (foo) {
    something;
  }

?

> One of my personal pet peeves is more indentation than necessary for
> large blocks of code, rather than just immediately handling the
> exception cases and escaping. Example:

Hmm I think I tend to do this :)

But I agree, your way IS more readable so I'll try to switch!

Mike

---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: java-dev-help@lucene.apache.org


Mime
View raw message