change log for rtems (2011-05-05)

rtems-vc at rtems.org rtems-vc at rtems.org
Thu May 5 17:10:38 UTC 2011


 *joel*:
2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>

	* base_sp/apptask.c, base_sp/init.c, fileio/init.c, fileio/system.h,
	nsecs/init.c, ticker/init.c, ticker/system.h, ticker/tasks.c: Remove
	warnings.

M  1.188  testsuites/samples/ChangeLog
M   1.16  testsuites/samples/base_sp/apptask.c
M   1.13  testsuites/samples/base_sp/init.c
M   1.32  testsuites/samples/fileio/init.c
M   1.17  testsuites/samples/fileio/system.h
M   1.15  testsuites/samples/nsecs/init.c
M   1.15  testsuites/samples/ticker/init.c
M   1.24  testsuites/samples/ticker/system.h
M   1.16  testsuites/samples/ticker/tasks.c

diff -u rtems/testsuites/samples/ChangeLog:1.187 rtems/testsuites/samples/ChangeLog:1.188
--- rtems/testsuites/samples/ChangeLog:1.187	Tue Feb 22 08:48:25 2011
+++ rtems/testsuites/samples/ChangeLog	Thu May  5 11:45:40 2011
@@ -1,3 +1,9 @@
+2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>
+
+	* base_sp/apptask.c, base_sp/init.c, fileio/init.c, fileio/system.h,
+	nsecs/init.c, ticker/init.c, ticker/system.h, ticker/tasks.c: Remove
+	warnings.
+
 2011-02-22	Ralf Corsépius <ralf.corsepius at rtems.org>
 
 	* configure.ac: Add AC_CONFIG_HEADER(config.h).

diff -u rtems/testsuites/samples/base_sp/apptask.c:1.15 rtems/testsuites/samples/base_sp/apptask.c:1.16
--- rtems/testsuites/samples/base_sp/apptask.c:1.15	Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/base_sp/apptask.c	Thu May  5 11:45:40 2011
@@ -8,7 +8,7 @@
  *
  *  Output parameters:  NONE
  *
- *  COPYRIGHT (c) 1989-1999.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -36,9 +36,12 @@
   rtems_status_code status;
 
   status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &tid );
+  directive_failed( status, "ident" );
 
-  printf( "Application task was invoked with argument (%" PRIdrtems_task_argument ") "
-          "and has id of 0x%" PRIxrtems_id "\n", argument, tid );
+  printf(
+    "Application task was invoked with argument (%" PRIdrtems_task_argument ") "
+    "and has id of 0x%" PRIxrtems_id "\n", argument, tid
+  );
 
   printf( "*** END OF SAMPLE SINGLE PROCESSOR APPLICATION ***\n" );
   exit( 0 );

diff -u rtems/testsuites/samples/base_sp/init.c:1.12 rtems/testsuites/samples/base_sp/init.c:1.13
--- rtems/testsuites/samples/base_sp/init.c:1.12	Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/base_sp/init.c	Thu May  5 11:45:40 2011
@@ -10,7 +10,7 @@
  *
  *  Output parameters:  NONE
  *
- *  COPYRIGHT (c) 1989-1999.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -26,6 +26,7 @@
 
 #define CONFIGURE_INIT
 #include "system.h"
+#include "tmacros.h"
 #include <stdio.h>
 
 #define ARGUMENT 0
@@ -45,8 +46,11 @@
 
   status = rtems_task_create( task_name, 1, RTEMS_MINIMUM_STACK_SIZE,
              RTEMS_INTERRUPT_LEVEL(0), RTEMS_DEFAULT_ATTRIBUTES, &tid );
+  directive_failed( status, "create" ); 
 
   status = rtems_task_start( tid, Application_task, ARGUMENT );
+  directive_failed( status, "start" ); 
 
   status = rtems_task_delete( RTEMS_SELF );
+  directive_failed( status, "delete" ); 
 }

diff -u rtems/testsuites/samples/fileio/init.c:1.31 rtems/testsuites/samples/fileio/init.c:1.32
--- rtems/testsuites/samples/fileio/init.c:1.31	Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/fileio/init.c	Thu May  5 11:45:40 2011
@@ -10,7 +10,7 @@
  *
  *  Output parameters:  NONE
  *
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -707,10 +707,13 @@
     RTEMS_DEFAULT_MODES ,
     RTEMS_FLOATING_POINT | RTEMS_DEFAULT_ATTRIBUTES, &Task_id
   );
+  directive_failed( status, "create" ); 
 
   status = rtems_task_start( Task_id, fileio_task, 1 );
+  directive_failed( status, "start" ); 
 
   status = rtems_task_delete( RTEMS_SELF );
+  directive_failed( status, "delete" ); 
 }
 
 #if defined(USE_SHELL)

diff -u rtems/testsuites/samples/fileio/system.h:1.16 rtems/testsuites/samples/fileio/system.h:1.17
--- rtems/testsuites/samples/fileio/system.h:1.16	Wed Aug 12 15:50:37 2009
+++ rtems/testsuites/samples/fileio/system.h	Thu May  5 11:45:40 2011
@@ -14,6 +14,7 @@
  */
 
 #include <rtems.h>
+#include "tmacros.h"
 
 /* functions */
 
@@ -63,56 +64,4 @@
 #define CONFIGURE_UNIFIED_WORK_AREAS
 #include <rtems/confdefs.h>
 
-/*
- *  Handy macros and static inline functions
- */
-
-/*
- *  Macro to hide the ugliness of printing the time.
- */
-
-#define print_time(_s1, _tb, _s2) \
-  do { \
-    printf( "%s%02d:%02d:%02d   %02d/%02d/%04d%s", \
-       _s1, (_tb)->hour, (_tb)->minute, (_tb)->second, \
-       (_tb)->month, (_tb)->day, (_tb)->year, _s2 ); \
-    fflush(stdout); \
-  } while ( 0 )
-
-/*
- *  Macro to print an task name that is composed of ASCII characters.
- *
- */
-
-#define put_name( _name, _crlf ) \
-  do { \
-    uint32_t   c0, c1, c2, c3; \
-    \
-    c0 = ((_name) >> 24) & 0xff; \
-    c1 = ((_name) >> 16) & 0xff; \
-    c2 = ((_name) >> 8) & 0xff; \
-    c3 = (_name) & 0xff; \
-    putchar( (char)c0 ); \
-    if ( c1 ) putchar( (char)c1 ); \
-    if ( c2 ) putchar( (char)c2 ); \
-    if ( c3 ) putchar( (char)c3 ); \
-    if ( (_crlf) ) \
-      putchar( '\n' ); \
-  } while (0)
-
-/*
- *  This allows us to view the "Test_task" instantiations as a set
- *  of numbered tasks by eliminating the number of application
- *  tasks created.
- *
- *  In reality, this is too complex for the purposes of this
- *  example.  It would have been easier to pass a task argument. :)
- *  But it shows how rtems_id's can sometimes be used.
- */
-
-#define task_number( tid ) \
-  ( rtems_object_id_get_index( tid ) - \
-      rtems_configuration_get_rtems_api_configuration()-> \
-        number_of_initialization_tasks )
-
 /* end of include file */

diff -u rtems/testsuites/samples/nsecs/init.c:1.14 rtems/testsuites/samples/nsecs/init.c:1.15
--- rtems/testsuites/samples/nsecs/init.c:1.14	Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/nsecs/init.c	Thu May  5 11:45:40 2011
@@ -2,7 +2,7 @@
  *  Nanoseconds accuracy timestamp test
  */
 
