Changeset 2034

Show
Ignore:
Timestamp:
02/03/12 03:20:28 (9 years ago)
Author:
jwoithe
Message:

rme: a return value of 0 from set*() indicates no error, so fix those locations where it was assumed a zero denoted an error. This is really only to prevent confusion later on: at present none of the affected controls rely on m_value being set.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/libffado/src/rme/fireface_settings_ctrls.cpp

    r2033 r2034  
    9494            break; 
    9595        case RME_CTRL_INPUT_LEVEL: 
    96             if (m_parent.setInputLevel(v)) { 
     96            if (m_parent.setInputLevel(v) == 0) { 
    9797                m_value = v; 
    9898            } 
    9999            break; 
    100100        case RME_CTRL_OUTPUT_LEVEL: 
    101             if (m_parent.setOutputLevel(v)) { 
     101            if (m_parent.setOutputLevel(v) == 0) { 
    102102                m_value = v; 
    103103            } 
     
    105105        case RME_CTRL_FF400_PAD_SW: 
    106106            // Object's "m_info" field is the channel 
    107             if (m_parent.setInputPadOpt(m_info, v)) { 
     107            if (m_parent.setInputPadOpt(m_info, v) == 0) { 
    108108                m_value = (v != 0); 
    109109            } 
     
    111111        case RME_CTRL_FF400_INSTR_SW: 
    112112            // Object's "m_info" field is the channel 
    113             if (m_parent.setInputInstrOpt(m_info, v)) { 
     113            if (m_parent.setInputInstrOpt(m_info, v) == 0) { 
    114114                m_value = (v != 0); 
    115115            } 
     
    141141            break; 
    142142        case RME_CTRL_PHONES_LEVEL: 
    143             if (m_parent.setPhonesLevel(v)) { 
     143            if (m_parent.setPhonesLevel(v) == 0) { 
    144144                m_value = v; 
    145145            }