Uploaded image for project: 'CDI Specification Issues'
  1. CDI Specification Issues
  2. CDI-28

Abstract methods on decorators

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • 1.1.EDR
    • 1.0
    • Decorators
    • None

      If a method is abstract:

      • if it is a method of a decorated type, it has a generated implementation that calls the delegate
      • otherwise, definition error

            [CDI-28] Abstract methods on decorators

            Pete Muir (Inactive) added a comment -

            We should hold this one open for 14 days.

            Pete Muir (Inactive) added a comment - We should hold this one open for 14 days.

            Marius Bogoevici (Inactive) added a comment -

            https://github.com/jboss/cdi/pull/26

            Marius Bogoevici (Inactive) added a comment - https://github.com/jboss/cdi/pull/26

            Marius Bogoevici (Inactive) added a comment - - edited

            George, Service Handlers is a different story (with the potential of being complementary if you will). Here we just leave decorator implementors to forgo stuff like

            void decoratedMethod() {
  return delegate.decoratedMethod();
}

            and leave decoratedMethod() unimplemented instead.

            Marius Bogoevici (Inactive) added a comment - - edited George, Service Handlers is a different story (with the potential of being complementary if you will). Here we just leave decorator implementors to forgo stuff like void decoratedMethod() { return delegate.decoratedMethod(); } and leave decoratedMethod() unimplemented instead.

            Pete Muir (Inactive) added a comment -

            Do you want to apply Gavin's change Marius?

            Pete Muir (Inactive) added a comment - Do you want to apply Gavin's change Marius?

            George Gastaldi added a comment -

            We should think about it when using Service Handlers

            George Gastaldi added a comment - We should think about it when using Service Handlers

              marius.bogoevici Marius Bogoevici (Inactive)
              pmuiratbleepbleep Pete Muir (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: