[dev] Re: [cvs] commit: imp mailbox.php

Jon Parise jon at horde.org
Sat Mar 22 20:03:21 PST 2003


On Fri, Mar 21, 2003 at 12:00:06PM -0800, Michael M Slusarz wrote:

> slusarz     2003/03/21 12:00:02 PST
> 
>   Modified files:        (Branch: RELENG_3)
>     .                    mailbox.php 
>   Log:
>   Optimization suggested by Jon.
>   
>   Revision    Changes    Path
>   2.296.2.56  +5 -5      imp/mailbox.php
> 
>   Chora Links:
>   http://cvs.horde.org/diff.php/imp/mailbox.php?r1=2.296.2.55&r2=2.296.2.56&ty=u

Not to nitpick too badly, but I'd prefer something more 80 column
friendly, such as:

$newmail_popup = ((Horde::getFormData('newmail_popup') == 'no') &&
                  $prefs->getValue('nav_popup')) ? false : true;

-- 
Jon Parise (jon at horde.org) :: The Horde Project (http://horde.org/)


More information about the dev mailing list