[ros-diffs] [hbelusca] 63206: [CABMAN] In any case, we are returning the (!= success) status in case we failed to perform the file copy, so just test for != success just one time. Patch by Victor Martinez. COR...

Author: hbelusca
Date: Fri May  9 20:25:41 2014
New Revision: 63206

URL: http://svn.reactos.org/svn/reactos?rev=63206&view=rev
Log:
[CABMAN]
In any case, we are returning the (!= success) status in case we failed to perform the file copy, so just test for != success just one time.
Patch by Victor Martinez.
CORE-8189 #resolve

Modified:
    trunk/reactos/tools/cabman/dfp.cxx

Modified: trunk/reactos/tools/cabman/dfp.cxx
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/cabman/dfp.cxx?rev=63...
==============================================================================
--- trunk/reactos/tools/cabman/dfp.cxx	[iso-8859-1] (original)
+++ trunk/reactos/tools/cabman/dfp.cxx	[iso-8859-1] Fri May  9 20:25:41 2014
@@ -361,10 +361,8 @@
                             {
                                 printf("ERROR: Directive file contains errors at line %u.\n", (UINT)CurrentLine);
                                 DPRINT(MID_TRACE, ("Error while copying file.\n"));
+                                return Status;
                             }
-
-                            if (Status != CAB_STATUS_SUCCESS)
-                                return Status;
                         }
                         break;