[Pidgin] #5613: Problems detected using static analysis tool

Pidgin trac at pidgin.im
Thu Apr 24 05:28:37 EDT 2008


#5613: Problems detected using static analysis tool
---------------------------+------------------------------------------------
  Reporter:  julianday     |       Owner:  lschiere
      Type:  patch         |      Status:  new     
  Priority:  minor         |   Milestone:          
 Component:  unclassified  |     Version:  2.4.1   
Resolution:                |    Keywords:          
   Pending:  0             |  
---------------------------+------------------------------------------------
Comment (by julianday):

 Replying to [comment:3 rlaager]:
 > Thank you for the patch. Could you provide some explanation on these
 changes? A patch with a bunch of unrelated changes and no explanation is
 hard to review.
 If I add comments at the location of each change, would that be
 appropriate? I didn't think you would want the problem db cluttered with a
 large number of similar issues, which don't have test cases and aren't
 necessarily possible to reproduce. Some are just stylistic conventions,
 others a less permissive interpretation of the C standard.

 > Also, what real name and e-mail for you should be used in COPYRIGHT and
 the commit message?
 Real name: Julian Day, email address: julianfday at googlemail.com

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


More information about the Tracker mailing list