Piotr Caban : oleview: Fixed incorrect brackets bug.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 2e80f11c5e9bf9da3163231e66d251d6780fd0d4
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=2e80f11c5e9bf9da3163231e66d251d6780fd0d4

Author: Piotr Caban <piotr.caban at gmail.com>
Date:   Fri Apr 11 22:58:26 2008 +0200

oleview: Fixed incorrect brackets bug.

---

 programs/oleview/typelib.c |   35 +++++++++++++++++++----------------
 1 files changed, 19 insertions(+), 16 deletions(-)

diff --git a/programs/oleview/typelib.c b/programs/oleview/typelib.c
index 8143906..5dbffb0 100644
--- a/programs/oleview/typelib.c
+++ b/programs/oleview/typelib.c
@@ -545,20 +545,7 @@ static int EnumFuncs(ITypeInfo *pTypeInfo, TYPEATTR *pTypeAttr, HTREEITEM hParen
             AddToTLDataStrW(tld, wszCloseBrackets2);
             memset(wszText, 0, sizeof(wszText));
         }
-        if(SysStringLen(bstrHelpString)) {
-            if(bFirst) AddToTLDataStrW(tld, wszOpenBrackets1);
-            else {
-                AddToTLDataStrW(tld, wszComa);
-                AddToTLDataStrW(tld, wszSpace);
-            }
-            bFirst = FALSE;
-            AddToTLDataStrW(tld, wszHelpstring);
-            AddToTLDataStrW(tld, wszOpenBrackets2);
-            AddToTLDataStrW(tld, wszInvertedComa);
-            AddToTLDataStrW(tld, bstrHelpString);
-            AddToTLDataStrW(tld, wszInvertedComa);
-            AddToTLDataStrW(tld, wszCloseBrackets2);
-        }
+
         CreateTypeInfo(wszText, wszAfter, pFuncDesc->elemdescFunc.tdesc, pTypeInfo);
         switch(pFuncDesc->invkind)
         {
@@ -594,6 +581,22 @@ static int EnumFuncs(ITypeInfo *pTypeInfo, TYPEATTR *pTypeAttr, HTREEITEM hParen
                 break;
             default:;
         }
+        if(SysStringLen(bstrHelpString))
+        {
+            if(bFirst) AddToTLDataStrW(tld, wszOpenBrackets1);
+            else
+            {
+                AddToTLDataStrW(tld, wszComa);
+                AddToTLDataStrW(tld, wszSpace);
+            }
+            bFirst = FALSE;
+            AddToTLDataStrW(tld, wszHelpstring);
+            AddToTLDataStrW(tld, wszOpenBrackets2);
+            AddToTLDataStrW(tld, wszInvertedComa);
+            AddToTLDataStrW(tld, bstrHelpString);
+            AddToTLDataStrW(tld, wszInvertedComa);
+            AddToTLDataStrW(tld, wszCloseBrackets2);
+        }
         if(!bFirst)
         {
             AddToTLDataStrW(tld, wszCloseBrackets1);
@@ -1104,11 +1107,11 @@ static void CreateCoclassHeader(ITypeInfo *pTypeInfo,
             AddToTLDataStrW(pTLData, wszNewLine);
             AddSpaces(pTLData, TAB_SIZE);
             AddToTLDataStrW(pTLData, wszHelpstring);
-            AddToTLDataStrW(pTLData, wszOpenBrackets1);
+            AddToTLDataStrW(pTLData, wszOpenBrackets2);
             AddToTLDataStrW(pTLData, wszInvertedComa);
             AddToTLDataStrW(pTLData, bstrHelpString);
             AddToTLDataStrW(pTLData, wszInvertedComa);
-            AddToTLDataStrW(pTLData, wszCloseBrackets1);
+            AddToTLDataStrW(pTLData, wszCloseBrackets2);
         }
         SysFreeString(bstrHelpString);
     }