Exclude packages from runtime proxy generation

[xfumihiro]

Fixes #32 #48

[xfumihiro]

For things other than nit fix, I think a PR will make things clear about what have changed.
And also, it's still a WIP.

[xfumihiro]

Now we wait for the patch get merged.

[autovalue]

Hi,
I'm not sure if I should file a new issue or just leave this as a comment.

This library doesn't seem to play nice with LeakCanary. It attempts to create a ProbeProxy subclass of its final custom views.

I think your processor code should probably check if a class is "final" before attempting to do stuff with it. Adding "com.squareup.leakcanary.internal.DisplayLeakConnectorView" to the "excludePackages" list doesn't seem to work either.

I think it would be nicer if the excludePackages entries were regex based, therefore removing the need to explicitly list every possible Views within a given package in the gradle file.

Do you mind isolating the resValue changes from the PR (as that's a new feature as far as I'm aware and we don't know how long google devs might take to merge your change in) and merging the bug fixes in so that we can test the fix on our apps?

[xfumihiro]

Automatically filtering out final views is an issue I afraid I don't have time to deal with because for the following several months I have to prepare for job interviews (:disappointed:). But hey, PRs are always welcome!
As for this issue (exclude package feature), I think the best approach requires the resValue patch (which also needed by #19) as an effective way to commute between plugin and runtime modules.

[autovalue]

Aah no worries dude, good luck with the interviews.
I've never really looked into plugin development. This might be the time I actually take the plunge.

All I need is IntelliJ idea right? No extra libraries/tool required.

[xfumihiro]

Yep. Go nuts! 👍

[xfumihiro]

To work with LeakCanary:

viewInspector {
  excludePackages = ['com.squareup.leakcanary.internal']
}

[xfumihiro]

@mikeyxkcd The package exclusion will skip views that starts with the given path. I don't think we need to make it regex based.