Forge 01/17 (unofficial BETA) version

[Mr.Chaos]

Thanks guys. Hopefully Memnarch won't cause me so much problems this time around. Last night's game with the 01/01 version was... less than entertaining.(see bugs section) But most of those errors seem to have been addressed in this new beta. Good times!

[Chris H.]

Yes, I haven't had a chance yet to test this on a mac... I assume that's why it won't work. Anyone else noticing this bug, on Linux or OSX maybe?

`
I have the same problem with the jar file that comes with your 01-17 beta release. I had hoped that the problem was related with corruption of some sort on my computer and that your build would be usable. No such luck.

I now think that there may be something about Apple's most recent java 6 update which may be incompatible with the google-collect or jsr305 jars.

We have several other people who also have Macs that use Forge. I will have to wait to see if they are having the same problem.

[Snacko]

Most likely it's a problem with path as mac os is case sensitive but windows isn't.

[Chris H.]

Most likely it's a problem with path as mac os is case sensitive but windows isn't.

`
My Mac's OS installed on my boot HD is formatted to be case insensitive. I think that I will try to reinstall snow leopard 6.0.2 OS combo update and the Java for Mac OS X 10.6 Update 1.

[Snacko]

I mean in Forge code, adding some logging to caught exceptions would probably help to pinpoint exact cause.

But this might also be some mac os related bug in java.

[Chris H.]

I mean in Forge code, adding some logging to caught exceptions would probably help to pinpoint exact cause.

But this might also be some mac os related bug in java.

`
I think that I may have discovered the source of the problem. Looking at my console output I see the line:

java.io.FileNotFoundException: /Applications/ Forge 01-17\res/pics\plains3.jpg (No such file or directory)

The problem appears to be with the slashes that signify the directory tree path to follow to find the jpg picture to display. Windows uses the back slash "\" and Mac OS uses the forward slash "/".

So, you are right about the problem being in the Forge code. I will see if I can find the hard coded path in my local copy and replace the "\" with "/" and Forge will then be able to find the pics to display.

[Chris H.]

- Fixed Mutavault.

`
Ahh, a new method:

card.clearSpellKeepManaAbility();

I saw this in the code for Svogthos, the Restless Tomb and suspected that it could be used to solve the Mutavault problem. I see that you beat me to it.

[Snacko]

For the path you need to repplace
GuiDisplayUtil.java
getURL()

Code: Select all

String fileString = path + "\\" +ForgeProps.getFile(loc)+"\\" +filename;


with

Code: Select all

String fileString = path + File.separator +ForgeProps.getFile(loc)+ File.separator +filename;

You should always use File.separator if you want the paths to work both under Windows and Linuxes/BSDs/Macs etcs.

[Chris H.]

Thank you, Snacko. I will try this with my local copy.

[DennisBergkamp]

Thanks guys. Hopefully Memnarch won't cause me so much problems this time around. Last night's game with the 01/01 version was... less than entertaining.(see bugs section) But most of those errors seem to have been addressed in this new beta. Good times!

Yes, I think I caught all of the bugs you brought up, but there might be a few more related to the planeswalkers. I'll go through the Planeswalker code sometime soon and double check each ability.

For the path you need to repplace
GuiDisplayUtil.java
getURL()

Code: Select all

String fileString = path + "\\" +ForgeProps.getFile(loc)+"\\" +filename;


with

Code: Select all

String fileString = path + File.separator +ForgeProps.getFile(loc)+ File.separator +filename;

You should always use File.separator if you want the paths to work both under Windows and Linuxes/BSDs/Macs etcs.

Ahh, my bad... I forgot about File.separator, thanks for pointing this out Snacko

[Chris H.]

Thank you, Snacko. I will try this with my local copy.

`
I tried it and it works.

I now get to see that the card picture panel is resizable. This is a great feature Dennis.


I used a file content searcher on my workspace folder and got five hits on "\\". I took a quick look and some of them are not hard coded file separators. The files that need to be checked are:

BackgroundPanel.java
CardFactory_Planeswalkers.java
Constant.java
GuiDispalyUtil.java (I found two different matches)
ImageJPanel.java

[DennisBergkamp]

Thank you, Snacko. I will try this with my local copy.

`
I tried it and it works.

I now get to see that the card picture panel is resizable. This is a great feature Dennis.


I used a file content searcher on my workspace folder and got five hits on "\\". I took a quick look and some of them are not hard coded file separators. The files that need to be checked are:

BackgroundPanel.java
CardFactory_Planeswalkers.java
Constant.java
GuiDispalyUtil.java (I found two different matches)
ImageJPanel.java

I didn't actually add the feature, the code Snacko posted automatically does this
One feature it did break though is the automatic zoom Vasiliy added. I'm not sure if it's worth adding back in since it probably causes performance issues.

[atar]

Thnx a lot for the new version of forge and for the great work you are doing.I experience severe lag while i'm playing,especialy when i'm choosing to play a spell from my hand? Any suggestions so i can optimize and to make it run without lagging?Thnx in advance?

[Chris H.]

I didn't actually add the feature, the code Snacko posted automatically does this.

`
Yes, we owe Snacko another thank you.

And, we also thank Vasiliy for the original code and you for your efforts to put this together.

[freestorageaccount]

Some of the prompts are a bit non-standard, such as 'destroy' rather than 'choose' target creature or enchantment for Ethersworn Adjudicator. I haven't run into any actual bugs except for storm-related ones so far though (other than incomplete implementations, which I'm sure won't be for long ). You all rock!

EDIT: My blocked Treva, the Renewer with double strike but without trample still triggered, and Megrim sometimes seems to forget its job especially with Gerrard's Verdict. Still, those are extremely minor.