Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-70449

Consolidate the error messages handling in the CAN plugins

    XMLWordPrintable

Details

    • Suggestion
    • Resolution: Done
    • P2: Important
    • 5.14
    • 5.12
    • SerialBus: CAN Bus
    • None
    • fd9b9c73ca60c66d01376ddbd6d36e9ea2dfa95c

    Description

      Right now we have two means to report errors during operation:

      1. print log messages with qCDebug/qCWarning
      2. call QCanBusDevice::setError(), which emits a signal

      For now, different places handle errors in different ways, some only log the error, some only emit the signal.

      The best solutions would IMHO be to do both, as it allows the end user to choose how to react to errors:

      • connect to errorOccurred() and handle the problem in user code
      • enable the logging to see errors in a log
      • ignore errors by disabling logging and not connecting to the errorOccurred() signal

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          People

            aha_1980 André Hartmann
            aha_1980 André Hartmann
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes