cleanup + formatting

This commit is contained in:
Hannes Janetzek 2013-09-03 05:33:47 +02:00
parent f26cc5b062
commit fba6f44f30
3 changed files with 13 additions and 44 deletions

View File

@ -82,10 +82,6 @@ public class RenderTheme implements IRenderTheme {
mElementCache[2] = new ElementCache(Element.POLY);
}
/*
* (non-Javadoc)
* @see org.oscim.theme.IRenderTheme#destroy()
*/
@Override
public void destroy() {
@ -98,29 +94,16 @@ public class RenderTheme implements IRenderTheme {
}
}
/*
* (non-Javadoc)
* @see org.oscim.theme.IRenderTheme#getLevels()
*/
@Override
public int getLevels() {
return mLevels;
}
/*
* (non-Javadoc)
* @see org.oscim.theme.IRenderTheme#getMapBackground()
*/
@Override
public int getMapBackground() {
return mMapBackground;
}
/*
* (non-Javadoc)
* @see org.oscim.theme.IRenderTheme#matchWay(org.oscim.core.Tag[], byte,
* boolean)
*/
@Override
public RenderInstruction[] matchElement(MapElement element, int zoomLevel) {
@ -143,7 +126,7 @@ public class RenderTheme implements IRenderTheme {
synchronized (cache) {
if (cache.prevItem == null || (cache.prevItem.zoom & zoomMask) == 0) {
if ((cache.prevItem == null) || (cache.prevItem.zoom & zoomMask) == 0) {
// previous instructions zoom does not match
cache.cacheKey.set(element.tags, null);
} else {
@ -266,10 +249,6 @@ public class RenderTheme implements IRenderTheme {
}
}
/*
* (non-Javadoc)
* @see org.oscim.theme.IRenderTheme#scaleStrokeWidth(float)
*/
@Override
public void scaleStrokeWidth(float scaleFactor) {
@ -277,10 +256,6 @@ public class RenderTheme implements IRenderTheme {
mRules[i].scaleStrokeWidth(scaleFactor * mBaseStrokeWidth);
}
/*
* (non-Javadoc)
* @see org.oscim.theme.IRenderTheme#scaleTextSize(float)
*/
@Override
public void scaleTextSize(float scaleFactor) {

View File

@ -110,16 +110,16 @@ public class RenderThemeHandler extends DefaultHandler {
*/
public static void logUnknownAttribute(String element, String name,
String value, int attributeIndex) {
StringBuilder stringBuilder = new StringBuilder();
stringBuilder.append("unknown attribute in element ");
stringBuilder.append(element);
stringBuilder.append(" (");
stringBuilder.append(attributeIndex);
stringBuilder.append("): ");
stringBuilder.append(name);
stringBuilder.append('=');
stringBuilder.append(value);
Log.d(TAG, stringBuilder.toString());
StringBuilder sb = new StringBuilder();
sb.append("unknown attribute in element ");
sb.append(element);
sb.append(" (");
sb.append(attributeIndex);
sb.append("): ");
sb.append(name);
sb.append('=');
sb.append(value);
Log.d(TAG, sb.toString());
}
private final ArrayList<Rule> mRulesList = new ArrayList<Rule>();
@ -140,7 +140,6 @@ public class RenderThemeHandler extends DefaultHandler {
}
mRenderTheme.complete(mRulesList, mLevel);
// mRenderTheme.mTextureAtlas = mTextureAtlas;
mTextureAtlas = null;
mRulesList.clear();
@ -281,8 +280,8 @@ public class RenderThemeHandler extends DefaultHandler {
if ((symbol.texture = mTextureAtlas.getTextureRegion(symbol.src)) == null)
Log.d(TAG, "missing texture atlas item '" + symbol.src + "'");
else
Log.d(TAG, "using atlas item '" + symbol.src + "'");
//else
// Log.d(TAG, "using atlas item '" + symbol.src + "'");
}
else if (ELEMENT_NAME_USE_STYLE_LINE.equals(localName)) {

View File

@ -223,7 +223,6 @@ public abstract class Rule {
// check subrules
for (Rule subRule : mSubRuleArray)
subRule.matchElement(type, tags, zoomLevel, matchingList);
}
}
@ -244,7 +243,6 @@ public abstract class Rule {
for (Rule subRule : mSubRuleArray)
subRule.onComplete();
}
public void onDestroy() {
@ -253,7 +251,6 @@ public abstract class Rule {
for (Rule subRule : mSubRuleArray)
subRule.onDestroy();
}
public void scaleStrokeWidth(float scaleFactor) {
@ -262,7 +259,6 @@ public abstract class Rule {
for (Rule subRule : mSubRuleArray)
subRule.scaleStrokeWidth(scaleFactor);
}
public void scaleTextSize(float scaleFactor) {
@ -270,6 +266,5 @@ public abstract class Rule {
ri.scaleTextSize(scaleFactor);
for (Rule subRule : mSubRuleArray)
subRule.scaleTextSize(scaleFactor);
}
}