NFORenum v3.4.6 released (NFO renumberer and linter)

Discussions about the technical aspects of graphics development, including NewGRF tools and utilities.

Moderator: Graphics Moderators

DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

That didn't take as long as I expected. 'Twas just a matter of reading the changes a few more times.

Changelog:
v2.3.2 to v2.3.3
- Parses a54's new variational action 2s.
- Checks for valid operation in new variational action 2s.
- Added <variable> to message 84. (Testing nonexistent variable)
- (bugfix) Unused industry cargoIDs are now reported.
- Added variable 18 for industries.

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
User avatar
George
Tycoon
Tycoon
Posts: 4362
Joined: 16 Apr 2003 16:09
Skype: george-vb
Location: Varna, Bulgaria
Contact:

Post by George »

Code: Select all

//!!Fatal Error (43): Linter requires 1 byte(s) at offset 19 which do(es) not exist.
   70 * 18	 02 01 F1 81 0C 00 FF 02 00 FF 12 12 00 FF 15 15 F0 00
What does it mean? What is wrong with this string?
Image Image Image Image
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

What it means is that I got lazy and assumed that I couldn't possbly do something *that* stupid. (OK, what it really means is that NFORenum tried to read bytes that didn't exist, but it's my fault it tried to do that.)

Changelog:
v2.3.3 to v2.3.4
- (bugfix) No longer overruns the sprite on variational action 2s where <nvar> is greater than 1.

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

I really would be much happier if I didn't have to post all these bugfix releases, but I suppose it's better than the alternative: leaving the bugs in the code.

Changelog:
v2.3.4 to v2.3.5
- (bugfix) Now gives the correct offset for an incorrect operation code.
- (bugfix) Enabled variable 80 for vehicles. (Used in AI callbacks.)
- (bugfix) Reports correct location of preceeding action 1.

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Csaboka keeps breaking NFORenum. Whether this is a good thing or a bad thing depends on whether I'm writing NFO files or trying to keep NFORenum up to date.

Changelog:
v2.3.5 to v2.3.6
- Up to date with alpha 55vcs
- - newcargos
- - global variable 1A
- - multiply, and, or, and xor operations
- Added warning for too many sprites in set.
- Added warning for an unusual set size (features 0-3, sizes 5-7)

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Newhouses grew two new properties, so we get to do this procedure again. Replace your old 0.dat (372 bytes) with this one (374 bytes).

Changelog:
- Added new house props 1D and 1E.

(New version; downloads removed)
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

With all the features I added, I probably broke something, but my test suite didn't find anything, so here's the latest from me, after catching up with Csaba yet again.

Changelog:
v2.3.6 to v2.4.0
- Checks actions D, E, F
- Up to date with alpha 55 vcs 3
- - House props 1D, 1E (a55vcs2)
- - Parameterized variable support
- - Station variables 48, 60..64
- - House variables 60, 61
- - New house/industry tile action 2 format
- (bugfix) probable crash on incorrect length basic 07/09 standard 2s.
- (bugfix) color translation checks didn't.

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

The post about the recursive action 2 bug disappeared, which is OK, I guess, because I've fixed the bug.

Changelog:
v2.4.0 to v2.4.1
- Cargo prop 15 (alpha 57)
- (bugfix) Action 2s can refer to their own ID.
- Train variable 60 (alpha 57)

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Hmm... Not good.

Please don't try to run NFORenum 2.4.1 or earlier on any NFO files containing binary include sprites (<spritenum> ** <filename>); it may well eat them. (If you've got the header set correctly, NFORenum will fail instead.)

The pseudo-sprites that grfcodec 0.9.6 and earlier will output when decoding binary includes are acceptable, AFAIK.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Rest assured, I got soundly LARTed for these last two bugs here. :oops: :oops:
I think I may have to go into hiding now.

Changelog:
v2.4.1 to 2.4.2
- Added check for Action E disabling reserved GRFID (FF .. .. ..)
- Increases nfo version, if necessary.
- Up to date with documentation for alpha 57:
- - Binary include sprites (and their pre-grfcodec 0.9.7 equivilants)
- - Action 11
- (bugfix) Fixed resource management feature check.
- (bugfix) Now complains about 79Dv.dat, not 7Dv.dat.
- Added <count> checks for resource management.
- Removed the whole "Can disable following warnings/errors" concept. (and some associated bugs.)
- (bugfix) No longer eats NFO files with an unknown version.
- (bugfix) No longer eats NFO files that follow a file with an unknown version.

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

I was hoping for more documentation before I released this.
I was also hoping that I wouldn't have quite this many bugfixes.
Unfortunately, neither was to be.

