In this case I think the helper function reduces clarity rather than adds.
git-svn-id: svn+ssh://src.earth.threerings.net/nenya/trunk@643 ed5b42cb-e716-0410-a449-f6a68f950b19
This commit is contained in:
@@ -93,7 +93,8 @@ public class BundledComponentRepository
|
|||||||
ResourceBundle[] rbundles = rmgr.getResourceSet(name);
|
ResourceBundle[] rbundles = rmgr.getResourceSet(name);
|
||||||
if (rbundles == null) {
|
if (rbundles == null) {
|
||||||
// Couldn't find a bundle with that name, so just make empty maps for safe enumerating
|
// Couldn't find a bundle with that name, so just make empty maps for safe enumerating
|
||||||
fillInWithEmptyMaps();
|
_actions = Maps.newHashMap();
|
||||||
|
_classes = Maps.newHashMap();
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -148,11 +149,6 @@ public class BundledComponentRepository
|
|||||||
|
|
||||||
// if we failed to load our classes or actions, create empty hashtables so that we can
|
// if we failed to load our classes or actions, create empty hashtables so that we can
|
||||||
// safely enumerate our emptiness
|
// safely enumerate our emptiness
|
||||||
fillInWithEmptyMaps();
|
|
||||||
}
|
|
||||||
|
|
||||||
protected void fillInWithEmptyMaps(){
|
|
||||||
|
|
||||||
if (_actions == null) {
|
if (_actions == null) {
|
||||||
_actions = Maps.newHashMap();
|
_actions = Maps.newHashMap();
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user