Andrew Lunn wrote:
Hi
This patch fixes a compiler warnings in the mbox code with recent
versions of gcc.
We should avoid this for time-sensitive code. It should be tagged with
__attribute__((unused)) (aka CYGBLD_ATTRIB_UNUSED) instead. In fact all
uninitialised variable warnings should be treated this way.
Jifl
------------------------------------------------------------------------
Index: kernel/current/ChangeLog
===================================================================
RCS file: /cvs/ecos/ecos/packages/kernel/current/ChangeLog,v
retrieving revision 1.139
diff -u -r1.139 ChangeLog
--- kernel/current/ChangeLog 12 Oct 2006 15:41:49 -0000 1.139
+++ kernel/current/ChangeLog 7 Jan 2007 17:08:13 -0000
@@ -1,3 +1,8 @@
+2007-01-07 Andrew Lunn <[EMAIL PROTECTED]>
+
+ * src/sync/mbox.cxx (Cyg_Mbox::get): Fix compiler warning with gcc
+ version 4.1.2.
+
2006-10-12 Nick Garnett <[EMAIL PROTECTED]>
* cdl/synch.cdl: Added CYGIMP_MBOX_USE_MBOXT_PLAIN option. This is
Index: kernel/current/src/sync/mbox.cxx
===================================================================
RCS file: /cvs/ecos/ecos/packages/kernel/current/src/sync/mbox.cxx,v
retrieving revision 1.7
diff -u -r1.7 mbox.cxx
--- kernel/current/src/sync/mbox.cxx 30 Jul 2005 11:58:25 -0000 1.7
+++ kernel/current/src/sync/mbox.cxx 7 Jan 2007 17:08:13 -0000
@@ -109,7 +109,7 @@
void *
Cyg_Mbox::get()
{
- void * p;
+ void * p=NULL;
if ( ! m.get( p ) )
return NULL;
return p;
@@ -119,7 +119,7 @@
void *
Cyg_Mbox::get( cyg_tick_count timeout )
{
- void * p;
+ void * p=NULL;
if ( ! m.get( p, timeout ) )
return NULL;
return p;
--
------["The best things in life aren't things."]------ Opinions==mine
