open-dis / open-dis-java

Java implementation of the IEEE-1278.1 Distributed Interactive Simulation (DIS) application protocol v6 and v7 :boom:

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Deprecate DIS_DR_* and DIS_DeadReckoning classes

leif81 opened this issue · comments

In general, I think the previous Dead Reckoning DIS_DR_* implementation should be deprecated in favor of the single DIS_DeadReckoning class I recently added. For example, the previous DIS_DR_* implementations have thread safety issues #41.

Originally posted by @camejia in #42 (comment)

I believe Chris made a typo in that comment and was recommending use of the DeadReckoner class he wrote instead of the DIS_DeadReckoning and DIS_DR_* classes.:

https://github.com/open-dis/open-dis-java/blob/master/src/main/java/edu/nps/moves/deadreckoning/DeadReckoner.java

Another advantage of DeadReckoner is that it has extensive unit tests:

https://github.com/open-dis/open-dis-java/blob/master/src/test/java/edu/nps/moves/deadreckoning/DeadReckonerTest.java

Other issues affecting the classes to be deprecated:

If anyone has concerns with this sugestion can you please share. For example if you are using the original DIS_DR_* implementation and cannot for some reason migrate to the DeadReckoner. Could you please share your thoughts. Thanks.