Fixed warnings due to missing @Override or missing @params in javadoc. In BasicProfilerState and BaseAppState

experimental
Rémy Bouquet 10 years ago
parent 74e4b9823a
commit 45c3776395
  1. 5
      jme3-core/src/main/java/com/jme3/app/BasicProfilerState.java
  2. 12
      jme3-core/src/main/java/com/jme3/app/state/BaseAppState.java

@ -61,7 +61,7 @@ public class BasicProfilerState extends BaseAppState {
private Geometry background; private Geometry background;
private float scale = 2; private float scale = 2;
private ProfilerKeyListener keyListener = new ProfilerKeyListener(); private final ProfilerKeyListener keyListener = new ProfilerKeyListener();
public BasicProfilerState() { public BasicProfilerState() {
this(false); this(false);
@ -84,6 +84,7 @@ public class BasicProfilerState extends BaseAppState {
* Sets the vertical scale of the visualization where * Sets the vertical scale of the visualization where
* each unit is a millisecond. Defaults to 2, ie: a * each unit is a millisecond. Defaults to 2, ie: a
* single millisecond stretches two pixels high. * single millisecond stretches two pixels high.
* @param scale the scale
*/ */
public void setGraphScale( float scale ) { public void setGraphScale( float scale ) {
if( this.scale == scale ) { if( this.scale == scale ) {
@ -101,6 +102,7 @@ public class BasicProfilerState extends BaseAppState {
/** /**
* Sets the number frames displayed and tracked. * Sets the number frames displayed and tracked.
* @param count the number of frames
*/ */
public void setFrameCount( int count ) { public void setFrameCount( int count ) {
if( profiler.getFrameCount() == count ) { if( profiler.getFrameCount() == count ) {
@ -229,6 +231,7 @@ public class BasicProfilerState extends BaseAppState {
private class ProfilerKeyListener implements ActionListener { private class ProfilerKeyListener implements ActionListener {
@Override
public void onAction(String name, boolean value, float tpf) { public void onAction(String name, boolean value, float tpf) {
if (!value) { if (!value) {
return; return;

@ -82,6 +82,7 @@ public abstract class BaseAppState implements AppState {
/** /**
* Called during initialization once the app state is * Called during initialization once the app state is
* attached and before onEnable() is called. * attached and before onEnable() is called.
* @param app the application
*/ */
protected abstract void initialize( Application app ); protected abstract void initialize( Application app );
@ -90,6 +91,7 @@ public abstract class BaseAppState implements AppState {
* application shutdown if the state is still attached. * application shutdown if the state is still attached.
* onDisable() is called before this cleanup() method if * onDisable() is called before this cleanup() method if
* the state is enabled at the time of cleanup. * the state is enabled at the time of cleanup.
* @param app the application
*/ */
protected abstract void cleanup( Application app ); protected abstract void cleanup( Application app );
@ -113,6 +115,7 @@ public abstract class BaseAppState implements AppState {
* This implementation calls initialize(app) and then onEnable() if the * This implementation calls initialize(app) and then onEnable() if the
* state is enabled. * state is enabled.
*/ */
@Override
public final void initialize( AppStateManager stateManager, Application app ) { public final void initialize( AppStateManager stateManager, Application app ) {
log.log(Level.FINEST, "initialize():{0}", this); log.log(Level.FINEST, "initialize():{0}", this);
@ -125,6 +128,7 @@ public abstract class BaseAppState implements AppState {
} }
} }
@Override
public final boolean isInitialized() { public final boolean isInitialized() {
return initialized; return initialized;
} }
@ -141,6 +145,7 @@ public abstract class BaseAppState implements AppState {
return getStateManager().getState(type); return getStateManager().getState(type);
} }
@Override
public final void setEnabled( boolean enabled ) public final void setEnabled( boolean enabled )
{ {
if( this.enabled == enabled ) if( this.enabled == enabled )
@ -157,22 +162,28 @@ public abstract class BaseAppState implements AppState {
} }
} }
@Override
public final boolean isEnabled() { public final boolean isEnabled() {
return enabled; return enabled;
} }
@Override
public void stateAttached( AppStateManager stateManager ) { public void stateAttached( AppStateManager stateManager ) {
} }
@Override
public void stateDetached( AppStateManager stateManager ) { public void stateDetached( AppStateManager stateManager ) {
} }
@Override
public void update( float tpf ) { public void update( float tpf ) {
} }
@Override
public void render( RenderManager rm ) { public void render( RenderManager rm ) {
} }
@Override
public void postRender() { public void postRender() {
} }
@ -182,6 +193,7 @@ public abstract class BaseAppState implements AppState {
* This implementation calls onDisable() if the state is enabled and * This implementation calls onDisable() if the state is enabled and
* then cleanup(app). * then cleanup(app).
*/ */
@Override
public final void cleanup() { public final void cleanup() {
log.log(Level.FINEST, "cleanup():{0}", this); log.log(Level.FINEST, "cleanup():{0}", this);

Loading…
Cancel
Save