Menu

#1724 Strange if block in the watchdog action trigger code

Future
closed
5
2012-05-25
2012-05-25
Anton Pak
No

File: plugins/dynamic_simulator/new_sim_watchdog.cpp
Function: NewSimulatorWatchdog::TriggerAction()

The following code contains very strange if block:

================
if ( m_state != PRETIMEOUT );
TriggerAction( PRETIMEOUT );
TriggerAction( TIMEOUT );
================

Discussion

  • Lars Wetzel

    Lars Wetzel - 2012-05-25
    • status: open --> closed
     
  • Lars Wetzel

    Lars Wetzel - 2012-05-25

    Fix in #7439
    :-) No comment

     
  • Lars Wetzel

    Lars Wetzel - 2012-05-25
    • status: closed --> closed-fixed
     
  • Anton Pak

    Anton Pak - 2012-05-25
    • status: closed-fixed --> closed
     
  • Anton Pak

    Anton Pak - 2012-05-25

    Lars, thanks!
    I suspected that it was simple but "m_state != PRETIMEOUT" brought some uncertainty.
    If it had been "m_state == PRETIMEOUT", I'd have been more sure! :)