While working with the AnnotatedClassDiscovery, I figured it adds namespaces to the reader once with a second argument and once without (parent implementation). Turns out, a second argument does not exist in the called method - so let's remove the dead code passing a not used second argument.

Beta phase evaluation

Disruption Only internals are changed, thus no disruption possible.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fago created an issue. See original summary.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Quickfix

+1

Checked SimpleAnnotationReader::addNamespace does not have a second parameter

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed c021146 and pushed to 8.0.x. Thanks!

  • alexpott committed c021146 on 8.0.x
    Issue #2548301 by fago: Remove dead code from AnnotatedClassDiscovery
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.