Changeset 55961 in vbox

Timestamp:

May 20, 2015 8:24:07 AM (10 years ago)

Author:

vboxsync

Message:

apiwrap-server.xsl: don't throw autoCaller failure, do 'hrc=autoCaller.rc(); if (SUCCEEDED()) {...}' instead. MSC generates smaller and more streamlined code then (here at least).

File:

• trunk/src/VBox/Main/idl/apiwrap-server.xsl (modified) (13 diffs)

trunk/src/VBox/Main/idl/apiwrap-server.xsl

@@ -1237 +1237 @@
     </xsl:if>
     <xsl:text>
-
         </xsl:text>
     <xsl:choose>
@@ -1248 +1247 @@
     </xsl:choose>
     <xsl:text> autoCaller(this);
-        if (FAILED(autoCaller.rc()))
-            throw autoCaller.rc();
-
-</xsl:text>
-    <xsl:value-of select="concat('        hrc = get', $attrbasename, '(')"/>
+        hrc = autoCaller.rc();
+        if (SUCCEEDED(hrc))
+        {
+</xsl:text>
+    <xsl:value-of select="concat('            hrc = get', $attrbasename, '(')"/>
     <xsl:variable name="passAutoCaller">
         <xsl:call-template name="checkoption">
@@ -1266 +1265 @@
     </xsl:apply-templates>
     <xsl:text>);
-</xsl:text>
+        }</xsl:text>
     <xsl:if test="$attrbasename != 'MidlDoesNotLikeEmptyInterfaces'">
         <xsl:text>
@@ -1277 +1276 @@
         </xsl:call-template>
         <xsl:text>);
-#endif
-</xsl:text>
+#endif</xsl:text>
     </xsl:if>
     <xsl:text>
@@ -1293 +1291 @@
     <xsl:call-template name="emitDTraceParamValNoTmp-DirNotIn"/>
     <xsl:text>);
-#endif
-</xsl:text>
+#endif</xsl:text>
     </xsl:if>
     <xsl:text>
@@ -1309 +1306 @@
     <xsl:call-template name="emitDTraceParamValNoTmp-DirNotIn"/>
     <xsl:text>);
-#endif
-</xsl:text>
+#endif</xsl:text>
     </xsl:if>
     <xsl:text>
@@ -1381 +1377 @@
         <xsl:text>);
 #endif
-
         </xsl:text>
         <xsl:choose>
@@ -1392 +1387 @@
         </xsl:choose>
         <xsl:text> autoCaller(this);
-        if (FAILED(autoCaller.rc()))
-            throw autoCaller.rc();
-
-</xsl:text>
-        <xsl:value-of select="concat('        hrc = set', $attrbasename, '(')"/>
+        hrc = autoCaller.rc();
+        if (SUCCEEDED(hrc))
+        {
+</xsl:text>
+        <xsl:value-of select="concat('            hrc = set', $attrbasename, '(')"/>
         <xsl:if test="$passAutoCaller = 'true'">
             <xsl:text>autoCaller, </xsl:text>
@@ -1404 +1399 @@
         </xsl:apply-templates>
         <xsl:text>);
-
+        }
 #ifdef VBOX_WITH_DTRACE_R3_MAIN
         </xsl:text>
@@ -1804 +1799 @@
     </xsl:for-each>
     <xsl:text>
+
 #ifdef VBOX_WITH_DTRACE_R3_MAIN
         </xsl:text>
@@ -1816 +1812 @@
     <xsl:text>);
 #endif
-
         </xsl:text>
     <xsl:choose>
@@ -1827 +1822 @@
     </xsl:choose>
     <xsl:text> autoCaller(this);
-        if (FAILED(autoCaller.rc()))
-            throw autoCaller.rc();
-
-</xsl:text>
-    <xsl:value-of select="concat('        hrc = ', @name, '(')"/>
+        hrc = autoCaller.rc();
+        if (SUCCEEDED(hrc))
+        {
+</xsl:text>
+    <xsl:value-of select="concat('            hrc = ', @name, '(')"/>
     <xsl:variable name="passAutoCaller">
         <xsl:call-template name="checkoption">
@@ -1852 +1847 @@
         <xsl:if test="not(position()=last())">
             <xsl:text>,
-               </xsl:text>
+                   </xsl:text>
             <xsl:value-of select="$methodindent"/>
         </xsl:if>
     </xsl:for-each>
     <xsl:text>);
-
+        }
 #ifdef VBOX_WITH_DTRACE_R3_MAIN
         </xsl:text>