Cleanup old functions/variables
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
51 posts
• Page 2 of 4 • 1, 2, 3, 4
Re: Cleanup old functions/variables
by Doublestrike » 10 Jan 2012, 07:30
Hi fellas, I'm back! Glad to be back, I've been excited about refining the new UI.
Although I haven't read all the posts I missed while I was gone, this seems like a good place to chime in.
In my opinion, the old UI code can be sliced out completely since it is somewhat chunky and doesn't really follow the neat package organization present througout the working parts of the project.
One of the things I was worried about was new users (like @mark, looks like he's making some great contributions, thanks mark) being confused with all the legacy code. So I think it should be out ASAP, for that reason and that the project organization will leap forward further - view in view packages, card stuff in card packages, etc. It's a good thing.
All this is to say +1 on the ol' cleanup and I'm stoked to be back!
Although I haven't read all the posts I missed while I was gone, this seems like a good place to chime in.
In my opinion, the old UI code can be sliced out completely since it is somewhat chunky and doesn't really follow the neat package organization present througout the working parts of the project.
One of the things I was worried about was new users (like @mark, looks like he's making some great contributions, thanks mark) being confused with all the legacy code. So I think it should be out ASAP, for that reason and that the project organization will leap forward further - view in view packages, card stuff in card packages, etc. It's a good thing.
All this is to say +1 on the ol' cleanup and I'm stoked to be back!
---
A joke is a very serious thing.
A joke is a very serious thing.
-
Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Cleanup old functions/variables
by Doublestrike » 17 Jan 2012, 10:49
I'm starting to streamline the old/Braids/new UIs. I just condensed the "forge.view.swing" package into plain old "forge.view", since they're pretty much the same thing. Unfortunately that required moving the Main method so I hope this doesn't cause any problems.
---
A joke is a very serious thing.
A joke is a very serious thing.
-
Doublestrike - UI Programmer
- Posts: 715
- Joined: 08 Aug 2011, 09:07
- Location: Bali
- Has thanked: 183 times
- Been thanked: 161 times
Re: Cleanup old functions/variables
by Chris H. » 17 Jan 2012, 12:19
`Doublestrike wrote:I'm starting to streamline the old/Braids/new UIs. I just condensed the "forge.view.swing" package into plain old "forge.view", since they're pretty much the same thing. Unfortunately that required moving the Main method so I hope this doesn't cause any problems.
I went to Eclipse' Run -> Run Configuration menu command and set the Main class to:
- Code: Select all
forge.view.Main
and I can once again run forge from within Eclipse.
This added note might be useful for someone else.
-
Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: Cleanup old functions/variables
by silly freak » 17 Jan 2012, 18:39
Make sure to check the pom.xml, I'm sure the main class must be referenced somewhere in there
___
where's the "trust me, that will work!" switch for the compiler?
Laterna Magica - blog, forum, project, 2010/09/06 release!
where's the "trust me, that will work!" switch for the compiler?
Laterna Magica - blog, forum, project, 2010/09/06 release!
- silly freak
- DEVELOPER
- Posts: 598
- Joined: 26 Mar 2009, 07:18
- Location: Vienna, Austria
- Has thanked: 93 times
- Been thanked: 25 times
Re: Cleanup old functions/variables
by Chris H. » 17 Jan 2012, 19:36
`silly freak wrote:Make sure to check the pom.xml, I'm sure the main class must be referenced somewhere in there
The pom looks OK:

- Code: Select all
<manifest>
<mainClass>forge.view.Main</mainClass>
<addDefaultImplementationEntries>true</addDefaultImplementationEntries>
</manifest>
-
Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: Cleanup old functions/variables
by friarsol » 18 Jan 2012, 03:54
Anyone else having problems with this? I can't seem to get my Eclipse to run properly now.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Cleanup old functions/variables
by slapshot5 » 18 Jan 2012, 03:57
Haven't had any problems. What are you seeing?friarsol wrote:Anyone else having problems with this? I can't seem to get my Eclipse to run properly now.
- slapshot5
- Programmer
- Posts: 1391
- Joined: 03 Jan 2010, 17:47
- Location: Mac OS X
- Has thanked: 25 times
- Been thanked: 68 times
Re: Cleanup old functions/variables
by friarsol » 18 Jan 2012, 04:04
java.lang.NoClassDefFoundError: forge/view/Main
Caused by: java.lang.ClassNotFoundException: forge.view.Main
at java.net.URLClassLoader$1.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
at sun.misc.Launcher$AppClassLoader.loadClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
Caused by: java.lang.ClassNotFoundException: forge.view.Main
at java.net.URLClassLoader$1.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
at sun.misc.Launcher$AppClassLoader.loadClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Cleanup old functions/variables
by slapshot5 » 18 Jan 2012, 04:12
- slapshot5
- Programmer
- Posts: 1391
- Joined: 03 Jan 2010, 17:47
- Location: Mac OS X
- Has thanked: 25 times
- Been thanked: 68 times
Re: Cleanup old functions/variables
by friarsol » 18 Jan 2012, 04:19
Yep I was just looking at it.. Not sure what's going on. Something strange.
Edit: Just ended up nuking my /src/ directory and closing/opening project. Seems like it might be working now.
Edit: Just ended up nuking my /src/ directory and closing/opening project. Seems like it might be working now.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Cleanup old functions/variables
by slapshot5 » 18 Jan 2012, 04:32
Yeah, unfortunately, sometimes Eclipse just sticks in finger in its ear. I've had to nuke more than once to fix random glitchy things.friarsol wrote:Edit: Just ended up nuking my /src/ directory and closing/opening project. Seems like it might be working now.
- slapshot5
- Programmer
- Posts: 1391
- Joined: 03 Jan 2010, 17:47
- Location: Mac OS X
- Has thanked: 25 times
- Been thanked: 68 times
Re: Cleanup old functions/variables
by moomarc » 18 Jan 2012, 05:23
I keep having to nuke my cardsfolder every few switches between main and DKA branch. Start getting conflicts that just won't resolve so bombs away. ](./images/smilies/eusa_wall.gif)
](./images/smilies/eusa_wall.gif)
-Marc
-
moomarc - Pixel Commander
- Posts: 2091
- Joined: 04 Jun 2010, 15:22
- Location: Johannesburg, South Africa
- Has thanked: 371 times
- Been thanked: 372 times
Re: Cleanup old functions/variables
by slapshot5 » 19 Jan 2012, 04:20
How about shouldCycle, shouldTypeCycle, shouldTransmute and shouldSoulshift in CardFactoryCreatures? They're not being called anywhere.
-slapshot5
-slapshot5
- slapshot5
- Programmer
- Posts: 1391
- Joined: 03 Jan 2010, 17:47
- Location: Mac OS X
- Has thanked: 25 times
- Been thanked: 68 times
Re: Cleanup old functions/variables
by ArsenalNut » 20 Jan 2012, 21:38
One way to avoid this is to create two or more projects in your Eclipse workspace. I like to keep one project where I do modifications and one project that is a clean pull from the repository. In your case, you can point one of the projects at the DKA branch and one at the trunk. Just remember to keep only one project open at a time. I've had an occasional issue running Forge from Eclipse when both projects are open.moomarc wrote:I keep having to nuke my cardsfolder every few switches between main and DKA branch. Start getting conflicts that just won't resolve so bombs away.
So many cards, so little time
-
ArsenalNut - Posts: 512
- Joined: 08 Jul 2011, 03:49
- Has thanked: 27 times
- Been thanked: 121 times
Re: Cleanup old functions/variables
by slapshot5 » 24 Jan 2012, 02:50
src/main/java/org/eclipse/wb/swing package? Looks like it's not being used. Not sure when/if it ever was used. Anyone know?
-slapshot5
-slapshot5
- slapshot5
- Programmer
- Posts: 1391
- Joined: 03 Jan 2010, 17:47
- Location: Mac OS X
- Has thanked: 25 times
- Been thanked: 68 times
51 posts
• Page 2 of 4 • 1, 2, 3, 4
Who is online
Users browsing this forum: No registered users and 35 guests