clean-up
Jonathan Ernst
Jonathan at ErnstFamily.ch
Thu Dec 23 16:44:50 CST 2004
Le jeudi 23 décembre 2004 à 22:39 +0100, Paul van Schayck a écrit :
> Hey Eric,
>
> On Thu, 23 Dec 2004 01:38:02 -0800 (PST), Eric Blade <ekdikeo4 at yahoo.com> wrote:
> > The attached patch includes a lot of cleaning up,
> > mostly to HTML, a bit to some PHP.
>
> What's this supposed to do:
> if(!$tempResult)
> {
> - echo "$query <br>\n";
> + echo "$query <<br />>\n";
> echo "An error occurred: ".mysql_error()."<p>";
> exit;
> }
> @@ -68,7 +68,7 @@
> if(debugging())
> {
> echo $query;
> - echo "<br><br>";
> + echo "<<br />><<br />>";
> }
>
Same question: what are those <<br />> ???
> And:
>
> -if(!loggedin())
> +if(!loggedin() || (!havepriv("admin"))
If you are not loggeddin() you can't havepriv("admin") btw
So we have:
_old version_
C=A
A=TRUE => C=TRUE
A=FALSE => C=FALSE
_your version_
C=(A OR B)
A=TRUE => B=TRUE => C=TRUE
A=FALSE, B=TRUE => C=TRUE
A=FALSE, B=FALSE => C=FALSE
So your version doesn't mean the same as the old one, is that correct ?
If it is what we need, why don't you simple use:
if(!havepriv("admin"))
C=B
B=TRUE => C=TRUE
B=FALSE => C=FALSE
Which means the same as your version.
>
> Shouldn't this be
> if(!loggedin() AND (!havepriv("admin"))
C=(A AND B)
A=TRUE => B=TRUE => C=TRUE
A=FALSE, B=TRUE => C=FALSE
A=FALSE, B=FALSE => C=FALSE
So this version means the same as the old one but then why do we make it
more complicated ?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Ceci est une partie de message
=?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20041223/56fa3c97/attachment.pgp
More information about the wine-devel
mailing list