You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sirona.apache.org by Romain Manni-Bucau <rm...@gmail.com> on 2014/03/21 00:22:25 UTC

AdviceAdapter

Hi

anyone knows well advice adapter from asm?

I tried to use it in javaagent but it fails:
https://gist.github.com/rmannibucau/376c1a6a85c4eb7cced3

I surely miss sthg but nothg obvious at this time

any help welcome

the idea is to remove the duplication of method...and support java 8
default methods ;)

here is a complete diff:
https://gist.github.com/rmannibucau/0d6b87e3f2ce88fc9afa

Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau

Re: AdviceAdapter

Posted by Romain Manni-Bucau <rm...@gmail.com>.

Ok found several issues.

Will commit today what I have (once I'll open a jira for this task ;)
but please test with caution since this is a very impacting change
Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau



2014-03-21 0:22 GMT+01:00 Romain Manni-Bucau <rm...@gmail.com>:
> Hi
>
> anyone knows well advice adapter from asm?
>
> I tried to use it in javaagent but it fails:
> https://gist.github.com/rmannibucau/376c1a6a85c4eb7cced3
>
> I surely miss sthg but nothg obvious at this time
>
> any help welcome
>
> the idea is to remove the duplication of method...and support java 8
> default methods ;)
>
> here is a complete diff:
> https://gist.github.com/rmannibucau/0d6b87e3f2ce88fc9afa
>
> Romain Manni-Bucau
> Twitter: @rmannibucau
> Blog: http://rmannibucau.wordpress.com/
> LinkedIn: http://fr.linkedin.com/in/rmannibucau
> Github: https://github.com/rmannibucau