-/*  COPYRIGHT (c) 1989-2007.
+/*  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -69,6 +69,7 @@
   time.ticks  = 0;
 
   status = rtems_clock_set( &time );
+  directive_failed( status, "clock set" ); 
 
   /*
    *  Iterate 10 times showing difference in TOD
@@ -91,7 +92,7 @@
       my_ctime(stop.tv_sec), stop.tv_nsec
     );
 
-   printf( " --> %" PRItime_t ":%ld\n", diff.tv_sec, diff.tv_nsec );
+    printf( " --> %" PRItime_t ":%ld\n", diff.tv_sec, diff.tv_nsec );
   }
 
   /*

diff -u rtems/testsuites/samples/ticker/init.c:1.14 rtems/testsuites/samples/ticker/init.c:1.15
--- rtems/testsuites/samples/ticker/init.c:1.14	Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/ticker/init.c	Thu May  5 11:45:40 2011
@@ -10,7 +10,7 @@
  *
  *  Output parameters:  NONE
  *
- *  COPYRIGHT (c) 1989-1999.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -52,6 +52,7 @@
   time.ticks  = 0;
 
   status = rtems_clock_set( &time );
+  directive_failed( status, "clock get" );
 
   Task_name[ 1 ] = rtems_build_name( 'T', 'A', '1', ' ' );
   Task_name[ 2 ] = rtems_build_name( 'T', 'A', '2', ' ' );
@@ -61,18 +62,29 @@
     Task_name[ 1 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 1 ]
   );
+  directive_failed( status, "create 1" ); 
+
   status = rtems_task_create(
     Task_name[ 2 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 2 ]
   );
+  directive_failed( status, "create 2" ); 
+
   status = rtems_task_create(
     Task_name[ 3 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 3 ]
   );
+  directive_failed( status, "create 3" ); 
 
   status = rtems_task_start( Task_id[ 1 ], Test_task, 1 );
+  directive_failed( status, "start 1" ); 
+
   status = rtems_task_start( Task_id[ 2 ], Test_task, 2 );
+  directive_failed( status, "start 2" ); 
+
   status = rtems_task_start( Task_id[ 3 ], Test_task, 3 );
+  directive_failed( status, "start 3" ); 
 
   status = rtems_task_delete( RTEMS_SELF );
+  directive_failed( status, "delete" ); 
 }

diff -u rtems/testsuites/samples/ticker/system.h:1.23 rtems/testsuites/samples/ticker/system.h:1.24
--- rtems/testsuites/samples/ticker/system.h:1.23	Wed Aug 12 15:50:38 2009
+++ rtems/testsuites/samples/ticker/system.h	Thu May  5 11:45:40 2011
@@ -16,6 +16,7 @@
 #include <rtems.h>
 #include <inttypes.h>
 #include "../../support/include/buffer_test_io.h"
+#include "tmacros.h"
 
 /* functions */
 
@@ -53,42 +54,6 @@
 #include <rtems/confdefs.h>
 
 /*
- *  Handy macros and static inline functions
- */
-
-/*
- *  Macro to hide the ugliness of printing the time.
- */
-
-#define print_time(_s1, _tb, _s2) \
-  do { \
-    printf( "%s%02" PRId32 ":%02" PRId32 ":%02" PRId32 "   %02" PRId32 "/%02" PRId32 "/%04" PRId32 "%s", \
-       _s1, (_tb)->hour, (_tb)->minute, (_tb)->second, \
-       (_tb)->month, (_tb)->day, (_tb)->year, _s2 ); \
-  } while ( 0 )
-
-/*
- *  Macro to print an task name that is composed of ASCII characters.
- *
- */
-
-#define put_name( _name, _crlf ) \
-  do { \
-    uint32_t   c0, c1, c2, c3; \
-    \
-    c0 = ((_name) >> 24) & 0xff; \
-    c1 = ((_name) >> 16) & 0xff; \
-    c2 = ((_name) >> 8) & 0xff; \
-    c3 = (_name) & 0xff; \
-    putchar( (char)c0 ); \
-    if ( c1 ) putchar( (char)c1 ); \
-    if ( c2 ) putchar( (char)c2 ); \
-    if ( c3 ) putchar( (char)c3 ); \
-    if ( (_crlf) ) \
-      putchar( '\n' ); \
-  } while (0)
-
-/*
  *  This allows us to view the "Test_task" instantiations as a set
  *  of numbered tasks by eliminating the number of application
  *  tasks created.

diff -u rtems/testsuites/samples/ticker/tasks.c:1.15 rtems/testsuites/samples/ticker/tasks.c:1.16
--- rtems/testsuites/samples/ticker/tasks.c:1.15	Tue Feb 22 04:54:32 2011
+++ rtems/testsuites/samples/ticker/tasks.c	Thu May  5 11:45:40 2011
@@ -29,10 +29,12 @@
 {
   rtems_id          tid;
   rtems_time_of_day time;
-  uint32_t    task_index;
+  uint32_t          task_index;
   rtems_status_code status;
 
   status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &tid );
+  directive_failed( status, "task ident" ); 
+
   task_index = task_number( tid );
   for ( ; ; ) {
     status = rtems_clock_get_tod( &time );
@@ -43,6 +45,8 @@
     put_name( Task_name[ task_index ], FALSE );
     print_time( " - rtems_clock_get_tod - ", &time, "\n" );
     status = rtems_task_wake_after(
-      task_index * 5 * rtems_clock_get_ticks_per_second() );
+      task_index * 5 * rtems_clock_get_ticks_per_second()
+    );
+    directive_failed( status, "wake after" ); 
   }
 }


 *joel*:
2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>

	* devnullfatal01/testcase.h, malloc02/init.c, malloc04/init.c,
	malloctest/init.c, termios03/termios_testdriver_polled.c,
	termios04/termios_testdriver_intr.c: Remove warnings.

M  1.265  testsuites/libtests/ChangeLog
M    1.2  testsuites/libtests/devnullfatal01/testcase.h
M    1.4  testsuites/libtests/malloc02/init.c
M    1.5  testsuites/libtests/malloc04/init.c
M   1.46  testsuites/libtests/malloctest/init.c
M    1.4  testsuites/libtests/termios03/termios_testdriver_polled.c
M    1.4  testsuites/libtests/termios04/termios_testdriver_intr.c

diff -u rtems/testsuites/libtests/ChangeLog:1.264 rtems/testsuites/libtests/ChangeLog:1.265
--- rtems/testsuites/libtests/ChangeLog:1.264	Tue Apr  5 04:19:53 2011
+++ rtems/testsuites/libtests/ChangeLog	Thu May  5 11:45:47 2011
@@ -1,3 +1,9 @@
+2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>
+
+	* devnullfatal01/testcase.h, malloc02/init.c, malloc04/init.c,
+	malloctest/init.c, termios03/termios_testdriver_polled.c,
+	termios04/termios_testdriver_intr.c: Remove warnings.
+
 2011-04-05	Sebastian Huber <sebastian.huber at embedded-brains.de>
 
 	* tar01/init.c: Fixed integer types.

diff -u rtems/testsuites/libtests/devnullfatal01/testcase.h:1.1 rtems/testsuites/libtests/devnullfatal01/testcase.h:1.2
--- rtems/testsuites/libtests/devnullfatal01/testcase.h:1.1	Tue Jul 27 08:57:17 2010
+++ rtems/testsuites/libtests/devnullfatal01/testcase.h	Thu May  5 11:45:47 2011
@@ -16,12 +16,14 @@
 #define FATAL_ERROR_EXPECTED_ERROR       RTEMS_TOO_MANY
 
 #include <rtems/devnull.h>
+#include "tmacros.h"
 
 void force_error()
 {
   int status;
 
   status = rtems_io_register_name( "/dev/null", 0, 0 );
+  directive_failed( status, "io register" );
 
   status = null_initialize( 0, 0, NULL );
 

diff -u rtems/testsuites/libtests/malloc02/init.c:1.3 rtems/testsuites/libtests/malloc02/init.c:1.4
--- rtems/testsuites/libtests/malloc02/init.c:1.3	Tue Feb 22 01:34:07 2011
+++ rtems/testsuites/libtests/malloc02/init.c	Thu May  5 11:45:47 2011
@@ -69,6 +69,7 @@
   puts( "Free from ISR successfully processed" );
   puts( "Now malloc'ing more memory to process the free" );
   pointer2 = malloc(20);
+  rtems_test_assert( pointer2 );
 
   puts( "*** END OF TEST MALLOC 02 ***" );
   rtems_test_exit( 0 );

diff -u rtems/testsuites/libtests/malloc04/init.c:1.4 rtems/testsuites/libtests/malloc04/init.c:1.5
--- rtems/testsuites/libtests/malloc04/init.c:1.4	Tue Feb 22 01:34:07 2011
+++ rtems/testsuites/libtests/malloc04/init.c	Thu May  5 11:45:47 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2010.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -111,15 +111,23 @@
   sbrk_count = 0;
   RTEMS_Malloc_Initialize( NULL, 0, 64 );
   p1 = malloc(64);
+  rtems_test_assert( p1 );
+
   p2 = malloc(64);
+  rtems_test_assert( p2 );
+
   p3 = malloc(48);
+  rtems_test_assert( p3 );
+
   p4 = malloc(48);
+  rtems_test_assert( p4 );
   
   puts( "Initialize heap with no memory (sbrk aligned)" );
   offset     = 0;
   sbrk_count = 0;
   RTEMS_Malloc_Initialize( NULL, 0, 64 );
   p1 = malloc(64);
+  rtems_test_assert( p1 );
   
   puts( "Set sbrk amount in heap to 0" );
   offset     = 0;
@@ -127,6 +135,7 @@
   RTEMS_Malloc_Initialize( NULL, 0, 64 );
   RTEMS_Malloc_Sbrk_amount = 0;
   p4 = malloc(48);
+  rtems_test_assert( p4 );
 
   /* Restore information on real heap */
   malloc_set_heap_pointer( TempHeap );