Changelog:
v2.4.2 to v2.4.3
- Removed check for not-variational action 2 (now it's a callback return).
- (bugfix) Fatal errors on action 2s define the associated ID.
- States version on startup.
- (bugfix) Using standard in/out works again. Also, explicitly state that stdin/out are being used.
- (bugfix) Correct handling of quoted newlines.
- Checks ID(s) given in actions 0,3,4 (1 byte IDs only in 4s)
- (bugfix) No longer errors on no-groundsprite house action 2s.
- Checks for at least one non-transparent building sprite if no ground sprite in house/industry tile action 2s.
- (bugfix) GRFids no longer come out reversed on LE systems.
- (bugfix) Reports the correct sprite number if an incorrect number of reals follow an action 5.
- (bugfix?) Probably some fixes in standard action 2 handling.
- Checks <num-cid> in action 3s (must be 0 for features 5,7,9..B).
- Attempted to be up-to-date with a58:
- - Train props 27..29
- - RV props 1C..1E
- - Ship props 17..19
- - Plane props 17..19
- - Cargo prop 16
- - Variable 1B
- - Industry tile variable 60
- Checks that livery overrides are only used with features 0 and 3.

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Due to something known as "lack of forethought", AKA "Dale ain't too smart", I have to dump a new executable on you folks if you want to set properties for newsounds.

The rest of the changes I have queued up are going to wait until I redo the the datafile system, in hopes of eliminating this problem in the future.

Changelog:
v2.4.3 to v2.4.4
- Up to date with a58:
- - Newsounds properties

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
User avatar
George
Tycoon
Tycoon
Posts: 4362
Joined: 16 Apr 2003 16:09
Skype: george-vb
Location: Varna, Bulgaria
Contact:

Post by George »

I have the following problems:
"Cargo 1E is undefined" for my trucks
and
"Variable 10 is undefined" for callback 2E
Image Image Image Image
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

George wrote:"Cargo 1E is undefined" for my trucks
That appears on an action 3, I hope? Assuming so, a fixed exe should be up in an hour or so.
George wrote:"Variable 10 is undefined" for callback 2E
I've fixed 2v.dat, but I guess I don't see much point in uploading it when I'm going to have to update the whole distro anyway.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Sometimes I'm tempted to back the whole linting system out and send NFORenum back to what it was originally planned to be. Somehow I think you all would complan about that, though.

Also, I appear to have given out the wrong versions for some of the 2.4.4 source, which will probably cause bad things to happen on newsound action 1..4s (which are verboten anyway); the exe was fine, though, and I'm pretty sure I've got the right files this time.

v2.4.4 to v2.4.5
- (bugfix) Added variables 10 and 18 to houses.
- (bugfix) Enabled cargotypes 1D..1F in action 3s.

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Come to think about it, I'd complain if I scrapped the linter too.

Another bugfix, some catchup, and some code reworking that will eventually make NFORenum more flexible.

v2.4.5 to v2.4.6
- (bugfix) Revert to 2.4.2's newline handling.
- (packing) Removed an old file from source dir.
- Up to date with a59pre3:
- - Action 2 variable 7F
- - Action 2 trains variable 46
- - Action D operations 09..0C
- Renumbered messages.

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Because D.dat somehow failed to get updated in the latest archive, here it is.

EDIT: It turns out I failed to update the entire .renum directory, but only for the .zip and .rar.

Here are the two files that should have been included:
2v.dat: 236 bytes -> 267 bytes
D.dat: 27 bytes -> 28 bytes

EDIT2: OP updated. If this line was present when you downloaded from the OP, you do not need this update.

EDIT3: New version; download from OP instead.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Nothing much to say here. More catch-up, moved more stuff to datafiles.

Changelog:
v2.4.6 to v2.4.7
- Up to date with a59:
- - Added variables 1C, 5F.
- - Added train variable 46.
- - Added station variable 49.
- (change) Added variable 42 to RVs, ships, planes. (only 1 byte, though)
- Added property to message 65 (unexpected end-of-sprite).

See first post for full details and download.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
User avatar
George
Tycoon
Tycoon
Posts: 4362
Joined: 16 Apr 2003 16:09
Skype: george-vb
Location: Varna, Bulgaria
Contact:

Post by George »

DaleStan wrote:Nothing much to say here. More catch-up, moved more stuff to datafiles.
2 problems wirh houses

//!!Warning (86): Testing nonexistant variable 10.
392 * 22 02 07 10 81 10 00 FF 03 03 9A 00 00 05 80 01 01 02 82 02 02 FF A0

//!!Warning (103): Shifting variable 44 past its length.
582 * 14 02 07 2B 81 44 10 FF 01 29 00 00 00 2A 00
Image Image Image Image
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

George wrote://!!Warning (103): Shifting variable 44 past its length.
582 * 14 02 07 2B 81 44 10 FF 01 29 00 00 00 2A 00
Officially, yes, that variable is a dword, but it looks to me like that line checks a reserved byte.
The wiki wrote:Building counts (44)

Returns dword rrrrCCcc,
The other is a bug, and can be fixed with this 2v.dat (293 bytes -> 297 bytes)

Changelog:
- Added variables 10 and 18 to houses (Note to self: Check the callbacks page carefully.)

Oh, and:
TODO: find some graceful way of dealing with var 1C. (Currently, the "duplicate-default" will get dropped on you if you try to use it as the wiki suggests.)

EDIT: updated, downloads removed
Last edited by DaleStan on 14 Sep 2005 15:37, edited 1 time in total.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
Locked

Return to “NewGRF Technical Discussions”

Who is online

Users browsing this forum: No registered users and 0 guests