[Libreoffice-commits] core.git: include/LibreOfficeKit

Tor Lillqvist tml at collabora.com
Wed Jan 10 15:31:01 UTC 2018


 include/LibreOfficeKit/LibreOfficeKit.hxx |    5 +++++
 1 file changed, 5 insertions(+)

New commits:
commit 4e5716b57c4ce75d35c76ea26a16d8d0acadaa76
Author: Tor Lillqvist <tml at collabora.com>
Date:   Wed Jan 10 17:28:20 2018 +0200

    Mention that the getError() semantics are unclear
    
    Change-Id: Ifd5b04cc13085749f0f800bdb69cc3fdd5581916

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);


More information about the Libreoffice-commits mailing list