> So now we have three different classes involved in the mix. > > Constants defines the strings. > JDOHelper defines the Map<String> containing those strings. > JDOImplHelper defines the method to check the Map. > > How about Constants creates the public static immutable Map<String> > containing the strings, and leave JDOHelper out of it? > > Craig > Yeah, I noticed that, too. Constants is an interface; can't do it there. I could put the immutable Set<String> in JDOImplHelper, but I thought users might want to make use of it. I think I prefer moving it to JDOImplHelper. JDOHelper already depends on JDOImplHelper as it is anyway.
-matthew
