The class SideScroller should be renamed to Main

Stephcraft / Project-16x16

2D Platformer game made with processing

GNU General Public License v3.0

95 stars 75 forks source link

The class SideScroller should be renamed to Main #133

Closed Stephcraft closed 4 years ago

Stephcraft commented 4 years ago

A good way to do it: In eclipse, right click on the class symbol, refactor -> rename
File to be renamed: Sidescroller.java

mapuya19 commented 4 years ago

* A good way to do it: In eclipse, right click on the class symbol, refactor -> rename

* File to be renamed: `Sidescroller.java`
Is this still needed? I think I may have refactored it correctly, including references made to SideScroller in comments.

Stephcraft commented 4 years ago

@mapuya19 The pull request fixing this issue has been closed https://github.com/Stephcraft/Project-16x16/pull/135. It had some good progress on issue #134 Warning fixing. So branching from there could be a good starting point.

micycle1 commented 4 years ago

Better to split out some code from SideScroller (such as PApplet.main(SideScroller.class, args);) into a new class called Main, retaining Processing code in SideScroller -- perhaps this could be renamed to something like Game.