Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-18576

Issue warnings if translator methods are shadowed

    Details

      Description

      Shadowing can happen if you add en explicit translator method and forget to remove it from the constant or trivial mappings list. Check for this scenario and warn. Also check that const/trivial methods are not being inherited from a parent and overriding the versions defined in the current class.

        Attachments

          Activity

          Hide
          tjenness Tim Jenness added a comment -

          Simon Krughoff I hope you can review this small change that improves consistency checking. You might be interested in the code that checks to see if a method has already been defined in the current class.

          Show
          tjenness Tim Jenness added a comment - Simon Krughoff I hope you can review this small change that improves consistency checking. You might be interested in the code that checks to see if a method has already been defined in the current class.
          Hide
          krughoff Simon Krughoff added a comment -

          See PR for trivial comments.

          Show
          krughoff Simon Krughoff added a comment - See PR for trivial comments.

            People

            • Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Simon Krughoff
              Watchers:
              Simon Krughoff, Tim Jenness
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: