Linux Ethernet Bridging - Mar 2009 - [Bridge] [PATCH] rstp compiling under gcc 4.3.3 fixes

After fetching current git code and compiling with gcc 4.3.3 got errors
Here is fix, so rstp compile fine with gcc 4.3.3

bridge_track.c and main.c have functions where is return value not checked.
 Because of -Werror it won't compile.
Example:
gcc -Wall -Werror -O2 -g -D_REENTRANT -D__LINUX__ -DVERSION=0.16 -DBUILD=1 -I.
-I./include -I./rstplib   -c -o
main.o main.c
cc1: warnings being treated as errors
main.c: In function 'main':
main.c:81: error: ignoring return value of 'daemon', declared with
attribute
warn_unused_result


Other fix is missing header file
gcc -Wall -O2 -g -D_REENTRANT -D__LINUX__ -DVERSION=0.16 -DBUILD=1 -I.
-I./include -I./rstplib   -c -o
netif_utils.o netif_utils.c
netif_utils.c: In function 'get_bridge_portno':
netif_utils.c:130: error: 'INT_MAX' undeclared (first use in this
function)
netif_utils.c:130: error: (Each undeclared identifier is reported only once
netif_utils.c:130: error: for each function it appears in.)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rstp-fixes.diff
Type: text/x-diff
Size: 1872 bytes
Desc: not available
Url :
http://lists.linux-foundation.org/pipermail/bridge/attachments/20090328/cbbc0d38/attachment.diff

Sorry, properly formatted

rstp git tree:
After fetching current git code and compiling with gcc 4.3.3 got errors 
related to Werror (2 functions was ignoring return value), and ulimits.h was 
not declared, but INT_MAX used
Here is fix, so rstp compile fine with gcc 4.3.3.

Signed-off-by: Denys Fedoryschenko <denys at visp.net.lb>
---
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rstp-fixes.diff
Type: text/x-diff
Size: 1872 bytes
Desc: not available
Url :
http://lists.linux-foundation.org/pipermail/bridge/attachments/20090328/1b592eed/attachment.diff

Can anybody just send me any diagrammatic representation of the code flow of
"rstpd" source code which is an updated version of
"rstplib". It will help me a lot to get into the depth of
understanding the code  ??
Kindly also list some imporatnt differences/updations in "rstpd" as
compared to "rstplib"??


-----Original Message-----
From: bridge-bounces at lists.linux-foundation.org [mailto:bridge-bounces at
lists.linux-foundation.org] On Behalf Of Denys Fedoryschenko
Sent: Saturday, March 28, 2009 7:07 PM
To: Stephen Hemminger; bridge at lists.linux-foundation.org
Subject: [Bridge] [PATCH] rstp compiling under gcc 4.3.3 fixes

After fetching current git code and compiling with gcc 4.3.3 got errors
Here is fix, so rstp compile fine with gcc 4.3.3

bridge_track.c and main.c have functions where is return value not checked.
 Because of -Werror it won't compile.
Example:
gcc -Wall -Werror -O2 -g -D_REENTRANT -D__LINUX__ -DVERSION=0.16 -DBUILD=1 -I.
-I./include -I./rstplib   -c -o
main.o main.c
cc1: warnings being treated as errors
main.c: In function 'main':
main.c:81: error: ignoring return value of 'daemon', declared with
attribute
warn_unused_result


Other fix is missing header file
gcc -Wall -O2 -g -D_REENTRANT -D__LINUX__ -DVERSION=0.16 -DBUILD=1 -I.
-I./include -I./rstplib   -c -o
netif_utils.o netif_utils.c
netif_utils.c: In function 'get_bridge_portno':
netif_utils.c:130: error: 'INT_MAX' undeclared (first use in this
function)
netif_utils.c:130: error: (Each undeclared identifier is reported only once
netif_utils.c:130: error: for each function it appears in.)

DISCLAIMER:
-----------------------------------------------------------------------------------------------------------------------
The contents of this e-mail and any attachment(s) are confidential and intended
for the named recipient(s) only.
It shall not attach any liability on the originator or HCL or its affiliates.
Any views or opinions presented in
this email are solely those of the author and may not necessarily reflect the
opinions of HCL or its affiliates.
Any form of reproduction, dissemination, copying, disclosure, modification,
distribution and / or publication of
this message without the prior written consent of the author of this e-mail is
strictly prohibited. If you have
received this email in error please delete it and notify the sender immediately.
Before opening any mail and
attachments please check them for viruses and defect.
-----------------------------------------------------------------------------------------------------------------------