diff -u rtems/testsuites/libtests/malloctest/init.c:1.45 rtems/testsuites/libtests/malloctest/init.c:1.46
--- rtems/testsuites/libtests/malloctest/init.c:1.45	Wed Mar  9 09:39:58 2011
+++ rtems/testsuites/libtests/malloctest/init.c	Thu May  5 11:45:47 2011
@@ -11,7 +11,7 @@
  *
  *  Output parameters:  NONE
  *
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  Copyright (c) 2009, 2010 embedded brains GmbH.
@@ -420,15 +420,11 @@
   uintptr_t previous_last_block_begin = 0;
   uintptr_t previous_last_page_begin = 0;
 
-  uintptr_t first_block_begin = 0;
-
   uintptr_t last_block_begin = 0;
   uintptr_t last_alloc_begin = 0;
 
   test_heap_init( TEST_DEFAULT_PAGE_SIZE );
 
-  first_block_begin = (uintptr_t) TestHeap.first_block;
-
   last_block_begin = (uintptr_t) TestHeap.last_block;
   last_alloc_begin = _Heap_Alloc_area_of_block( TestHeap.last_block );
 
@@ -467,6 +463,8 @@
   alloc_size = test_page_size();
   p1 = test_init_and_alloc_simple( alloc_size, alignment, boundary );
   p2 = test_alloc_simple( alloc_size, alignment, boundary );
+  rtems_test_assert( p2 );
+
   test_free( p1 );
 
   alloc_size = 2 * alloc_size;
@@ -890,7 +888,11 @@
   puts( "\t\tlet the next block be used alredy and try to get a size bigger than the actual block" );
   test_heap_init( TEST_DEFAULT_PAGE_SIZE );
   p1 = test_alloc_one_page();
+  rtems_test_assert( p1 );
+
   p2 = test_alloc_one_page();
+  rtems_test_assert( p2 );
+
   new_alloc_size = 3 * TEST_DEFAULT_PAGE_SIZE / 2;
   test_simple_resize_block( p1, new_alloc_size, HEAP_RESIZE_UNSATISFIED );
 
@@ -903,8 +905,14 @@
   puts( "\t\tlet the block after the next be used and try to allocate more then one pagesize more" );
   test_heap_init( TEST_DEFAULT_PAGE_SIZE );
   p1 = test_alloc_one_page();
+  rtems_test_assert( p1 );
+
   p2 = test_alloc_one_page();
+  rtems_test_assert( p2 );
+
   p3 = test_alloc_one_page();
+  rtems_test_assert( p3 );
+
   _Heap_Free( &TestHeap, p2 );
   new_alloc_size = 5 * TEST_DEFAULT_PAGE_SIZE / 2;
   test_simple_resize_block( p1, new_alloc_size, HEAP_RESIZE_UNSATISFIED );

diff -u rtems/testsuites/libtests/termios03/termios_testdriver_polled.c:1.3 rtems/testsuites/libtests/termios03/termios_testdriver_polled.c:1.4
--- rtems/testsuites/libtests/termios03/termios_testdriver_polled.c:1.3	Tue Feb 22 01:34:08 2011
+++ rtems/testsuites/libtests/termios03/termios_testdriver_polled.c	Thu May  5 11:45:47 2011
@@ -1,7 +1,7 @@
 /*
  *  This file contains a test fixture termios device driver
  *
- *  COPYRIGHT (c) 1989-2010.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -134,6 +134,7 @@
   }
 
   sc = rtems_termios_open (major, minor, arg, &Callbacks);
+  directive_failed( sc, "termios open" ); 
 
   return RTEMS_SUCCESSFUL;
 }

diff -u rtems/testsuites/libtests/termios04/termios_testdriver_intr.c:1.3 rtems/testsuites/libtests/termios04/termios_testdriver_intr.c:1.4
--- rtems/testsuites/libtests/termios04/termios_testdriver_intr.c:1.3	Tue Feb 22 01:34:08 2011
+++ rtems/testsuites/libtests/termios04/termios_testdriver_intr.c	Thu May  5 11:45:47 2011
@@ -1,7 +1,7 @@
 /*
  *  This file contains a test fixture termios device driver
  *
- *  COPYRIGHT (c) 1989-2010.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -205,6 +205,7 @@
   }
 
   sc = rtems_termios_open (major, minor, arg, &Callbacks);
+  directive_failed( sc, "termios open" ); 
 
   Ttyp = args->iop->data1;   /* Keep cookie returned by termios_open */
 


 *joel*:
2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>

	* psx04/task3.c, psxfile01/test.c, psxhdrs/Makefile.am,
	psxmsgq01/init.c, psxreaddir/test.c, psxsignal01/init.c,
	psxtimes01/init.c, psxualarm/init.c: Remove warnings.

M  1.338  testsuites/psxtests/ChangeLog
M   1.13  testsuites/psxtests/psx04/task3.c
M   1.32  testsuites/psxtests/psxfile01/test.c
M   1.18  testsuites/psxtests/psxhdrs/Makefile.am
M   1.26  testsuites/psxtests/psxmsgq01/init.c
M   1.21  testsuites/psxtests/psxreaddir/test.c
M   1.11  testsuites/psxtests/psxsignal01/init.c
M    1.4  testsuites/psxtests/psxtimes01/init.c
M    1.7  testsuites/psxtests/psxualarm/init.c

diff -u rtems/testsuites/psxtests/ChangeLog:1.337 rtems/testsuites/psxtests/ChangeLog:1.338
--- rtems/testsuites/psxtests/ChangeLog:1.337	Fri Apr 15 09:21:19 2011
+++ rtems/testsuites/psxtests/ChangeLog	Thu May  5 11:45:51 2011
@@ -1,3 +1,9 @@
+2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>
+
+	* psx04/task3.c, psxfile01/test.c, psxhdrs/Makefile.am,
+	psxmsgq01/init.c, psxreaddir/test.c, psxsignal01/init.c,
+	psxtimes01/init.c, psxualarm/init.c: Remove warnings.
+
 2011-04-15	Sebastian Huber <Sebastian.Huber at embedded-brains.de>
 
 	* psxstat/test.c: Fixed integer type.  Check status codes.

diff -u rtems/testsuites/psxtests/psx04/task3.c:1.12 rtems/testsuites/psxtests/psx04/task3.c:1.13
--- rtems/testsuites/psxtests/psx04/task3.c:1.12	Tue Feb 22 03:47:31 2011
+++ rtems/testsuites/psxtests/psx04/task3.c	Thu May  5 11:45:52 2011
@@ -7,7 +7,7 @@
  *
  *  Output parameters:  NONE
  *
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -112,6 +112,7 @@
   printf( "Task_3: sleep so the Init task can reguest a signal\n" );
   remaining = sleep( 1 );
   rtems_test_assert( !status );
+  rtems_test_assert( remaining == 0 );
 
      /* end of task 3 */
   printf( "Task_3: exit\n" );

diff -u rtems/testsuites/psxtests/psxfile01/test.c:1.31 rtems/testsuites/psxtests/psxfile01/test.c:1.32
--- rtems/testsuites/psxtests/psxfile01/test.c:1.31	Tue Feb 22 03:47:32 2011
+++ rtems/testsuites/psxtests/psxfile01/test.c	Thu May  5 11:45:52 2011
@@ -10,7 +10,7 @@
  *    /dev
  *    /dev/XXX   [where XXX includes at least console]
  *
- *  COPYRIGHT (c) 1989-2010.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -162,6 +162,7 @@
 
   build_time( &time, 12, 31, 1988, 9, 0, 0, 0 );
   rtems_status = rtems_clock_set( &time );
+  directive_failed( rtems_status, "clock set" );
 
   /*
    *  Dump an empty file system
@@ -656,6 +657,7 @@
   rtems_test_assert( status == -1);
 
   rtems_status = rtems_io_register_name( "/dev/console", 0, 0 );
+  directive_failed( rtems_status, "io register" );
 
   test_case_reopen_append();
 

diff -u rtems/testsuites/psxtests/psxhdrs/Makefile.am:1.17 rtems/testsuites/psxtests/psxhdrs/Makefile.am:1.18
--- rtems/testsuites/psxtests/psxhdrs/Makefile.am:1.17	Mon Oct  4 10:53:45 2010
+++ rtems/testsuites/psxtests/psxhdrs/Makefile.am	Thu May  5 11:45:52 2011
@@ -30,4 +30,5 @@
 include $(top_srcdir)/../automake/compile.am
 include $(top_srcdir)/../automake/leaf.am
 
+AM_CPPFLAGS += -Wno-unused-but-set-variable
 include $(top_srcdir)/../automake/local.am

diff -u rtems/testsuites/psxtests/psxmsgq01/init.c:1.25 rtems/testsuites/psxtests/psxmsgq01/init.c:1.26
--- rtems/testsuites/psxtests/psxmsgq01/init.c:1.25	Tue Feb 22 03:47:35 2011
+++ rtems/testsuites/psxtests/psxmsgq01/init.c	Thu May  5 11:45:52 2011
@@ -1205,6 +1205,7 @@
   timeout.tv_nsec = tv1.tv_usec * 1000;
 
   status = mq_timedsend( Test_q[que].mq, msg, len , 0, &timeout );
+  rtems_test_assert( status == 0 );
 
   gettimeofday( &tv2, &tz2 );
   tv3.tv_sec  = tv2.tv_sec - tv1.tv_sec;

diff -u rtems/testsuites/psxtests/psxreaddir/test.c:1.20 rtems/testsuites/psxtests/psxreaddir/test.c:1.21
--- rtems/testsuites/psxtests/psxreaddir/test.c:1.20	Tue Feb 22 03:47:35 2011
+++ rtems/testsuites/psxtests/psxreaddir/test.c	Thu May  5 11:45:52 2011
@@ -15,7 +15,7 @@
  *  implementation of this appears to seek to the ((off/DIRENT_SIZE) + 1)
  *  record where DIRENT_SIZE seems to be 12 bytes.
  *
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -79,8 +79,10 @@
 
   the_dir = opendir( path );
   rtems_test_assert( the_dir );
+
   printdir( the_dir );
   status = closedir( the_dir );
+  rtems_test_assert( status );
 }
 
 char *many_files[] = {
@@ -145,13 +147,13 @@
         "END"
 };
 
-int select1 ( struct dirent *entry )
+int select1 ( const struct dirent *entry )
 {
    printf("SCANDIR SELECT1 accepts  nodename: %s\n", entry->d_name );
    return 1;
 }
 
-int select2 ( struct dirent *entry )
+int select2 ( const struct dirent *entry )
 {
    if( strcmp( entry->d_name, "y") == 0 ) {
       printf("SCANDIR SELECT accepted nodename: %s\n", entry->d_name );
@@ -273,6 +275,7 @@
   directory_not = opendir( "/many" );
   printdir ( directory_not );
   d_not = readdir( directory_not );
+  rtems_test_assert( d_not == 0 );
 
   printf("open /b/myfile\n");
   fd = open ("/b/my_file", O_CREAT, S_IRWXU);

diff -u rtems/testsuites/psxtests/psxsignal01/init.c:1.10 rtems/testsuites/psxtests/psxsignal01/init.c:1.11
--- rtems/testsuites/psxtests/psxsignal01/init.c:1.10	Tue Feb 22 03:47:35 2011
+++ rtems/testsuites/psxtests/psxsignal01/init.c	Thu May  5 11:45:52 2011
@@ -77,7 +77,9 @@
 )
 {
   int  status;
+
   status = pthread_kill( pthread_self(), SIGUSR1 );
+  rtems_test_assert( status == 0 );
 }
 
 

diff -u rtems/testsuites/psxtests/psxtimes01/init.c:1.3 rtems/testsuites/psxtests/psxtimes01/init.c:1.4
--- rtems/testsuites/psxtests/psxtimes01/init.c:1.3	Tue Feb 22 03:47:36 2011
+++ rtems/testsuites/psxtests/psxtimes01/init.c	Thu May  5 11:45:52 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2010.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -57,6 +57,7 @@
   puts( "_times( &start_tm ) -- OK" );
   now = _times( &start_tm );
   rtems_test_assert( start != 0 );
+  rtems_test_assert( now != 0 );
   
   rtems_test_spin_for_ticks(5);
 

diff -u rtems/testsuites/psxtests/psxualarm/init.c:1.6 rtems/testsuites/psxtests/psxualarm/init.c:1.7
--- rtems/testsuites/psxtests/psxualarm/init.c:1.6	Tue Feb 22 03:47:36 2011
+++ rtems/testsuites/psxtests/psxualarm/init.c	Thu May  5 11:45:52 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -45,7 +45,9 @@
 )
 {
   int               status;
+
   status = kill( getpid(), SIGUSR1 );
+  rtems_test_assert( status == 0 );
 }
 
 
@@ -79,7 +81,10 @@
   puts( "Init: ualarm in 1 us" );
   sleep(3);
   result = ualarm(1,0);
+  rtems_test_assert( result == 0 );
+  
   status = sleep(10);
+  rtems_test_assert( status == 0 );
 
   /* unblock Signal and see if it happened */
   status = sigemptyset( &mask );


 *joel*:
