Changeset 6805ac37 in rtems


Ignore:
Timestamp:
Oct 10, 2009, 3:21:41 PM (10 years ago)
Author:
Joel Sherrill <joel.sherrill@…>
Branches:
4.10, 4.11, master
Children:
171bbec
Parents:
681d4526
Message:

2009-10-10 Joel Sherrill <joel.sherrill@…>

  • posix/src/mqueuetimedreceive.c, posix/src/mqueuetimedsend.c, posix/src/mutextimedlock.c, posix/src/prwlocktimedrdlock.c, posix/src/prwlocktimedwrlock.c, posix/src/semtimedwait.c: Switch from switch to if's because only one value needed to be tested. This shrinks the code and makes it easier to do coverage analysis on.
Location:
cpukit
Files:
7 edited

Legend:

Unmodified
Added
Removed
  • cpukit/ChangeLog

    r681d4526 r6805ac37  
     12009-10-10      Joel Sherrill <joel.sherrill@oarcorp.com>
     2
     3        * posix/src/mqueuetimedreceive.c, posix/src/mqueuetimedsend.c,
     4        posix/src/mutextimedlock.c, posix/src/prwlocktimedrdlock.c,
     5        posix/src/prwlocktimedwrlock.c, posix/src/semtimedwait.c: Switch from
     6        switch to if's because only one value needed to be tested. This
     7        shrinks the code and makes it easier to do coverage analysis on.
     8
    192009-10-10      Joel Sherrill <joel.sherrill@oarcorp.com>
    210
  • cpukit/posix/src/mqueuetimedreceive.c

    r681d4526 r6805ac37  
    5757)
    5858{
    59   Watchdog_Interval ticks;
    60   bool              do_wait;
     59  Watchdog_Interval                            ticks;
     60  bool                                         do_wait = true;
     61  POSIX_Absolute_timeout_conversion_results_t  status;
    6162
    6263  /*
     
    6869   *  then we do a polling operation and convert the UNSATISFIED
    6970   *  status into the appropriate error.
     71   *
     72   *  If the status is POSIX_ABSOLUTE_TIMEOUT_INVALID,
     73   *  POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST, or POSIX_ABSOLUTE_TIMEOUT_IS_NOW,
     74   *  then we should not wait.
    7075   */
    71   switch ( _POSIX_Absolute_timeout_to_ticks( abstime, &ticks ) ) {
    72     case POSIX_ABSOLUTE_TIMEOUT_INVALID:
    73     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST:
    74     case POSIX_ABSOLUTE_TIMEOUT_IS_NOW:
    75       do_wait = false;
    76       break;
    77     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE:
    78     default:  /* only to silence warnings */
    79       do_wait = true;
    80       break;
    81   }
     76  status = _POSIX_Absolute_timeout_to_ticks( abstime, &ticks );
     77  if ( status != POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE )
     78    do_wait = false;
    8279
    8380  return _POSIX_Message_queue_Receive_support(
  • cpukit/posix/src/mqueuetimedsend.c

    r681d4526 r6805ac37  
    5757)
    5858{
    59   Watchdog_Interval ticks;
    60   bool              do_wait;
     59  Watchdog_Interval                            ticks;
     60  bool                                         do_wait = true;
     61  POSIX_Absolute_timeout_conversion_results_t  status;
    6162
    6263  /*
     
    6869   *  then we do a polling operation and convert the UNSATISFIED
    6970   *  status into the appropriate error.
     71   *
     72   *  If the status is POSIX_ABSOLUTE_TIMEOUT_INVALID,
     73   *  POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST, or POSIX_ABSOLUTE_TIMEOUT_IS_NOW,
     74   *  then we should not wait.
    7075   */
    71   switch ( _POSIX_Absolute_timeout_to_ticks( abstime, &ticks ) ) {
    72     case POSIX_ABSOLUTE_TIMEOUT_INVALID:
    73     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST:
    74     case POSIX_ABSOLUTE_TIMEOUT_IS_NOW:
    75       do_wait = false;
    76       break;
    77     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE:
    78     default:  /* only to silence warnings */
    79       do_wait = true;
    80       break;
    81   }
     76  status = _POSIX_Absolute_timeout_to_ticks( abstime, &ticks );
     77  if ( status != POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE )
     78    do_wait = false;
    8279
    8380  return _POSIX_Message_queue_Send_support(
  • cpukit/posix/src/mutextimedlock.c

    r681d4526 r6805ac37  
    4444{
    4545  Watchdog_Interval                            ticks;
    46   bool                                         do_wait;
     46  bool                                         do_wait = true;
    4747  POSIX_Absolute_timeout_conversion_results_t  status;
    4848  int                                          lock_status;
     
    5656   *  then we do a polling operation and convert the UNSATISFIED
    5757   *  status into the appropriate error.
     58   *
     59   *  If the status is POSIX_ABSOLUTE_TIMEOUT_INVALID,
     60   *  POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST, or POSIX_ABSOLUTE_TIMEOUT_IS_NOW,
     61   *  then we should not wait.
    5862   */
    5963  status = _POSIX_Absolute_timeout_to_ticks( abstime, &ticks );
    60   switch ( status ) {
    61     case POSIX_ABSOLUTE_TIMEOUT_INVALID:
    62     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST:
    63     case POSIX_ABSOLUTE_TIMEOUT_IS_NOW:
    64       do_wait = false;
    65       break;
    66     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE:
    67     default:  /* only to silence warnings */
    68       do_wait = true;
    69       break;
    70   }
     64  if ( status != POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE )
     65    do_wait = false;
    7166
    7267  lock_status = _POSIX_Mutex_Lock_support( mutex, do_wait, ticks );
  • cpukit/posix/src/prwlocktimedrdlock.c

    r681d4526 r6805ac37  
    4444  Objects_Locations                            location;
    4545  Watchdog_Interval                            ticks;
    46   bool                                         do_wait;
     46  bool                                         do_wait = true;
    4747  POSIX_Absolute_timeout_conversion_results_t  status;
    4848 
     
    5858   *  then we do a polling operation and convert the UNSATISFIED
    5959   *  status into the appropriate error.
     60   *
     61   *  If the status is POSIX_ABSOLUTE_TIMEOUT_INVALID,
     62   *  POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST, or POSIX_ABSOLUTE_TIMEOUT_IS_NOW,
     63   *  then we should not wait.
    6064   */
    6165  status = _POSIX_Absolute_timeout_to_ticks( abstime, &ticks );
    62   switch (status) {
    63     case POSIX_ABSOLUTE_TIMEOUT_INVALID:
    64     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST:
    65     case POSIX_ABSOLUTE_TIMEOUT_IS_NOW:
    66       do_wait = false;
    67       break;
    68     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE:
    69     default:  /* only to silence warnings */
    70       do_wait = true;
    71       break;
    72   }
     66  if ( status != POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE )
     67    do_wait = false;
    7368
    7469  the_rwlock = _POSIX_RWLock_Get( rwlock, &location );
  • cpukit/posix/src/prwlocktimedwrlock.c

    r681d4526 r6805ac37  
    4444  Objects_Locations                            location;
    4545  Watchdog_Interval                            ticks;
    46   bool                                         do_wait;
     46  bool                                         do_wait = true;
    4747  POSIX_Absolute_timeout_conversion_results_t  status;
    4848 
     
    5858   *  then we do a polling operation and convert the UNSATISFIED
    5959   *  status into the appropriate error.
     60   *
     61   *  If the status is POSIX_ABSOLUTE_TIMEOUT_INVALID,
     62   *  POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST, or POSIX_ABSOLUTE_TIMEOUT_IS_NOW,
     63   *  then we should not wait.
    6064   */
    6165  status = _POSIX_Absolute_timeout_to_ticks( abstime, &ticks );
    62   switch (status) {
    63     case POSIX_ABSOLUTE_TIMEOUT_INVALID:
    64     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST:
    65     case POSIX_ABSOLUTE_TIMEOUT_IS_NOW:
    66       do_wait = false;
    67       break;
    68     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE:
    69     default:  /* only to silence warnings */
    70       do_wait = true;
    71       break;
    72   }
     66  if ( status != POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE )
     67    do_wait = false;
    7368
    7469  the_rwlock = _POSIX_RWLock_Get( rwlock, &location );
  • cpukit/posix/src/semtimedwait.c

    r681d4526 r6805ac37  
    5353   *  then we do a polling operation and convert the UNSATISFIED
    5454   *  status into the appropriate error.
     55   *
     56   *  If the status is POSIX_ABSOLUTE_TIMEOUT_INVALID,
     57   *  POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST, or POSIX_ABSOLUTE_TIMEOUT_IS_NOW,
     58   *  then we should not wait.
    5559   */
    5660  status = _POSIX_Absolute_timeout_to_ticks( abstime, &ticks );
    57   switch ( status ) {
    58     case POSIX_ABSOLUTE_TIMEOUT_INVALID:
    59     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_PAST:
    60     case POSIX_ABSOLUTE_TIMEOUT_IS_NOW:
    61       do_wait = false;
    62       break;
    63     case POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE:
    64       do_wait = true;
    65       break;
    66   }
     61  if ( status != POSIX_ABSOLUTE_TIMEOUT_IS_IN_FUTURE )
     62    do_wait = false;
    6763
    6864  lock_status = _POSIX_Semaphore_Wait_support( sem, do_wait, ticks );
Note: See TracChangeset for help on using the changeset viewer.