Update of patch #6102 (project phpgroupware):

                  Status:                    None => In Progress            
             Assigned to:                    None => johang                 
                 Summary: random hacks in {ipc_, so, bo}notes => various
fixes in {ipc_, so, bo}notes

    _______________________________________________________

Follow-up Comment #1:

I am feeling lazy tonight.  Here are the comments I made on IRC about the
patches.

<skwashd> johang: btw please provide 1 patch when it should all be applied in
1 hit
<johang> oh, I see. makes sense.
<johang> I'll do that next time
<skwashd> np
<skwashd> 6102
<skwashd> bonotes fine
<skwashd> sonotes ... did you check for occurances of $this->db2?
<johang> yepp. was unused.
<skwashd> ok np with that one then
<skwashd> the ipc notes
<skwashd> intval($var) has more overhead than a caste (int) $var
<johang> shoot. thought it was the other way around :)
<skwashd> the rest looks like good optimisations
<skwashd> a function call always has more overhead
<skwashd> you will see a lot of intval()s in our code ... i added them until
someone pointedit out to me

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?6102>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/



_______________________________________________
phpGroupWare-tracker mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/phpgroupware-tracker

Reply via email to