Menu

Commit [r946]  Maximize  Restore  History

Cleaning up class net.blockout.impl.LocalBlockoutBoard a little bit, in

preparation for the implementation of a Tetris board:
- Removed unused import statements, java.awt.Point & ClipRectangle.
- Now implements interface SuspendResume, as it should.
- No longer synchronizing around 'this' in calls to
addEventProcessedListener() and removeEventProcessedListener(); using new
private member 'm_evtProcessedListenerChainLock' instead, as per correct
multithreading practices; don't expose synchronization objects publicly.
- Marking the following methods as 'protected'; they used to be 'private':
* determinePerspective()
* determineProjectionToViewportTransform()
* drawEmptyWell()
* drawWellLayer()
* drawBlockUnit()
- Marking the following public methods as 'final', for integrity:
* suspend()
* resume()
* destroy()
* update(Graphics)
* getSnapshot()
* blockoutBoardAltered(BlockoutBoardEvent[])
* run()
- Marking private inner classes 'CalcPixBuff' and 'DummyEvtProcessedListener'
as final; event though this isn't necessary it certainly does not hurt.
- Minor changes to Javadoc comments.

nerius 2023-09-02

changed /blockout/src/net/blockout/impl/LocalBlockoutBoard.java
/blockout/src/net/blockout/impl/LocalBlockoutBoard.java Diff Switch to side-by-side view
Loading...
Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.