bugphpGroupWare - Bugs: bug #3640, class.schema_proc_mssql.inc.php

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #3640: class.schema_proc_mssql.inc.php

Submitted by:  Kai Hofmann <powerstat>
Submitted on:  Fri 16 May 2003 12:19:22 PM UTC  
 
Item Group: NoneCategory: None
Severity: 3 - NormalPriority: 5 - Normal
Status: InvalidPrivacy: Public
Assigned to: NoneOpen/Closed: Closed
Component Version: NoneOperating System: None
Reproducibility: NonePlanned Release: None
Fixed Release: 

Mon 19 May 2003 01:28:31 PM UTC, comment #3:

It has the same effect. Why bother changing it?

Michael L. Dean <mdean>
Mon 19 May 2003 08:45:32 AM UTC, comment #2:

http://www.php.net/manual/en/language.types.boolean.php

Converting to boolean

To explicitly convert a value to boolean, use either the
(bool) or the (boolean) cast.

Kai Hofmann <powerstat>
Sat 17 May 2003 07:26:02 PM UTC, comment #1:

It is not redundant - it forces the result to a boolean. It's the same as saying return (myResult != 0); and hardly a performance killer.

Michael L. Dean <mdean>
Fri 16 May 2003 12:19:22 PM UTC, original submission:

I found the following code:

return !!($oProc->m_odb->query("DROP TABLE " . $sTableName));

!! looks like it is redundant and a performance killer ;-)

Kai Hofmann <powerstat>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 2 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Sat 17 May 2003 07:26:02 PM UTCmdeanStatusNone=>(Error - Not Found)
  Open/ClosedOpen=>(Error - Not Found)

Back to the top


Powered by Savane 3.1-cleanup1