Doom monster
Register | User Profile | Member List | F.A.Q | Privacy Policy | New Blog | Search Forums | Forums Home
Doomworld Forums : Powered by vBulletin version 2.2.5 Doomworld Forums > Special Interest > Freedoom > MFADEMAP
 
Author
All times are GMT. The time now is 03:20. Post New Thread    Post A Reply
wesleyjohnson
Forum Regular


Posts: 987
Registered: 04-09


I think that there is some extra garbage in the MFADEMAP lump.

The yadex editor keeps complaining (with my changes, so this may not be standard behavior) that the size of the MFADEMAP lump is not what it should be. It does not complain about the other fog maps.

The lump size should be 8704, but MFADEMAP is currently 8776 bytes.

Is anyone responsible for MFADEMAP, and is that extra actually contain anything ??

Old Post 03-05-10 19:47 #
wesleyjohnson is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
andrewj
Senior Member


Posts: 1625
Registered: 04-02


Yadex does not read or check colormap tables or fog maps.

This MFADEMAP lump must be sitting somewhere it does not belong, e.g. amongst all the flats.

Old Post 03-06-10 01:35 #
andrewj is online now Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
wesleyjohnson
Forum Regular


Posts: 987
Registered: 04-09


This Yadex does know about color map tables because I fixed it to have that capability, and sent the patches to the author.

MFADEMAP is the only one that gets this error message, otherwise it works fine. It looks like the lump has extraneous stuff in it and I am asking if the lump MFADEMAP is intended to be larger than needed.

Old Post 03-08-10 03:34 #
wesleyjohnson is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Gez
Why don't I have a custom title by now?!


Posts: 10987
Registered: 07-07


These numbers remind me of something...

I'm going to guess the added weight should simply be ignored; it's Inkworks' signature, and it's just a string that goes: "This lump was created by The Cookie Monsters InkWorks for DOOM (ver 1.1)" (version number may vary of course).

Old Post 04-06-10 10:56 #
Gez is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
wesleyjohnson
Forum Regular


Posts: 987
Registered: 04-09


That would explain where it came from.
The problem is that it sets off checks specifically put in to detect corrupt and malformed lumps.
Also we do not really need to include advertising in the FreeDoom lumps.
I request that we trim the advertising off.

Old Post 04-09-10 20:29 #
wesleyjohnson is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Gez
Why don't I have a custom title by now?!


Posts: 10987
Registered: 07-07



wesleyjohnson said:
The problem is that it sets off checks specifically put in to detect corrupt and malformed lumps.

Given that Inkworks is still the go-to program for creating palettes and colormaps, it would be a good idea for the checks to take that into account. After all, if this signature is there, it's because it works with vanilla.

Even if it is trimmed from Freedoom, there would still be a lot of other mods where a colormap features that signature.

Old Post 04-09-10 20:49 #
Gez is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
wesleyjohnson
Forum Regular


Posts: 987
Registered: 04-09


I doubt that I would change the checks just to allow for tools to include advertisements. I do not use Inkworks. If I had a program that put a little "extra" into the output, I would junk it and find another program. Sneaking something extra into the output is behavior that I find intolerable in a tool.
I would rather warn the user that advertising has been detected in a lump, so they can deal with it.

Who do they think is ever going to see that advertising in a colormap lump. They only do that so they can claim some sort of ownership over any output from their program.

Old Post 04-29-10 22:09 #
wesleyjohnson is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Gez
Why don't I have a custom title by now?!


Posts: 10987
Registered: 07-07


Look, I've made some minor contributions to SLADE3. It's during testing its ability to view colormaps and palettes that I've discovered the Inkworks issue. There are A LOT of mods with that.

Whether you want your port to be incompatible with vanilla-compatible mods is your own problem. I'm just pointing out that this thing exists, that it's not going to be fixed on Inkworks' side (it's not maintained anymore IIRC), it's not going to be fixed in the hundreds of mods which feature lumps created with Inkworks, and it's not a problem for the original game and all other ports that exist out there.

Old Post 04-29-10 22:48 #
Gez is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Graf Zahl
Why don't I have a custom title by now?!


Posts: 7687
Registered: 01-03


Ugh...

Reminds me of this


The general rule when developing tools is to make them work with the existing data, not to declare that data invalid and start whiny threads.

Bottom line: If you refuse to make your tool/port handle these lumps it'll be considered broken. End of story.

Old Post 04-29-10 22:55 #
Graf Zahl is offline Profile || Blog || PM || Email || Search || Add Buddy IP || Edit/Delete || Quote
wesleyjohnson
Forum Regular


Posts: 987
Registered: 04-09


I don't know what issue those last two reply posts have wandered off into, so I will restate what I said, and make it more obvious.

I do not like tools that slip advertising into their output. I have encountered them before, and I will abandon such tools given any chance to use another tool that does not do such things.
I have had to strip off such labeling by hand to use the output.
I have encountered tools that had an option to leave off the labeling
(someone else must have complained too).

I say this loudly so that any tool developer that might read this forum, that thinks it is cute to label the output with such advertising, might be warned off from doing such a thing.

Please stop reading other interpretations into this.

My patch to Yadex, that detects this odd sized lump, was included because there were other tests doing the same thing for other lumps and I included similar code in my patch for handling colormaps.

It correctly detects an odd sized colormap lump, and prints one line of warning that the lump is odd sized. It in no way prevents the user from proceeding. I have been using Yadex with this patch to create FreeDoom MAP13 and MAP09 over many months. It prints one warning line each time that FADEMAP is odd sized for a colormap lump.

The other colormaps must have been made with a different tool.

I believe that it is wrong to change the test of correct lump size in order to give into one badly behaved tool. The tool that that did it right should be credited for not invoking warning messages.

It is YADEX, the wad editor, that is detecting this.
It is possible that such advertising could break ZDoom, Legacy, or Eternity engines, if it was used on other lumps. I wish to discourage such ideas.

Last edited by wesleyjohnson on 04-30-10 at 22:32

Old Post 04-30-10 22:15 #
wesleyjohnson is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Graf Zahl
Why don't I have a custom title by now?!


Posts: 7687
Registered: 01-03


Much ado about nothing...

Old Post 04-30-10 23:00 #
Graf Zahl is offline Profile || Blog || PM || Email || Search || Add Buddy IP || Edit/Delete || Quote
All times are GMT. The time now is 03:20. Post New Thread    Post A Reply
 
Doomworld Forums : Powered by vBulletin version 2.2.5 Doomworld Forums > Special Interest > Freedoom > MFADEMAP

Show Printable Version | Email this Page | Subscribe to this Thread

 

Forum Rules:
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts
HTML code is OFF
vB code is ON
Smilies are OFF
[IMG] code is ON
 

< Contact Us - Doomworld >

Powered by: vBulletin Version 2.2.5
Copyright ©2000, 2001, Jelsoft Enterprises Limited.