Uploaded image for project: 'Qt Design Studio'
  1. Qt Design Studio
  2. QDS-7691

Break dependencies of the model

    XMLWordPrintable

Details

    Description

      The model acquired many global dependencies over time. For example Exception::showException(). This is then used to be "better safe not sorry" mode in some code which makes it hard to break dependencies(it is even not clean what exception should be handled). I think the code should document which exception should be handled.

      I can only think about the rewriter exception and I think the code should describe that. With the new validation code we can remove most of the other exceptions and hopefully make it more clear.

      You can use different compile settings but that is complicating the build system and makes testing harder. There are different ways to fix it. Sometime a global variable(callback) is working but that has drawbacks too. Better are interfaces but that would result in large refactorings. So we should be pragmatical and use what works and clean it up over time.

      Attachments

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

        Activity

          People

            thohartm Thomas Hartmann
            bubke Marco Bubke
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:

              Gerrit Reviews

                There are no open Gerrit changes