2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>

	* sp09/screen12.c, sp09/sp09.scn, sp21/Makefile.am, sp35/priinv.c,
	sp39/init.c, sp50/init.c, sp57/init.c, sp72/init.c, sp73/init.c,
	spintrcritical01/init.c, spprivenv01/init.c, spsimplesched01/init.c,
	spsimplesched02/init.c: Remove warnings.

M  1.448  testsuites/sptests/ChangeLog
M   1.25  testsuites/sptests/sp09/screen12.c
M   1.28  testsuites/sptests/sp09/sp09.scn
M   1.26  testsuites/sptests/sp21/Makefile.am
M   1.18  testsuites/sptests/sp35/priinv.c
M   1.12  testsuites/sptests/sp39/init.c
M    1.6  testsuites/sptests/sp50/init.c
M    1.4  testsuites/sptests/sp57/init.c
M    1.3  testsuites/sptests/sp72/init.c
M    1.2  testsuites/sptests/sp73/init.c
M    1.7  testsuites/sptests/spintrcritical01/init.c
M    1.4  testsuites/sptests/spprivenv01/init.c
M    1.2  testsuites/sptests/spsimplesched01/init.c
M    1.2  testsuites/sptests/spsimplesched02/init.c

diff -u rtems/testsuites/sptests/ChangeLog:1.447 rtems/testsuites/sptests/ChangeLog:1.448
--- rtems/testsuites/sptests/ChangeLog:1.447	Fri Apr 15 08:21:09 2011
+++ rtems/testsuites/sptests/ChangeLog	Thu May  5 11:45:57 2011
@@ -1,3 +1,10 @@
+2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>
+
+	* sp09/screen12.c, sp09/sp09.scn, sp21/Makefile.am, sp35/priinv.c,
+	sp39/init.c, sp50/init.c, sp57/init.c, sp72/init.c, sp73/init.c,
+	spintrcritical01/init.c, spprivenv01/init.c, spsimplesched01/init.c,
+	spsimplesched02/init.c: Remove warnings.
+
 2011-04-15	Sebastian Huber <sebastian.huber at embedded-brains.de>
 
 	* sp09/screen02.c: Use integer value that fits into one character.

diff -u rtems/testsuites/sptests/sp09/screen12.c:1.24 rtems/testsuites/sptests/sp09/screen12.c:1.25
--- rtems/testsuites/sptests/sp09/screen12.c:1.24	Tue Feb 22 01:36:58 2011
+++ rtems/testsuites/sptests/sp09/screen12.c	Thu May  5 11:45:57 2011
@@ -6,7 +6,7 @@
  *
  *  Output parameters:  NONE
  *
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -27,9 +27,6 @@
   void                   *segment_address_1;
   void                   *segment_address_2;
   void                   *segment_address_3;
-  uint32_t                good_back_flag;
-  uint32_t                good_front_flag;
-  uint32_t                offset;
   uintptr_t               segment_size;
   rtems_status_code       status;
   Heap_Information_block  the_info;
@@ -400,48 +397,6 @@
   puts( "TA1 - rtems_debug_disable - RTEMS_DEBUG_REGION" );
   rtems_debug_disable( RTEMS_DEBUG_REGION );
 
-#if 0
-
-  offset = (segment_address_1 - (void *)Region_good_area) / 4;
-
-/* bad FRONT_FLAG error */
-
-  good_front_flag = Region_good_area[ offset - 1 ];
-  Region_good_area[ offset - 1 ] = good_front_flag + 2;
-
-  status = rtems_region_return_segment( Region_id[ 1 ], segment_address_1 );
-  fatal_directive_status(
-    status,
-    RTEMS_INVALID_ADDRESS,
-    "rtems_region_return_segment with back_flag != front_flag"
-  );
-  puts( "TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS" );
-
-  Region_good_area[ offset - 1 ] = good_front_flag;
-
-/* bad FRONT_FLAG error */
-
-  good_back_flag = Region_good_area[ offset - 2 ];
-  Region_good_area[ offset - 2 ] = 1024;
-
-  status = rtems_region_return_segment( Region_id[ 1 ], segment_address_1 );
-  fatal_directive_status(
-    status,
-    RTEMS_INVALID_ADDRESS,
-    "rtems_region_return_segment with back_flag != front_flag"
-  );
-  puts( "TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS" );
-
-  Region_good_area[ offset - 2 ] = good_back_flag;
-
-#else
-  offset = 0;
-  good_front_flag = 0;
-  good_back_flag = 0;
-  puts( "TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS - SKIPPED" );
-  puts( "TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS - SKIPPED" );
-#endif
-
   puts( "TA1 - rtems_debug_enable - RTEMS_DEBUG_REGION" );
   rtems_debug_enable( RTEMS_DEBUG_REGION );
 

diff -u rtems/testsuites/sptests/sp09/sp09.scn:1.27 rtems/testsuites/sptests/sp09/sp09.scn:1.28
--- rtems/testsuites/sptests/sp09/sp09.scn:1.27	Fri Mar 11 14:26:59 2011
+++ rtems/testsuites/sptests/sp09/sp09.scn	Thu May  5 11:45:57 2011
@@ -273,8 +273,6 @@
 TA1 - rtems_region_return_segment - RTEMS_INVALID_ID
 TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS
 TA1 - rtems_debug_disable - RTEMS_DEBUG_REGION
-TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS - SKIPPED
-TA1 - rtems_region_return_segment - RTEMS_INVALID_ADDRESS - SKIPPED
 TA1 - rtems_debug_enable - RTEMS_DEBUG_REGION
 TA1 - rtems_region_extend - RTEMS_INVALID_ID
 TA1 - rtems_region_extend - within heap - RTEMS_INVALID_ADDRESS

diff -u rtems/testsuites/sptests/sp21/Makefile.am:1.25 rtems/testsuites/sptests/sp21/Makefile.am:1.26
--- rtems/testsuites/sptests/sp21/Makefile.am:1.25	Tue Jul  6 17:27:23 2010
+++ rtems/testsuites/sptests/sp21/Makefile.am	Thu May  5 11:45:57 2011
@@ -13,6 +13,7 @@
 include $(top_srcdir)/../automake/leaf.am
 
 AM_CPPFLAGS += -I$(top_srcdir)/../support/include
+AM_CPPFLAGS += -Wno-deprecated-declarations
 
 LINK_OBJS = $(sp21_OBJECTS) $(sp21_LDADD)
 LINK_LIBS = $(sp21_LDLIBS)

diff -u rtems/testsuites/sptests/sp35/priinv.c:1.17 rtems/testsuites/sptests/sp35/priinv.c:1.18
--- rtems/testsuites/sptests/sp35/priinv.c:1.17	Tue Feb 22 01:37:00 2011
+++ rtems/testsuites/sptests/sp35/priinv.c	Thu May  5 11:45:57 2011
@@ -243,17 +243,20 @@
 {
   rtems_status_code     Sts;
 
-  rtems_task_priority   EnterPrio;   /* Statistics log */
+#if defined(TEST_PRINT_STATISTICS)
   rtems_task_priority   AccessPrio;  /*      :         */
-  rtems_task_priority   LeavePrio;   /*      :         */
-  uint32_t              EnterCnt;    /*      :         */
   uint32_t              AccessCnt;   /*      :         */
+  rtems_task_priority   EnterPrio;   /* Statistics log */
+  uint32_t              EnterCnt;    /*      :         */
+  rtems_task_priority   LeavePrio;   /*      :         */
   uint32_t              LeaveCnt;    /*      :         */
+#endif
 
+#if defined(TEST_PRINT_STATISTICS)
   /* Store information about the current situation */
   EnterPrio = _Thread_Executing->current_priority;
   EnterCnt  = _Thread_Executing->resource_count;
-
+#endif
 
   printf("  AccessLocalHw called by %s\n", CallerName());
 
@@ -269,18 +272,20 @@
   Sts = rtems_semaphore_obtain(LocalHwSync_S, RTEMS_WAIT, RTEMS_NO_TIMEOUT);
   directive_failed( Sts, "rtems_semaphore_obtain(LocalHwAccess_R...)" );
 
+#if defined(TEST_PRINT_STATISTICS)
   /* Store information about the current situation */
   AccessPrio = _Thread_Executing->current_priority;
   AccessCnt  = _Thread_Executing->resource_count;
+#endif
 
   Sts = rtems_semaphore_release(LocalHwAccess_R);
   directive_failed( Sts, "rtems_semaphore_release(LocalHwAccess_R)" );
 
+#if defined(TEST_PRINT_STATISTICS)
   /* Store information about the current situation */
   LeavePrio = _Thread_Executing->current_priority;
   LeaveCnt  = _Thread_Executing->resource_count;
 
-#if defined(TEST_PRINT_STATISTICS)
   printf(
     "  AccessLocalHw from %s statistics:\n"
     " - Prio: %d -> %d -> %d\n - Cnt: %d -> %d -> %d\n",
@@ -304,17 +309,20 @@
 {
   rtems_status_code     Sts;
 
+#if defined(TEST_PRINT_STATISTICS)
   rtems_task_priority   EnterPrio;   /* Statistics log */
   rtems_task_priority   AccessPrio;  /*      :         */
   rtems_task_priority   LeavePrio;   /*      :         */
   uint32_t              EnterCnt;    /*      :         */
   uint32_t              AccessCnt;   /*      :         */
   uint32_t              LeaveCnt;    /*      :         */
+#endif
 
+#if defined(TEST_PRINT_STATISTICS)
   /* Store information about the current situation */
   EnterPrio = _Thread_Executing->current_priority;
   EnterCnt  = _Thread_Executing->resource_count;
-
+#endif
 
   printf("AccessRemoteHw called by %s\n", CallerName());
 
@@ -329,18 +337,20 @@
   printf("AccessRemoteHw access local %s\n", CallerName());
   AccessLocalHw();
 
+#if defined(TEST_PRINT_STATISTICS)
   /* Store information about the current situation */
   AccessPrio = _Thread_Executing->current_priority;
   AccessCnt  = _Thread_Executing->resource_count;
+#endif
 
   Sts = rtems_semaphore_release(RemoteHwAccess_R);
   directive_failed( Sts, "rtems_semaphore_release(RemoreHwAccess_R" );
 
+#if defined(TEST_PRINT_STATISTICS)
   /* Store information about the current situation */
   LeavePrio = _Thread_Executing->current_priority;
   LeaveCnt  = _Thread_Executing->resource_count;
 
-#if defined(TEST_PRINT_STATISTICS)
   printf(
     "\nAccessRemoteHw from %s statistics:\n"
     " - Prio: %d -> %d -> %d\n - Cnt: %d -> %d -> %d\n",

diff -u rtems/testsuites/sptests/sp39/init.c:1.11 rtems/testsuites/sptests/sp39/init.c:1.12
--- rtems/testsuites/sptests/sp39/init.c:1.11	Tue Feb 22 01:37:00 2011
+++ rtems/testsuites/sptests/sp39/init.c	Thu May  5 11:45:57 2011
@@ -1,7 +1,7 @@
 /*
  *  Classic API Signal to Task from ISR
  *
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -44,17 +44,20 @@
      *  This event send hits the critical section but sends to
      *  another task so doesn't impact this critical section.
      */
-    rtems_event_send( other_task, 0x02 );
+    status = rtems_event_send( other_task, 0x02 );
+    directive_failed( status, "event send" );
 
     /*
      *  This event send hits the main task but doesn't satisfy
      *  it's blocking condition so it will still block
      */
-    rtems_event_send( main_task, 0x02 );
+    status = rtems_event_send( main_task, 0x02 );
+    directive_failed( status, "event send" );
 
     case_hit = TRUE;
   }
   status = rtems_event_send( main_task, 0x01 );
+  directive_failed( status, "event send" );
 }
 
 rtems_timer_service_routine test_event_with_timeout_from_isr(
@@ -72,6 +75,7 @@
     case_hit = TRUE;
   }
   status = rtems_event_send( main_task, 0x01 );
+  directive_failed( status, "event send" );
 }
 
 rtems_task Init(

diff -u rtems/testsuites/sptests/sp50/init.c:1.5 rtems/testsuites/sptests/sp50/init.c:1.6
--- rtems/testsuites/sptests/sp50/init.c:1.5	Tue Feb 22 01:37:01 2011
+++ rtems/testsuites/sptests/sp50/init.c	Thu May  5 11:45:57 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -20,17 +20,19 @@
 
 rtems_timer_service_routine Timer_Routine( rtems_id id, void *ignored )
 {
-  rtems_status_code sc;
+  rtems_status_code status;
 
   Fired++;
   timerRan = true;
 
-  sc = rtems_timer_server_fire_after(
+  status = rtems_timer_server_fire_after(
     id,
     rtems_clock_get_ticks_per_second(),
     Timer_Routine,
     NULL
   );
+  directive_failed( status, "fire after" );
+
 }
 
 rtems_task Init(

diff -u rtems/testsuites/sptests/sp57/init.c:1.3 rtems/testsuites/sptests/sp57/init.c:1.4
--- rtems/testsuites/sptests/sp57/init.c:1.3	Tue Feb 22 01:37:01 2011
+++ rtems/testsuites/sptests/sp57/init.c	Thu May  5 11:45:57 2011
@@ -1,6 +1,6 @@
 /*  Restart a task which is delaying
  *
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -24,6 +24,7 @@
 
   puts( "Delay - rtems_task_wake_after - OK" );
   status = rtems_task_wake_after( RTEMS_MILLISECONDS_TO_TICKS(2000) );
+  directive_failed( status, "wake after" );
 
   puts( "ERROR - delay task woke up!!" );
   rtems_test_exit(0);

diff -u rtems/testsuites/sptests/sp72/init.c:1.2 rtems/testsuites/sptests/sp72/init.c:1.3
--- rtems/testsuites/sptests/sp72/init.c:1.2	Tue Feb 22 01:37:01 2011
+++ rtems/testsuites/sptests/sp72/init.c	Thu May  5 11:45:57 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2010.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -30,6 +30,8 @@
     RTEMS_DEFAULT_OPTIONS,
     rtems_clock_get_ticks_per_second() * 10
   );
+  directive_failed( status, "obtain" );
+
 }
 
 rtems_task Init(

diff -u rtems/testsuites/sptests/sp73/init.c:1.1 rtems/testsuites/sptests/sp73/init.c:1.2
--- rtems/testsuites/sptests/sp73/init.c:1.1	Fri Mar 11 10:39:27 2011
+++ rtems/testsuites/sptests/sp73/init.c	Thu May  5 11:45:57 2011
@@ -31,6 +31,8 @@
   task_index = arg;
   for ( ; ; ) {
     status = rtems_clock_get_tod( &time );
+    directive_failed( status, "get tod" );
+
     if ( time.second >= 15 ) {
       puts( "*** END OF SP73 (YIELD) TEST ***" );
       rtems_test_exit( 0 );
@@ -38,11 +40,12 @@
     put_name( Task_name[ task_index ], FALSE );
     print_time( " - rtems_clock_get_tod - ", &time, "\n" );
     status = rtems_task_wake_after(
-      task_index * 5 * rtems_clock_get_ticks_per_second() );
+      task_index * 5 * rtems_clock_get_ticks_per_second()
+    );
+    directive_failed( status, "wake after" );
   }
 }
 
-
 rtems_task Init(
   rtems_task_argument argument
 )
@@ -71,25 +74,39 @@
     Task_name[ 1 ], 1, RTEMS_MINIMUM_STACK_SIZE, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 1 ]
   );
+  directive_failed( status, "create 1" );
+
   status = rtems_task_create(
     Task_name[ 2 ], 1, RTEMS_MINIMUM_STACK_SIZE, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 2 ]
   );
+  directive_failed( status, "create 2" );
+
   status = rtems_task_create(
     Task_name[ 3 ], 1, RTEMS_MINIMUM_STACK_SIZE, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 3 ]
   );
+  directive_failed( status, "create 3" );
 
   status = rtems_task_start( Task_id[ 1 ], Test_task, 1 );
+  directive_failed( status, "start 1" );
   rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+
   status = rtems_task_start( Task_id[ 2 ], Test_task, 2 );
+  directive_failed( status, "start 2" );
   rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+
   status = rtems_task_start( Task_id[ 3 ], Test_task, 3 );
+  directive_failed( status, "start 3" );
 
-  rtems_task_set_priority(Task_id[1], 1, &old);
-  rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+  status = rtems_task_set_priority(Task_id[1], 1, &old);
+  directive_failed( status, "set priority" );
+
+  status = rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+  directive_failed( status, "wake after" );
 
   status = rtems_task_delete( RTEMS_SELF );
+  directive_failed( status, "delete" );
 }
 
 /* configuration information */

diff -u rtems/testsuites/sptests/spintrcritical01/init.c:1.6 rtems/testsuites/sptests/spintrcritical01/init.c:1.7
--- rtems/testsuites/sptests/spintrcritical01/init.c:1.6	Tue Feb 22 01:37:02 2011
+++ rtems/testsuites/sptests/spintrcritical01/init.c	Thu May  5 11:45:57 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -82,7 +82,9 @@
   if ( getState() == THREAD_BLOCKING_OPERATION_NOTHING_HAPPENED ) {
     case_hit = true;
   }
+
   status = rtems_semaphore_release( Semaphore );
+  directive_failed( status, "release" );
 }
 
 

diff -u rtems/testsuites/sptests/spprivenv01/init.c:1.3 rtems/testsuites/sptests/spprivenv01/init.c:1.4
--- rtems/testsuites/sptests/spprivenv01/init.c:1.3	Tue Feb 22 01:37:03 2011
+++ rtems/testsuites/sptests/spprivenv01/init.c	Thu May  5 11:45:57 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2010.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -20,11 +20,12 @@
 
 rtems_task task_routine( rtems_task_argument not_used )
 {
-  int sc = 0;
+  rtems_status_code sc;
 
   puts( "task_routine - setting up a private environment" );
 
   sc = rtems_libio_set_private_env();
+  directive_failed( sc, "set private env" );
   sleep( 1 );
 
   rtems_task_delete( RTEMS_SELF );
@@ -34,12 +35,12 @@
   rtems_task_argument argument
 )
 {
-  int sc = 0;
-  bool status = 0;
-  void *alloc_ptr = (void *)0;
-  rtems_id current_task_id;
-  rtems_id task_id;
-  rtems_name another_task_name;
+  rtems_status_code       sc;
+  bool                    status;
+  void                   *alloc_ptr;
+  rtems_id                current_task_id;
+  rtems_id                task_id;
+  rtems_name              another_task_name;
   Heap_Information_block  Info;
   
   puts( "\n\n*** TEST USER ENVIRONMENT ROUTINE - 01 ***" );

diff -u rtems/testsuites/sptests/spsimplesched01/init.c:1.1 rtems/testsuites/sptests/spsimplesched01/init.c:1.2
--- rtems/testsuites/sptests/spsimplesched01/init.c:1.1	Wed Mar 16 11:33:04 2011
+++ rtems/testsuites/sptests/spsimplesched01/init.c	Thu May  5 11:45:57 2011
@@ -25,15 +25,18 @@
   rtems_task_argument unused
 )
 {
-  rtems_id          tid;
-  rtems_time_of_day time;
-  uint32_t    task_index;
-  rtems_status_code status;
+  rtems_id           tid;
+  rtems_time_of_day  time;
+  uint32_t           task_index;
+  rtems_status_code  status;
 
   status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &tid );
+  directive_failed( status, "task ident" );
+
   task_index = task_number( tid );
   for ( ; ; ) {
     status = rtems_clock_get_tod( &time );
+    directive_failed( status, "clock get tod" );
     if ( time.second >= 35 ) {
       puts( "*** END OF SIMPLE01 TEST ***" );
       rtems_test_exit( 0 );
@@ -41,7 +44,9 @@
     put_name( Task_name[ task_index ], FALSE );
     print_time( " - rtems_clock_get_tod - ", &time, "\n" );
     status = rtems_task_wake_after(
-      task_index * 5 * rtems_clock_get_ticks_per_second() );
+      task_index * 5 * rtems_clock_get_ticks_per_second()
+    );
+    directive_failed( status, "wake after" );
   }
 }
 
@@ -73,28 +78,43 @@
     Task_name[ 1 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 1 ]
   );
+  directive_failed( status, "create 1" );
+
   status = rtems_task_create(
     Task_name[ 2 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 2 ]
   );
+  directive_failed( status, "create 2" );
+
   status = rtems_task_create(
     Task_name[ 3 ], 1, RTEMS_MINIMUM_STACK_SIZE * 2, RTEMS_DEFAULT_MODES,
     RTEMS_DEFAULT_ATTRIBUTES, &Task_id[ 3 ]
   );
+  directive_failed( status, "create 3" );
 
   status = rtems_task_start( Task_id[ 1 ], Test_task, 1 );
+  directive_failed( status, "start 1" );
   rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+
   status = rtems_task_start( Task_id[ 2 ], Test_task, 2 );
+  directive_failed( status, "start 2" );
   rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+
   status = rtems_task_start( Task_id[ 3 ], Test_task, 3 );
+  directive_failed( status, "start 3" );
 
-  rtems_task_set_priority(Task_id[1], 2, &old);
-  rtems_task_set_priority(Task_id[2], 2, &old);
-  rtems_task_set_priority(Task_id[3], 2, &old);
+  status = rtems_task_set_priority( Task_id[1], 2, &old );
+  directive_failed( status, "set priority 1" );
+  status = rtems_task_set_priority( Task_id[2], 2, &old );
+  directive_failed( status, "set priority 2" );
+  status = rtems_task_set_priority( Task_id[3], 2, &old );
+  directive_failed( status, "set priority 3" );
 
-  rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+  status = rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+  directive_failed( status, "yield" );
 
   status = rtems_task_delete( RTEMS_SELF );
+  directive_failed( status, "delete self" );
 }
 
 /* configuration information */

diff -u rtems/testsuites/sptests/spsimplesched02/init.c:1.1 rtems/testsuites/sptests/spsimplesched02/init.c:1.2
--- rtems/testsuites/sptests/spsimplesched02/init.c:1.1	Wed Mar 16 11:33:04 2011
+++ rtems/testsuites/sptests/spsimplesched02/init.c	Thu May  5 11:45:57 2011
@@ -28,14 +28,14 @@
 )
 {
   rtems_id          tid;
-  uint32_t    task_index;
   rtems_status_code status;
 
   status = rtems_task_ident( RTEMS_SELF, RTEMS_SEARCH_ALL_NODES, &tid );
-  task_index = task_number( tid );
+  directive_failed( status, "wake after" );
 
   for ( ; ; ) {
-    rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+    status = rtems_task_wake_after( RTEMS_YIELD_PROCESSOR );
+    directive_failed( status, "yield" );
   }
 }
 


 *joel*:
2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>

	* tm01/task1.c, tm02/task1.c, tm03/task1.c, tm15/task1.c: Remove
	warnings.

M  1.100  testsuites/tmtests/ChangeLog
M   1.20  testsuites/tmtests/tm01/task1.c
M   1.20  testsuites/tmtests/tm02/task1.c
M   1.20  testsuites/tmtests/tm03/task1.c
M   1.19  testsuites/tmtests/tm15/task1.c

