[dev] Re: [cvs] commit: imp/lib/Quota command.php

Chuck Hagenbuch chuck@horde.org
Mon Oct 14 17:15:29 2002


Quoting Eric Rostetter <eric.rostetter@physics.utexas.edu>:

> The change in question is:
> 
>   *     'params' => 'array(
>   *       'quota_path' => '/usr/bin/quota',
> - *       'grep_path'  => '/bin/grep'))
> + *       'grep_path'  => '/bin/grep'
> + *     )
> + *   )
> 
> So which way is correct/endorsed/approved/etc. for use??? Should they end
> (as I would like and as Michael shows here) one per line indented to the
> proper level, or should they be done as was originally shown (all
> together on the last line).

Yeah, this has fluctuated a lot, and this is somewhat my fault. Sorry. I
think, having everything laid out like you did, that I'm in favor of the
above style as well (the one that Michael changed it to).

> So, I don't care which way it is to be done, but I think we need to
> decide which way it is to be done, and add it to the CODING_STANDARDS 
> docs. Otherwise we're going to continue to have the caos we've had for the 
> last year or so over this formating issue.

Yup. I say go ahead and add the above one.

-chuck

--
Charles Hagenbuch, <chuck@horde.org>
"People ask me all the time what it will be like living without otters."
 - Google, thanks to Harpers