dev-list notifications (was: Re: [CHANGEUPLOAD FAILED] [ONGERRIT] leaked ODBC statement handles)

2012-08-11 Thread Ivan Timofeev
On 10.08.2012 17:50, Bjoern Michaelsen wrote: rationale: We are now sending a mail to the dev-list for every patch anyway. It does not seem to send a mail to the dev-list, at least with [CHERRYPICK]. I received: From: Gerrit ger...@gerrit.libreoffice.org CC: Ivan Timofeev

[CHANGEUPLOAD FAILED] [ONGERRIT] leaked ODBC statement handles

2012-08-10 Thread gerrit
2012-08-10 13:09:19,202 INFO Handling incoming mail from Terrence Enger ten...@iseries-guru.com 2012-08-10 13:09:19,202 INFO subject: '[ONGERRIT] leaked ODBC statement handles' 2012-08-10 13:09:19,202 INFO executing command: None, branch: master, commit: leaked 2012-08-10 13:09

Re: [CHANGEUPLOAD FAILED] [ONGERRIT] leaked ODBC statement handles

2012-08-10 Thread Bjoern Michaelsen
ten...@iseries-guru.com 2012-08-10 13:09:19,202 INFO subject: '[ONGERRIT] leaked ODBC statement handles' 2012-08-10 13:09:19,202 INFO executing command: None, branch: master, commit: leaked 2012-08-10 13:09:19,202 ERROREmailCommand instance has no attribute 'do_None

Re: [CHANGEUPLOAD FAILED] [ONGERRIT] leaked ODBC statement handles

2012-08-10 Thread Terrence Enger
-- View this message in context: http://nabble.documentfoundation.org/CHANGEUPLOAD-FAILED-ONGERRIT-leaked-ODBC-statement-handles-tp4000433p4000479.html Sent from the Dev mailing list archive at Nabble.com. ___ LibreOffice mailing list LibreOffice

Re: [ONGERRIT] leaked ODBC statement handles

2012-07-19 Thread Lionel Elie Mamane
On Wed, Jul 18, 2012 at 03:09:55PM -0400, Terrence Enger wrote: Patch attached. I think it as we discussed. Thanks. Applied pushed to master. I split your patch into three commits: - avoid freeing the NULL handle - ODBMetaDataRS ctor: abort if statement handle allocation failed - stop

Re: [ONGERRIT] leaked ODBC statement handles

2012-07-19 Thread Michael Meeks
On Thu, 2012-07-19 at 07:18 -0400, Terrence Enger wrote: Again, thank you for all the help you have given me along the way to this small code contribution. Hey ! this really helps - it's really great to have people tackling bugs - there are really plenty to go around :-)

[ONGERRIT] leaked ODBC statement handles

2012-07-18 Thread Terrence Enger
Patch attached. I think it as we discussed. I do not have OpenId, so I am not registered with gerrit, so I think that I shall not be able to enter your as reviewer. Sorry to be awkward. Thank you for your patience with me through all the to-ing and fro-ing so far. Terry. From

Re: leaked ODBC statement handles

2012-07-18 Thread Terrence Enger
at 05:15 +0200, Lionel Elie Mamane wrote: On Tue, Jul 17, 2012 at 03:46:14PM -0400, Terrence Enger wrote: On Tue, 2012-07-17 at 19:08 +0200, Lionel Elie Mamane wrote: On Mon, Jul 16, 2012 at 10:15:02PM -0400, Terrence Enger wrote: I am chasing some leaked ODBC statement handles. I see

Re: leaked ODBC statement handles

2012-07-18 Thread Lionel Elie Mamane
On Wed, Jul 18, 2012 at 09:55:21AM -0400, Terrence Enger wrote: (*) Do you agree that ODatabaseMetaDataResultSet ctor should warn if it does not get a statement handle? It never gets a statement handle as an argument, it allocates it by calling _pConnection-createStatementHandle(). If the

Re: leaked ODBC statement handles

2012-07-17 Thread Lionel Elie Mamane
On Mon, Jul 16, 2012 at 10:15:02PM -0400, Terrence Enger wrote: I am chasing some leaked ODBC statement handles. I see that ODatabaseMetaDataResultSet.cxx takes care *not* to free a statement handle which has not been subjected to one of 13 member functions with names starting open

Re: leaked ODBC statement handles

2012-07-17 Thread Terrence Enger
I am sorry for the length. I hope some of it is interesting. On Tue, 2012-07-17 at 19:08 +0200, Lionel Elie Mamane wrote: On Mon, Jul 16, 2012 at 10:15:02PM -0400, Terrence Enger wrote: I am chasing some leaked ODBC statement handles. I see that ODatabaseMetaDataResultSet.cxx takes care

Re: leaked ODBC statement handles

2012-07-17 Thread Lionel Elie Mamane
On Tue, Jul 17, 2012 at 03:46:14PM -0400, Terrence Enger wrote: On Tue, 2012-07-17 at 19:08 +0200, Lionel Elie Mamane wrote: On Mon, Jul 16, 2012 at 10:15:02PM -0400, Terrence Enger wrote: I am chasing some leaked ODBC statement handles. I see that ODatabaseMetaDataResultSet.cxx takes care

leaked ODBC statement handles

2012-07-16 Thread Terrence Enger
Greetings, I am chasing some leaked ODBC statement handles. I see that ODatabaseMetaDataResultSet.cxx takes care *not* to free a statement handle which has not been subjected to one of 13 member functions with names starting open Questions arising ... (1) Why should this protection