Re: patches addressing error handling

2011-03-12 Thread Kristian Høgsberg
Those are all good fixes, thanks. Kristian On Fri, Mar 11, 2011 at 10:24 AM, Iskren Chernev wrote: > Ooops I didn't sent you the latest version of the final patch :) > > On Fri, Mar 11, 2011 at 4:54 PM, Iskren Chernev > wrote: >> >> Some resources were not freed if the create function exited ea

Re: patches addressing error handling

2011-03-11 Thread Iskren Chernev
Ooops I didn't sent you the latest version of the final patch :) On Fri, Mar 11, 2011 at 4:54 PM, Iskren Chernev wrote: > Some resources were not freed if the create function exited earlier. > > I'm not sure how important these patches are. If they are I can make > another pass on the code to fix

patches addressing error handling

2011-03-11 Thread Iskren Chernev
Some resources were not freed if the create function exited earlier. I'm not sure how important these patches are. If they are I can make another pass on the code to fix similar errors. Also adding warn unused result to the creation functions that may fail will help spotting all places. Regards,