ExecutionControl::unsubscribe() comment disagreement with code

[simtheverse]

TrickHLA/source/SpaceFOM/ExecutionControl.cpp

Lines 477 to 483 in af2df5e

 void ExecutionControl::unsubscribe() 
 { 
    // Check to see if we are the Master federate. 
    if ( this->is_master() ) { 
       // Unsubscribe from the execution configuration if we are NOT the Master federate. 
       execution_configuration->unsubscribe_all_object_attributes(); 
    } 

Lines 480 and 481 seem to contradict each other. I am reading the implementation to be: Unsubscribe from the execution configuration if we ARE the Master federate.

Awesome project, thanks!

[Dan Dexter]

Yes, this is a bug. The check needs to be for if this federate is not the master. Thank you for your bug report.

[Dan Dexter]

Fixed and tested.