No need to check out execv return value.

If we ever return from execv, it is an error and the code already handle that.
This fixes a dead assignment warning under CLang++.

1 files changed, 3 insertions, 2 deletions

diff --git a/sal/osl/unx/process.c b/sal/osl/unx/process.c
index b41301c9e..7d65b5499 100644
--- a/sal/osl/unx/process.c
+++ b/sal/osl/unx/process.c
@@ -544,8 +544,9 @@ static void ChildStatusProc(void *pData)
                 if (stdError[1] != -1) close( stdError[1] );
             }
 
-            pid=execv(data.m_pszArgs[0], (sal_Char **)data.m_pszArgs);
-
+            // No need to check the return value of execv. If we return from
+            // it, an error has occurred.
+            execv(data.m_pszArgs[0], (sal_Char **)data.m_pszArgs);
         }
 
         OSL_TRACE("Failed to exec, errno=%d (%s)\n", errno, strerror(errno));