This deletion wasn't required for MacroGroovy, Cedric did it during his work on AST Matchers, and AFAIK it wasn't required for matcher's functionality as well, so it should be safe to put it back.
On Mon, Oct 12, 2015 at 10:03 AM Pascal Schumacher <pascalschumac...@gmx.net> wrote: > I was surprised that Cédric removed a public method from > ClassCodeVisitorSupport > with > https://github.com/apache/incubator-groovy/commit/248e0fd9530573d234212d7b3627e3d42168fde4, > because he is a big proponent of maintaining binary compatibility. > > -------- Weitergeleitete Nachricht -------- Betreff: Re: > [incubator-groovy] remove duplicated method visitBlockStatement added in > 9436c55fc5a14a20bc503c46c9a837753c846ff6 (5771929) Datum: Sun, 11 Oct > 2015 15:53:42 -0700 Von: Paul King <notificati...@github.com> > <notificati...@github.com> Antwort an: apache/incubator-groovy > <reply+00294f2d644f99731c2a962218e4120866bfc875c1257e3f92cf000000011232aa7692a163ce00d12...@reply.github.com> > <reply+00294f2d644f99731c2a962218e4120866bfc875c1257e3f92cf000000011232aa7692a163ce00d12...@reply.github.com> > An: > apache/incubator-groovy <incubator-gro...@noreply.github.com> > <incubator-gro...@noreply.github.com> Kopie (CC): Pascal Schumacher > <pascalschumac...@gmx.net> <pascalschumac...@gmx.net> > > Yes, this was removed as part of the MacroGroovy PR (which isn't part of > GROOVY_2_4_X). I had to put it back in for master to make the > LineColumnChecks work. We'll have to keep an eye out as to whether it > affects MacroGroovy or was just removed because it didn't seem to be > necessary during MacroGroovy rejigging. > > — > Reply to this email directly or view it on GitHub > <https://github.com/apache/incubator-groovy/commit/57719291957e34c409552412304f4c6268ff1735#commitcomment-13707349> > . > > >