diff -u rtems/testsuites/tmtests/ChangeLog:1.99 rtems/testsuites/tmtests/ChangeLog:1.100
--- rtems/testsuites/tmtests/ChangeLog:1.99	Wed Mar 16 12:11:32 2011
+++ rtems/testsuites/tmtests/ChangeLog	Thu May  5 11:46:00 2011
@@ -1,3 +1,8 @@
+2011-05-05	Joel Sherrill <joel.sherrill at oarcorp.com>
+
+	* tm01/task1.c, tm02/task1.c, tm03/task1.c, tm15/task1.c: Remove
+	warnings.
+
 2011-03-16	Jennifer Averett <jennifer.averett at OARcorp.com>
 
 	PR 1743/cpu

diff -u rtems/testsuites/tmtests/tm01/task1.c:1.19 rtems/testsuites/tmtests/tm01/task1.c:1.20
--- rtems/testsuites/tmtests/tm01/task1.c:1.19	Tue Feb 22 03:57:13 2011
+++ rtems/testsuites/tmtests/tm01/task1.c	Thu May  5 11:46:00 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -115,6 +115,7 @@
     RTEMS_NO_PRIORITY,
     &smid
   );
+  directive_failed( status, "rtems_task_create of TA1" );
 
   for ( iterations=OPERATION_COUNT ; iterations ; iterations-- ) {
 

diff -u rtems/testsuites/tmtests/tm02/task1.c:1.19 rtems/testsuites/tmtests/tm02/task1.c:1.20
--- rtems/testsuites/tmtests/tm02/task1.c:1.19	Tue Feb 22 03:57:13 2011
+++ rtems/testsuites/tmtests/tm02/task1.c	Thu May  5 11:46:00 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -121,10 +121,10 @@
   rtems_task_argument argument
 )
 {
-  rtems_status_code status;
+  /* start blocking rtems_semaphore_obtain time */
+  benchmark_timer_initialize();
 
-  benchmark_timer_initialize();           /* start blocking rtems_semaphore_obtain time */
-  status = rtems_semaphore_obtain(
+  (void) rtems_semaphore_obtain(
     Semaphore_id,
     RTEMS_DEFAULT_OPTIONS,
     RTEMS_NO_TIMEOUT
@@ -135,9 +135,7 @@
   rtems_task_argument argument
 )
 {
-  rtems_status_code status;
-
-  status = rtems_semaphore_obtain(
+  (void) rtems_semaphore_obtain(
     Semaphore_id,
     RTEMS_DEFAULT_OPTIONS,
     RTEMS_NO_TIMEOUT

diff -u rtems/testsuites/tmtests/tm03/task1.c:1.19 rtems/testsuites/tmtests/tm03/task1.c:1.20
--- rtems/testsuites/tmtests/tm03/task1.c:1.19	Tue Feb 22 03:57:13 2011
+++ rtems/testsuites/tmtests/tm03/task1.c	Thu May  5 11:46:00 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -118,24 +118,20 @@
   rtems_task_argument argument
 )
 {
-  rtems_status_code status;
-
-  status = rtems_semaphore_obtain(
+  (void) rtems_semaphore_obtain(
     Semaphore_id,
     RTEMS_DEFAULT_OPTIONS,
     RTEMS_NO_TIMEOUT
   );
 
-  status = rtems_semaphore_release( Semaphore_id );
+  (void) rtems_semaphore_release( Semaphore_id );
 }
 
 rtems_task High_task(
   rtems_task_argument argument
 )
 {
-  rtems_status_code status;
-
-  status = rtems_semaphore_obtain(
+  (void) rtems_semaphore_obtain(
     Semaphore_id,
     RTEMS_DEFAULT_OPTIONS,
     RTEMS_NO_TIMEOUT

diff -u rtems/testsuites/tmtests/tm15/task1.c:1.18 rtems/testsuites/tmtests/tm15/task1.c:1.19
--- rtems/testsuites/tmtests/tm15/task1.c:1.18	Tue Feb 22 03:57:13 2011
+++ rtems/testsuites/tmtests/tm15/task1.c	Thu May  5 11:46:00 2011
@@ -1,5 +1,5 @@
 /*
- *  COPYRIGHT (c) 1989-2009.
+ *  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -203,10 +203,8 @@
   rtems_task_argument argument
 )
 {
-  rtems_status_code status;
-
   if ( time_set )
-    status = rtems_event_receive(
+    (void) rtems_event_receive(
       RTEMS_EVENT_16,
       RTEMS_DEFAULT_OPTIONS,
       RTEMS_NO_TIMEOUT,
@@ -214,8 +212,9 @@
     );
   else {
     time_set = true;
-    benchmark_timer_initialize();            /* start blocking rtems_event_receive time */
-    status = rtems_event_receive(
+    /* start blocking rtems_event_receive time */
+    benchmark_timer_initialize();
+    (void) rtems_event_receive(
       RTEMS_EVENT_16,
       RTEMS_DEFAULT_OPTIONS,
       RTEMS_NO_TIMEOUT,



--

Generated by Deluxe Loginfo [http://www.codewiz.org/projects/index.html#loginfo] 2.122 by Bernardo Innocenti <bernie at develer.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/vc/attachments/20110505/b2e4791e/attachment-0001.html>


More information about the vc mailing list