tree 7bd3cc862c4b1799d424ace09f72c135f1801708
parent ad13485b77a05590f767920224fb44c1bb564894
author Yigit Boyar <yboyar@google.com> 1609365799 -0800
committer Yigit Boyar <yboyar@google.com> 1609366752 -0800

Make sure there is only 1 instance of each type element

When traversing classes, we'll frequently wrap the same type
element in different cases (e.g. finding all methods of a type
element). Previously, we only cached results in findTypeElement
which meant we could wrap the same type element multiple times
if XProcessingEnv.wrap is called.

This CL updates the type element store to also accept the backing
type as key. When invoked, it will first get the qualified name
to query the cache before wrapping a new item.

Note that java does not allow conflicts on qualified names so
there is no uniqueness concerns (e.g. you cannot have foo.Bar.Baz
and foo.Bar class with Baz static inner class)

Bug: 160322705
Test: XProcessingEnvTest.typeElementsAreCached
Change-Id: Ic0fba8ea8ce80946afc642cd49ab48aa18a0e70b