Fixup meaning of having multiple SignalMultiplexerValues

Author: Andreas Ringlstetter

include/dbcppp/Signal.h

@@ -136,11 +136,11 @@ namespace dbcppp
         DBCPPP_MAKE_ITERABLE(ISignal, ValueEncodingDescriptions, IValueEncodingDescription);
         DBCPPP_MAKE_ITERABLE(ISignal, AttributeValues, IAttribute);
 
-        /// \brief Mapping of this multiplexed signal to specific value ranges of a selected multiplexor switch signal.
+        /// \brief Mapping of this multiplexed signal to specific value ranges of a selected multiplexor switch signals.
         /// 
-        /// In a valid DBC, this can only have 0 or 1 entries.
         /// Requires EMultiplexer::MuxValue to be set in MultiplexerIndicator in order to be valid.
         /// If empty, simple multiplexing rules by ISignal::MultiplexerSwitchValue and IMessage::MuxSignal apply instead.
+        /// If not empty, all listed multiplexor signals must match the specified value ranges simultaniously.
         DBCPPP_MAKE_ITERABLE(ISignal, SignalMultiplexerValues, ISignalMultiplexerValue);
 
     protected:

src/SignalImpl.cpp

@@ -517,22 +517,6 @@ SignalImpl::SignalImpl(
         _phys_to_raw = ::phys_to_raw<double>;
         break;
     }
-    // Permitted due to faulty interface design in ISignal only - doesn't happen with real DBC files.
-    if (_signal_multiplexer_values.size() > 1)
-    {
-        SetError(EErrorCode::ConflictingMultiplexDefinition);
-    }
-    else if (_signal_multiplexer_values.size() == 1)
-    {
-        const auto ranges = _signal_multiplexer_values[0].ValueRanges();
-        if (!std::any_of(std::begin(ranges), std::end(ranges), [&](const ISignalMultiplexerValue::Range& range) -> bool
-            {
-                return range.from <= _multiplexer_switch_value && range.to >= _multiplexer_switch_value;
-            }))
-        {
-            SetError(EErrorCode::ConflictingMultiplexDefinition);
-        }
-    }
 }
 std::unique_ptr<ISignal> SignalImpl::Clone() const
 {