Login | Register
My pages Projects Community openCollabNet

Discussions > dev > Re: [joist-dev] Re: [joist-cvs] CVS update: joist/java/org/joist/database UserManager.java

Project highlights: Architectural Overview

joist
Discussion topic

Back to topic list

Re: [joist-dev] Re: [joist-cvs] CVS update: joist/java/org/joist/database UserManager.java

Author brian
Full name Brian Behlendorf
Date 2000-09-26 10:01:53 PDT
Message Just a vote in favor of dlr's improvement.

    Brian

On Tue, 26 Sep 2000, Ed Korthof wrote:
> is this really an improvement? i understand the value in shortening the
> code where appropriate, but the old version seems far more intuitive than
> the new one (to me, anyway -- i'd have to look at the javadocs for
> PreparedStatement to be sure of what's going on).
>
> maybe that doesn't matter, but there are times when i'd prefer code to be
> clear rather concise -- maybe i've just read too many obfuscated c code
> contest entries. fwiw -- in terms of the actual byte code generated --
> i'd guess these two are equivalent.
>
> thanks --
>
> ed
>

--
-=-=-=-=-=-=-=-=-=-=​-=-=-=-=-=-=-=-=-=-=​-=-=-=-=-=-=-=-=-=-=​-=-=-=-=-=-=-=
CollabNet | open source | do what's right | now hiring

« Previous message in topic | 2 of 6 | Next message in topic »

Messages

Show all messages in topic

Re: [joist-cvs] CVS update: joist/java/org/joist/database UserManager.java edk Ed Korthof 2000-09-26 08:55:29 PDT
     Re: [joist-dev] Re: [joist-cvs] CVS update: joist/java/org/joist/database UserManager.java brian Brian Behlendorf 2000-09-26 10:01:53 PDT
     Re: [joist-dev] Re: [joist-cvs] CVS update: joist/java/org/joist/database UserManager.java David Pellegrini <davidp at collab dot net> David Pellegrini <davidp at collab dot net> 2000-09-27 00:39:31 PDT
     Re: [joist-dev] Re: [joist-cvs] CVS update: joist/java/org/joist/database UserManager.java dlr Daniel Rall 2000-09-27 01:06:48 PDT
         Re: [joist-dev] Re: [joist-cvs] CVS update: joist/java/org/joist/database UserManager.java edk Ed Korthof 2000-09-27 21:58:02 PDT
             Re: [joist-dev] Re: [joist-cvs] CVS update: joist/java/org/joist/databaseUserManager.java dlr Daniel Rall 2000-09-27 22:02:31 PDT
Messages per page: