[AppDb] Comments handling cleanup

Jonathan Ernst jonathan at ernstfamily.ch
Sun Jun 25 13:26:41 CDT 2006


Le dimanche 25 juin 2006 à 10:59 -0600, Tony Lambregts a écrit :
[...]
> I am more in favor of this approach than using makeSafe() and do the same thing 
> as Chris's query_parameters() patch. However I am hard pressed to say whether 
> this method is really better or safer than query_parameters().
> 
> I have had real issues with makeSafe() the primary one is that IMO the place to 
> make sure that we are safe from sql injection is where we create the SQL. The 
> makeSafe() did not do that.
> 
> The changes to that are only cosmetic and make it harder to see the actual 
> changes. Formating changes should be in a separate patch.
> 
> Also I would really appreciate a "Files Changed:" section that lists the files 
> changed/added/removed by this patch. I find that it really helps in reviewing 
> patches.
> 
> Please resubmit with the formating changes in a separate patch.

Thanks for your comments.

I was aware that my changes weren't really atomic but I sent the patch
to get some comments.

If everyone agrees with the approach I'll make separate patches
tomorrow.

Thanks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 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/20060625/7c57f74b/attachment.pgp


More information about the wine-devel mailing list