[Pidgin] #7340: Bad programming practice

Pidgin trac at pidgin.im
Sun Jul 19 04:24:36 EDT 2009


#7340: Bad programming practice
-----------------------------------------+----------------------------------
 Reporter:  fsilveira                    |        Owner:           
     Type:  patch                        |       Status:  new      
Milestone:  Patches Needing Improvement  |    Component:  libpurple
  Version:  2.5.2                        |   Resolution:           
 Keywords:                               |  
-----------------------------------------+----------------------------------

Comment(by qulogic at pidgin.im):

 (In [826fe96cf74adec56a74a298023c17a9eece9c8b]):[[BR]]
 ALLOW_TAG expands to multiple statements, so we should put braces around
 this if(). Otherwise, I think we'd accept <html/> in the middle of the
 markup.

 References #7340.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/7340#comment:11>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list