 |
Andrus Adamchik |
at Oct 21, 2008 at 4:21 pm
|
⇧ | 
| |
On Oct 21, 2008, at 11:31 AM, mgentry@apache.org wrote:
- public List get(QueryMetadata metadata) {
+ public List<?> get(QueryMetadata metadata) {
String key = metadata.getCacheKey();
if (key == null) {
return null;
Michael,
you may check this discussion on why returning List<?> from a method
is a bad idea:
http://markmail.org/message/ts4zjelowa55nfgghttp://markmail.org/message/cp63by2swbdnfp7vI suggest @SuppressWarnings("unchecked") instead.
Thanks,
Andrus
On Oct 21, 2008, at 11:12 AM, Michael Gentry wrote:
I was doing a little cleanup under org.apache.cayenne.cache and
FindBugs was issuing warnings about the final inner classes at the
bottom of MapQueryCache.java (CacheEntry) and OSQueryCache.java
(OSCacheAdministrator). Should these be declared as "final static
class" instead of "final class"?
Thanks!
