change log for rtems (2011-11-26)

[rtems-vc at rtems.org]

 *gedare*:
2011-11-26	Gedare Bloom <gedare at rtems.org>

	PR 1964
	* spchain/init.c, spchain/spchain.scn: Add testcases for chain is first
	and last

M  1.499  testsuites/sptests/ChangeLog
M   1.12  testsuites/sptests/spchain/init.c
M    1.5  testsuites/sptests/spchain/spchain.scn

diff -u rtems/testsuites/sptests/ChangeLog:1.498 rtems/testsuites/sptests/ChangeLog:1.499
--- rtems/testsuites/sptests/ChangeLog:1.498	Thu Nov 10 08:43:33 2011
+++ rtems/testsuites/sptests/ChangeLog	Sat Nov 26 12:07:33 2011
@@ -1,3 +1,9 @@
+2011-11-26	Gedare Bloom <gedare at rtems.org>
+
+	PR 1964
+	* spchain/init.c, spchain/spchain.scn: Add testcases for chain is first
+	and last
+
 2011-11-10	Sebastian Huber <sebastian.huber at embedded-brains.de>
 
 	PR 1924/cpukit

diff -u rtems/testsuites/sptests/spchain/init.c:1.11 rtems/testsuites/sptests/spchain/init.c:1.12
--- rtems/testsuites/sptests/spchain/init.c:1.11	Sun Jul 24 17:10:20 2011
+++ rtems/testsuites/sptests/spchain/init.c	Sat Nov 26 12:07:43 2011
@@ -59,6 +59,25 @@
   rtems_test_assert( p == NULL );
 }
 
+static void test_chain_first_and_last(void)
+{
+  rtems_chain_control   chain;
+  rtems_chain_node      node1, node2;
+  rtems_chain_node     *cnode;
+
+  rtems_chain_initialize_empty( &chain );
+  rtems_chain_append( &chain, &node1 );
+  rtems_chain_insert( &node1, &node2 );
+
+  puts( "INIT - Verify rtems_chain_is_first" );
+  cnode = rtems_chain_first(&chain);  
+  rtems_test_assert( rtems_chain_is_first( cnode ) );
+
+  puts( "INIT - Verify rtems_chain_is_last" );
+  cnode = rtems_chain_last(&chain);
+  rtems_test_assert( rtems_chain_is_last( cnode ) );
+}
+
 static void test_chain_with_notification(void)
 {
   rtems_status_code sc = RTEMS_SUCCESSFUL;
@@ -214,6 +233,7 @@
      }
   }
 
+  test_chain_first_and_last();
   test_chain_with_empty_check();
   test_chain_with_notification();
   test_chain_get_with_wait();

diff -u rtems/testsuites/sptests/spchain/spchain.scn:1.4 rtems/testsuites/sptests/spchain/spchain.scn:1.5
--- rtems/testsuites/sptests/spchain/spchain.scn:1.4	Thu Nov 25 06:38:57 2010
+++ rtems/testsuites/sptests/spchain/spchain.scn	Sat Nov 26 12:07:52 2011
@@ -1,6 +1,8 @@
 *** TEST OF RTEMS CHAIN API ***
 Init - Initialize chain empty
 INIT - Verify rtems_chain_insert
+INIT - Verify rtems_chain_is_first
+INIT - Verify rtems_chain_is_last
 INIT - Verify rtems_chain_append_with_empty_check
 INIT - Verify rtems_chain_prepend_with_empty_check
 INIT - Verify rtems_chain_get_with_empty_check


 *gedare*:
2011-11-26	Gedare Bloom <gedare at rtems.org>

	PR 1964
	* score/inline/rtems/score/chain.inl: Fix chain is first and last.

M 1.2993  cpukit/ChangeLog
M   1.24  cpukit/score/inline/rtems/score/chain.inl

diff -u rtems/cpukit/ChangeLog:1.2992 rtems/cpukit/ChangeLog:1.2993
--- rtems/cpukit/ChangeLog:1.2992	Fri Nov 18 14:04:39 2011
+++ rtems/cpukit/ChangeLog	Sat Nov 26 12:10:38 2011
@@ -1,3 +1,8 @@
+2011-11-26	Gedare Bloom <gedare at rtems.org>
+
+	PR 1964
+	* score/inline/rtems/score/chain.inl: Fix chain is first and last.
+
 2011-11-18	Jennifer Averett <Jennifer.Averett at OARcorp.com>
 
 	* sapi/include/confdefs.h: Added a bsp override option for

diff -u rtems/cpukit/score/inline/rtems/score/chain.inl:1.23 rtems/cpukit/score/inline/rtems/score/chain.inl:1.24
--- rtems/cpukit/score/inline/rtems/score/chain.inl:1.23	Thu Nov 25 05:48:11 2010
+++ rtems/cpukit/score/inline/rtems/score/chain.inl	Sat Nov 26 12:10:39 2011
@@ -297,7 +297,7 @@
   const Chain_Node *the_node
 )
 {
-  return (the_node->previous == NULL);
+  return (the_node->previous->previous == NULL);
 }
 
 /** @brief Is this the Last Node on the Chain
@@ -314,7 +314,7 @@
   const Chain_Node *the_node
 )
 {
-  return (the_node->next == NULL);
+  return (the_node->next->next == NULL);
 }
 
 /** @brief Does this Chain have only One Node



--

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/20111126/0e2dfc08/attachment.html>