[Libreoffice-commits] core.git: sfx2/source

[Stephan Bergmann]

 sfx2/source/appl/appdde.cxx |    7 +++++++
 1 file changed, 7 insertions(+)

New commits:
commit 04f4d4bfb708ef477618fa47d5f17779f4e4cf3c
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Jul 30 09:25:32 2013 +0200

    Add TODO comment expressing doubts about existing code
    
    Change-Id: I5fd8aa43da4ea879ec2adf31bb904a157b3425b0

diff --git a/sfx2/source/appl/appdde.cxx b/sfx2/source/appl/appdde.cxx
index 5f2ab9c..563d378 100644
--- a/sfx2/source/appl/appdde.cxx
+++ b/sfx2/source/appl/appdde.cxx
@@ -256,6 +256,13 @@ sal_Bool SfxAppEvent_Impl( const OUString& rCmd, const OUString& rEvent,
         if ( rCmd.getLength() - start >= 2 )
         {
             // Transform into the ApplicationEvent Format
+            //TODO: I /assume/ that rCmd should match the syntax of
+            // <http://msdn.microsoft.com/en-us/library/ms648995.aspx>
+            // "WM_DDE_EXECUTE message" but does not (handle commands enclosed
+            // in [...]; handle commas separating multiple arguments; handle
+            // double "", ((, )), [[, ]] in quoted arguments); see also the mail
+            // thread starting at <http://lists.freedesktop.org/archives/
+            // libreoffice/2013-July/054779.html> "DDE on Windows."
             std::vector<OUString> aData;
             for ( sal_Int32 n = start; n < rCmd.getLength() - 1; )
             {