Index: firmware/App/Services/TxParams.c =================================================================== diff -u -r7652af03a25dfbaa4af4a7aeb0a228a16dcf979f -rc1df3cdad1542bda2dae4cc4a848d04ad47a479d --- firmware/App/Services/TxParams.c (.../TxParams.c) (revision 7652af03a25dfbaa4af4a7aeb0a228a16dcf979f) +++ firmware/App/Services/TxParams.c (.../TxParams.c) (revision c1df3cdad1542bda2dae4cc4a848d04ad47a479d) @@ -527,9 +527,7 @@ reasons[ TREATMENT_PARAM_TREATMENT_MODALITY ] = REQUEST_REJECT_REASON_PARAM_OUT_OF_RANGE; result = FALSE; } - - // TODO should execute the below for loop only if result is true, should I have this conditional check? - if ( TRUE == result ) + else { // Range check treatment parameters excluding treatment modality and up to (but not including) UF volume for ( param = TREATMENT_PARAM_HDF_TREATMENT_MODE; param < TREATMENT_PARAM_UF_VOLUME; param++ )