diff options
author | Tor Lillqvist <tml@collabora.com> | 2018-01-10 17:28:20 +0200 |
---|---|---|
committer | Tor Lillqvist <tml@collabora.com> | 2018-01-10 17:30:08 +0200 |
commit | 4e5716b57c4ce75d35c76ea26a16d8d0acadaa76 (patch) | |
tree | 57661a9b6a3c06c182c91fcbf68a9b81ea7da678 /include | |
parent | d2763bde00f0f840585e1ec4c8f94c0ef804a580 (diff) |
Mention that the getError() semantics are unclear
Change-Id: Ifd5b04cc13085749f0f800bdb69cc3fdd5581916
Diffstat (limited to 'include')
-rw-r--r-- | include/LibreOfficeKit/LibreOfficeKit.hxx | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/include/LibreOfficeKit/LibreOfficeKit.hxx b/include/LibreOfficeKit/LibreOfficeKit.hxx index 310b9cfcee88..e158ee62cab7 100644 --- a/include/LibreOfficeKit/LibreOfficeKit.hxx +++ b/include/LibreOfficeKit/LibreOfficeKit.hxx @@ -583,6 +583,11 @@ public: } /// Returns the last error as a string, the returned pointer has to be freed by the caller. + + /// Exact semantics somewhat unclear (sometimes the code clears the string that the next call to + /// getError() will return if no error happens in another function, sometimes not), and + /// unfortunately cleaning up that is harder than it seems, because of lovely global variables + /// and a unit test that uses the LibreOfficeKit API in an untypical manner. char* getError() { return mpThis->pClass->getError(mpThis); |