Page 2 of 4

Re: Cleanup old functions/variables

PostPosted: 10 Jan 2012, 07:30
by Doublestrike
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!

Re: Cleanup old functions/variables

PostPosted: 17 Jan 2012, 10:49
by Doublestrike
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.

Re: Cleanup old functions/variables

PostPosted: 17 Jan 2012, 12:19
by Chris H.
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.

Re: Cleanup old functions/variables

PostPosted: 17 Jan 2012, 18:39
by silly freak
Make sure to check the pom.xml, I'm sure the main class must be referenced somewhere in there

Re: Cleanup old functions/variables

PostPosted: 17 Jan 2012, 19:36
by Chris H.
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: 8)

Code: Select all
<manifest>
    <mainClass>forge.view.Main</mainClass>
    <addDefaultImplementationEntries>true</addDefaultImplementationEntries>
</manifest>

Re: Cleanup old functions/variables

PostPosted: 18 Jan 2012, 03:54
by friarsol
Anyone else having problems with this? I can't seem to get my Eclipse to run properly now.

Re: Cleanup old functions/variables

PostPosted: 18 Jan 2012, 03:57
by slapshot5
friarsol wrote:Anyone else having problems with this? I can't seem to get my Eclipse to run properly now.
Haven't had any problems. What are you seeing?

Re: Cleanup old functions/variables

PostPosted: 18 Jan 2012, 04:04
by friarsol
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)

Re: Cleanup old functions/variables

PostPosted: 18 Jan 2012, 04:12
by slapshot5
Does that class exist?

Main.png

Re: Cleanup old functions/variables

PostPosted: 18 Jan 2012, 04:19
by friarsol
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.

Re: Cleanup old functions/variables

PostPosted: 18 Jan 2012, 04:32
by slapshot5
friarsol wrote:Edit: Just ended up nuking my /src/ directory and closing/opening project. Seems like it might be working now.
Yeah, unfortunately, sometimes Eclipse just sticks in finger in its ear. I've had to nuke more than once to fix random glitchy things.

Re: Cleanup old functions/variables

PostPosted: 18 Jan 2012, 05:23
by moomarc
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. ](*,)

Re: Cleanup old functions/variables

PostPosted: 19 Jan 2012, 04:20
by slapshot5
How about shouldCycle, shouldTypeCycle, shouldTransmute and shouldSoulshift in CardFactoryCreatures? They're not being called anywhere.

-slapshot5

Re: Cleanup old functions/variables

PostPosted: 20 Jan 2012, 21:38
by ArsenalNut
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. ](*,)
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.

Re: Cleanup old functions/variables

PostPosted: 24 Jan 2012, 02:50
by slapshot5
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