It is currently 10 Sep 2025, 21:18
   
Text Size

Cleanup old functions/variables

Post MTG Forge Related Programming Questions Here

Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins

Re: Cleanup old functions/variables

Postby 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!
---
A joke is a very serious thing.
User avatar
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

Postby 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.
User avatar
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

Postby 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.
User avatar
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

Postby 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!
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

Postby 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: 8)

Code: Select all
<manifest>
    <mainClass>forge.view.Main</mainClass>
    <addDefaultImplementationEntries>true</addDefaultImplementationEntries>
</manifest>
User avatar
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

Postby 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

Postby slapshot5 » 18 Jan 2012, 03:57

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?
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

Postby 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)
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Cleanup old functions/variables

Postby slapshot5 » 18 Jan 2012, 04:12

Does that class exist?

Main.png
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

Postby 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.
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Cleanup old functions/variables

Postby slapshot5 » 18 Jan 2012, 04:32

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.
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

Postby 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. ](*,)
-Marc
User avatar
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

Postby slapshot5 » 19 Jan 2012, 04:20

How about shouldCycle, shouldTypeCycle, shouldTransmute and shouldSoulshift in CardFactoryCreatures? They're not being called anywhere.

-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

Postby ArsenalNut » 20 Jan 2012, 21:38

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.
So many cards, so little time
User avatar
ArsenalNut
 
Posts: 512
Joined: 08 Jul 2011, 03:49
Has thanked: 27 times
Been thanked: 121 times

Re: Cleanup old functions/variables

Postby 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
Programmer
 
Posts: 1391
Joined: 03 Jan 2010, 17:47
Location: Mac OS X
Has thanked: 25 times
Been thanked: 68 times

PreviousNext

Return to Developer's Corner

Who is online

Users browsing this forum: No registered users and 35 guests

Main Menu

User Menu

Our Partners


Who is online

In total there are 35 users online :: 0 registered, 0 hidden and 35 guests (based on users active over the past 10 minutes)
Most users ever online was 7967 on 09 Sep 2025, 23:08

Users browsing this forum: No registered users and 35 guests

Login Form