diff options
author | Lionel Elie Mamane <lionel@mamane.lu> | 2011-11-20 17:37:39 +0100 |
---|---|---|
committer | Lionel Elie Mamane <lionel@mamane.lu> | 2017-06-24 15:22:50 +0200 |
commit | 31b8ba7f8a8efbc6e04023c6138e17738d07a490 (patch) | |
tree | 3e207f0641fd0ef98ce1421b57b50d33cd00a473 | |
parent | 134a0b8cf0812865659b065055b488b480af19d3 (diff) |
LEM TODO note
-rw-r--r-- | connectivity/source/drivers/ado/ADriver.cxx | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/connectivity/source/drivers/ado/ADriver.cxx b/connectivity/source/drivers/ado/ADriver.cxx index 7d14a58206ae..ea193df66ebc 100644 --- a/connectivity/source/drivers/ado/ADriver.cxx +++ b/connectivity/source/drivers/ado/ADriver.cxx @@ -54,6 +54,13 @@ ODriver::ODriver(const css::uno::Reference< css::lang::XMultiServiceFactory >& _ ++h; } } +// LEM TODO: this h business above is fishy. The value is incremented and nothing done with it? +// The return value of CoInitializeEx is not checked the second time? +// Will calls to CoInitializeEx and CoUninitialize be balanced? +// Why is CoInitialize called again in destructor? +// See http://msdn.microsoft.com/en-us/library/windows/desktop/ms695279(v=VS.85).aspx +// Need to understand what runs on which thread, since this initialisation business is per-thread +// ------------------------------------------------------------------------- ODriver::~ODriver() { |