StdLib: Fix more GCC warnings/errors caused by variables being set but not used.

Removed variables that had no effect on code behavior.
Normalized comment formatting.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Olivier Martin <olivier.martin@arm.com>
Reviewed by: Daryl McDaniel <daryl.mcdaniel@intel.com>


git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16286 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
Olivier Martin 2014-10-31 20:32:02 +00:00 committed by darylm503
parent fe7ad7f6af
commit b81cc7d624
9 changed files with 68 additions and 113 deletions

View File

@ -1,4 +1,14 @@
/* /** @file
Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials are licensed and made available under
the terms and conditions of the BSD License that accompanies this distribution.
The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
* Copyright (C) 1995, 1996, 1997, and 1998 WIDE Project. * Copyright (C) 1995, 1996, 1997, and 1998 WIDE Project.
* All rights reserved. * All rights reserved.
* *
@ -25,14 +35,12 @@
* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
* OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
* SUCH DAMAGE. * SUCH DAMAGE.
*/
/*
* Issues to be discussed: * Issues to be discussed:
* - Thread safe-ness must be checked * - Thread safe-ness must be checked
* - Return values. There seems to be no standard for return value (RFC2133) * - Return values. There seems to be no standard for return value (RFC2133)
* but INRIA implementation returns EAI_xxx defined for getaddrinfo(). * but INRIA implementation returns EAI_xxx defined for getaddrinfo().
*/ **/
#if 0 #if 0
#include <sys/types.h> #include <sys/types.h>
@ -104,8 +112,8 @@ getnameinfo(sa, salen, host, hostlen, serv, servlen, flags)
u_long v4a; u_long v4a;
#ifdef ENABLE_IPV6 #ifdef ENABLE_IPV6
u_char pfx; u_char pfx;
#endif
int h_error; int h_error;
#endif
char numserv[512]; char numserv[512];
char numaddr[512]; char numaddr[512];
@ -181,7 +189,7 @@ getnameinfo(sa, salen, host, hostlen, serv, servlen, flags)
hp = getipnodebyaddr(addr, gni_afd->a_addrlen, gni_afd->a_af, &h_error); hp = getipnodebyaddr(addr, gni_afd->a_addrlen, gni_afd->a_af, &h_error);
#else #else
hp = gethostbyaddr(addr, gni_afd->a_addrlen, gni_afd->a_af); hp = gethostbyaddr(addr, gni_afd->a_addrlen, gni_afd->a_af);
h_error = h_errno; //h_error = h_errno;
#endif #endif
if (hp) { if (hp) {

View File

@ -1,6 +1,15 @@
/** @file
Parser-tokenizer link implementation.
/* Parser-tokenizer link implementation */ Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials are licensed and made available under
the terms and conditions of the BSD License that accompanies this distribution.
The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/
#include "pgenheaders.h" #include "pgenheaders.h"
#include "tokenizer.h" #include "tokenizer.h"
#include "node.h" #include "node.h"
@ -131,7 +140,9 @@ parsetok(struct tok_state *tok, grammar *g, int start, perrdetail *err_ret,
{ {
parser_state *ps; parser_state *ps;
node *n; node *n;
int started = 0, handling_import = 0, handling_with = 0; int started = 0;
//int handling_import = 0;
//int handling_with = 0;
if ((ps = PyParser_New(g, start)) == NULL) { if ((ps = PyParser_New(g, start)) == NULL) {
fprintf(stderr, "no mem for new parser\n"); fprintf(stderr, "no mem for new parser\n");
@ -163,7 +174,8 @@ parsetok(struct tok_state *tok, grammar *g, int start, perrdetail *err_ret,
} }
if (type == ENDMARKER && started) { if (type == ENDMARKER && started) {
type = NEWLINE; /* Add an extra newline */ type = NEWLINE; /* Add an extra newline */
handling_with = handling_import = 0; //handling_with = 0;
//handling_import = 0;
started = 0; started = 0;
/* Add the right number of dedent tokens, /* Add the right number of dedent tokens,
except if a certain flag is given -- except if a certain flag is given --

View File

@ -1,17 +1,15 @@
/** @file /** @file
Translate the host name into an IP address Translate the host name into an IP address
Copyright (c) 2011-2012, Intel Corporation Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>
All rights reserved. This program and the accompanying materials This program and the accompanying materials are licensed and made available under
are licensed and made available under the terms and conditions of the BSD License the terms and conditions of the BSD License that accompanies this distribution.
which accompanies this distribution. The full text of the license may be found at The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/ **/
#include <errno.h> #include <errno.h>
#include <netdb.h> #include <netdb.h>
#include <string.h> #include <string.h>
@ -28,8 +26,7 @@ struct hostent * _gethostbydnsname (const char *, int);
char mBuffer[65536]; char mBuffer[65536];
/** /** Translate the host name into an IP address
Translate the host name into an IP address
@param [in] Argc The number of arguments @param [in] Argc The number of arguments
@param [in] Argv The argument value array @param [in] Argv The argument value array
@ -43,7 +40,6 @@ main (
IN char **Argv IN char **Argv
) )
{ {
int AppStatus;
UINT8 * pIpAddress; UINT8 * pIpAddress;
struct hostent * pHost; struct hostent * pHost;
@ -51,17 +47,12 @@ main (
"%a starting\r\n", "%a starting\r\n",
Argv[0])); Argv[0]));
//
// Determine if the host name is specified // Determine if the host name is specified
//
AppStatus = 0;
if ( 1 == Argc ) { if ( 1 == Argc ) {
Print ( L"%a <host name>\r\n", Argv[0]); Print ( L"%a <host name>\r\n", Argv[0]);
} }
else { else {
//
// Translate the host name // Translate the host name
//
pHost = _gethostbydnsname ( Argv[1], AF_INET ); pHost = _gethostbydnsname ( Argv[1], AF_INET );
if ( NULL == pHost ) { if ( NULL == pHost ) {
Print ( L"ERROR - host not found, h_errno: %d\r\n", h_errno ); Print ( L"ERROR - host not found, h_errno: %d\r\n", h_errno );
@ -77,9 +68,6 @@ main (
pIpAddress[3]); pIpAddress[3]);
} }
} }
//
// All done // All done
//
return errno; return errno;
} }

View File

@ -1,17 +1,15 @@
/** @file /** @file
Translate the host name into an IP address Translate the host name into an IP address
Copyright (c) 2011-2012, Intel Corporation Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>
All rights reserved. This program and the accompanying materials This program and the accompanying materials are licensed and made available under
are licensed and made available under the terms and conditions of the BSD License the terms and conditions of the BSD License that accompanies this distribution.
which accompanies this distribution. The full text of the license may be found at The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/ **/
#include <errno.h> #include <errno.h>
#include <netdb.h> #include <netdb.h>
#include <string.h> #include <string.h>
@ -26,11 +24,10 @@
char mBuffer[65536]; char mBuffer[65536];
/** /** Translate the host name into an IP address
Translate the host name into an IP address
@param [in] Argc The number of arguments @param[in] Argc The number of arguments
@param [in] Argv The argument value array @param[in] Argv The argument value array
@retval 0 The application exited normally. @retval 0 The application exited normally.
@retval Other An error occurred. @retval Other An error occurred.
@ -41,7 +38,6 @@ main (
IN char **Argv IN char **Argv
) )
{ {
int AppStatus;
UINTN Index; UINTN Index;
struct hostent * pHost; struct hostent * pHost;
UINT8 * pIpAddress; UINT8 * pIpAddress;
@ -51,17 +47,12 @@ main (
"%a starting\r\n", "%a starting\r\n",
Argv[0])); Argv[0]));
//
// Determine if the host name is specified // Determine if the host name is specified
//
AppStatus = 0;
if ( 1 == Argc ) { if ( 1 == Argc ) {
Print ( L"%a <host name>\r\n", Argv[0]); Print ( L"%a <host name>\r\n", Argv[0]);
} }
else { else {
//
// Translate the host name // Translate the host name
//
pHost = gethostbyname ( Argv[1]); pHost = gethostbyname ( Argv[1]);
if ( NULL == pHost ) { if ( NULL == pHost ) {
Print ( L"ERROR - host not found, h_errno: %d\r\n", h_errno ); Print ( L"ERROR - host not found, h_errno: %d\r\n", h_errno );
@ -76,9 +67,7 @@ main (
pHost->h_addrtype, pHost->h_addrtype,
pHost->h_name ); pHost->h_name );
//
// Display the other addresses // Display the other addresses
//
for ( Index = 1; NULL != pHost->h_addr_list[Index]; Index++ ) { for ( Index = 1; NULL != pHost->h_addr_list[Index]; Index++ ) {
pIpAddress = (UINT8 *)pHost->h_addr_list[Index]; pIpAddress = (UINT8 *)pHost->h_addr_list[Index];
Print ( L"%d.%d.%d.%d\r\n", Print ( L"%d.%d.%d.%d\r\n",
@ -88,9 +77,7 @@ main (
pIpAddress[3]); pIpAddress[3]);
} }
//
// Display the list of aliases // Display the list of aliases
//
ppName = pHost->h_aliases; ppName = pHost->h_aliases;
if (( NULL == ppName ) || ( NULL == *ppName )) { if (( NULL == ppName ) || ( NULL == *ppName )) {
Print ( L"No aliases\r\n" ); Print ( L"No aliases\r\n" );
@ -115,9 +102,6 @@ main (
} }
} }
} }
//
// All done // All done
//
return errno; return errno;
} }

View File

@ -1,17 +1,15 @@
/** @file /** @file
Translate the network name into an IP address Translate the network name into an IP address
Copyright (c) 2011-2012, Intel Corporation Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>
All rights reserved. This program and the accompanying materials This program and the accompanying materials are licensed and made available under
are licensed and made available under the terms and conditions of the BSD License the terms and conditions of the BSD License that accompanies this distribution.
which accompanies this distribution. The full text of the license may be found at The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/ **/
#include <errno.h> #include <errno.h>
#include <netdb.h> #include <netdb.h>
#include <string.h> #include <string.h>
@ -26,11 +24,10 @@
char mBuffer[65536]; char mBuffer[65536];
/** /** Translate the network name into an IP address
Translate the network name into an IP address
@param [in] Argc The number of arguments @param[in] Argc The number of arguments
@param [in] Argv The argument value array @param[in] Argv The argument value array
@retval 0 The application exited normally. @retval 0 The application exited normally.
@retval Other An error occurred. @retval Other An error occurred.
@ -41,7 +38,6 @@ main (
IN char **Argv IN char **Argv
) )
{ {
int AppStatus;
UINT8 * pIpAddress; UINT8 * pIpAddress;
struct netent * pNetwork; struct netent * pNetwork;
@ -49,17 +45,12 @@ main (
"%a starting\r\n", "%a starting\r\n",
Argv[0])); Argv[0]));
//
// Determine if the network name is specified // Determine if the network name is specified
//
AppStatus = 0;
if ( 1 == Argc ) { if ( 1 == Argc ) {
Print ( L"%a <network name>\r\n", Argv[0]); Print ( L"%a <network name>\r\n", Argv[0]);
} }
else { else {
//
// Translate the net name // Translate the net name
//
pNetwork = getnetbyname ( Argv[1]); pNetwork = getnetbyname ( Argv[1]);
if ( NULL == pNetwork ) { if ( NULL == pNetwork ) {
Print ( L"ERROR - network not found, errno: %d\r\n", errno ); Print ( L"ERROR - network not found, errno: %d\r\n", errno );
@ -75,9 +66,6 @@ main (
pIpAddress[3]); pIpAddress[3]);
} }
} }
//
// All done // All done
//
return errno; return errno;
} }

View File

@ -1,17 +1,15 @@
/** @file /** @file
Translate the service name into a port number Translate the service name into a port number
Copyright (c) 2011-2012, Intel Corporation Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>
All rights reserved. This program and the accompanying materials This program and the accompanying materials are licensed and made available under
are licensed and made available under the terms and conditions of the BSD License the terms and conditions of the BSD License that accompanies this distribution.
which accompanies this distribution. The full text of the license may be found at The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/ **/
#include <errno.h> #include <errno.h>
#include <netdb.h> #include <netdb.h>
#include <string.h> #include <string.h>
@ -26,11 +24,10 @@
char mBuffer[65536]; char mBuffer[65536];
/** /** Translate the service name into a port number
Translate the service name into a port number
@param [in] Argc The number of arguments @param[in] Argc The number of arguments
@param [in] Argv The argument value array @param[in] Argv The argument value array
@retval 0 The application exited normally. @retval 0 The application exited normally.
@retval Other An error occurred. @retval Other An error occurred.
@ -41,21 +38,15 @@ main (
IN char **Argv IN char **Argv
) )
{ {
int AppStatus;
int PortNumber; int PortNumber;
struct servent * pService; struct servent * pService;
//
// Determine if the service name is specified // Determine if the service name is specified
//
AppStatus = 0;
if ( 1 == Argc ) { if ( 1 == Argc ) {
Print ( L"%a <service name>\r\n", Argv[0]); Print ( L"%a <service name>\r\n", Argv[0]);
} }
else { else {
//
// Translate the service name // Translate the service name
//
pService = getservbyname ( Argv[1], NULL ); pService = getservbyname ( Argv[1], NULL );
if ( NULL == pService ) { if ( NULL == pService ) {
Print ( L"ERROR - service not found, errno: %d\r\n", errno ); Print ( L"ERROR - service not found, errno: %d\r\n", errno );
@ -68,9 +59,6 @@ main (
pService->s_proto ); pService->s_proto );
} }
} }
//
// All done // All done
//
return errno; return errno;
} }

View File

@ -1,17 +1,15 @@
/** @file /** @file
Translate the port number into a service name Translate the port number into a service name
Copyright (c) 2011-2012, Intel Corporation Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>
All rights reserved. This program and the accompanying materials This program and the accompanying materials are licensed and made available under
are licensed and made available under the terms and conditions of the BSD License the terms and conditions of the BSD License that accompanies this distribution.
which accompanies this distribution. The full text of the license may be found at The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/ **/
#include <errno.h> #include <errno.h>
#include <netdb.h> #include <netdb.h>
#include <stdio.h> #include <stdio.h>
@ -27,11 +25,10 @@
char mBuffer[65536]; char mBuffer[65536];
/** /** Translate the port number into a service name
Translate the port number into a service name
@param [in] Argc The number of arguments @param[in] Argc The number of arguments
@param [in] Argv The argument value array @param[in] Argv The argument value array
@retval 0 The application exited normally. @retval 0 The application exited normally.
@retval Other An error occurred. @retval Other An error occurred.
@ -42,22 +39,16 @@ main (
IN char **Argv IN char **Argv
) )
{ {
int AppStatus;
int PortNumber; int PortNumber;
struct servent * pService; struct servent * pService;
//
// Determine if the service name is specified // Determine if the service name is specified
//
AppStatus = 0;
if (( 2 != Argc ) if (( 2 != Argc )
|| ( 1 != sscanf ( Argv[1], "%d", &PortNumber ))) { || ( 1 != sscanf ( Argv[1], "%d", &PortNumber ))) {
Print ( L"%a <port number>\r\n", Argv[0]); Print ( L"%a <port number>\r\n", Argv[0]);
} }
else { else {
//
// Translate the port number // Translate the port number
//
pService = getservbyport ( htons ( PortNumber ), NULL ); pService = getservbyport ( htons ( PortNumber ), NULL );
if ( NULL == pService ) { if ( NULL == pService ) {
Print ( L"ERROR - service not found, errno: %d\r\n", errno ); Print ( L"ERROR - service not found, errno: %d\r\n", errno );
@ -69,9 +60,6 @@ main (
pService->s_proto ); pService->s_proto );
} }
} }
//
// All done // All done
//
return errno; return errno;
} }

View File

@ -4006,6 +4006,7 @@ EslSocketPortCloseComplete (
// Determine if the receive operation is pending // Determine if the receive operation is pending
Status = EslSocketPortCloseRxDone ( pPort ); Status = EslSocketPortCloseRxDone ( pPort );
DBG_EXIT_STATUS ( Status ); DBG_EXIT_STATUS ( Status );
--Status;
} }

View File

@ -586,7 +586,6 @@ da_ConPoll(
short events short events
) )
{ {
EFI_SIMPLE_TEXT_INPUT_PROTOCOL *Proto;
ConInstance *Stream; ConInstance *Stream;
EFI_STATUS Status = RETURN_SUCCESS; EFI_STATUS Status = RETURN_SUCCESS;
short RdyMask = 0; short RdyMask = 0;
@ -600,7 +599,6 @@ da_ConPoll(
} }
if(Stream->InstanceNum == 0) { if(Stream->InstanceNum == 0) {
// STDIN: Only input is supported for this device // STDIN: Only input is supported for this device
Proto = (EFI_SIMPLE_TEXT_INPUT_PROTOCOL *)Stream->Dev;
Status = da_ConRawRead (filp, &Stream->UnGetKey); Status = da_ConRawRead (filp, &Stream->UnGetKey);
if(Status == RETURN_SUCCESS) { if(Status == RETURN_SUCCESS) {
RdyMask = POLLIN; RdyMask = POLLIN;