Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assertion `bp.height > 0' failed while using csdsetw.grf #168

Closed
DorpsGek opened this issue May 11, 2006 · 5 comments
Closed

Assertion `bp.height > 0' failed while using csdsetw.grf #168

DorpsGek opened this issue May 11, 2006 · 5 comments
Labels
component: NewGRF This issue is related to NewGRFs flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

Anonymous opened the ticket and wrote:

I am using CSDset v1.4.1 from www.tycoonez.com (http://ttd.tycoonez.com/download.php?id=36) with OpenTTD r4831 on Gentoo Linux. Whenever I try in arctic climate around year 1930 to open new rail vehicles tab game exits with "openttd: gfx.c:1397: GfxMainBlitter: Assertion `bp.height > 0' failed.".

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/168
@DorpsGek
Copy link
Member Author

peter1138 wrote:

This appears to be a bug in the GRF file. It explicitly sets the Multihead flag for the first steam engine (engine ID 24 / 0x18), but doesn't have GUI sprites for the second part.

It is actually pure luck that it doesn't crash in Temperate.


This comment was imported from FlySpray: https://bugs.openttd.org/task/168#comment295

@DorpsGek
Copy link
Member Author

DorpsGek commented Jun 7, 2006

PeS wrote:

Do you know the exact pseudosprite number, where this dual=head flag is set? I've tryed searching through nfo extrected using grfcodec and no 13 01 sequence was found in Action0 Train ( line# * SZ 00 00 <# ids> 13 01 ), or did I get it from ttdpatch wiki wrong? (0x13 should be for dual-head, 01 is on, 00 off). Thanks for any hints.


This comment was imported from FlySpray: https://bugs.openttd.org/task/168#comment314

@DorpsGek
Copy link
Member Author

DorpsGek commented Jun 7, 2006

PeS wrote:

I'v removed the dual-head flag with extra line in nfo using
3661 * 07 00 00 01 01 18 13 00
It works, but why I can't find sequence 13 01 in this file ?


This comment was imported from FlySpray: https://bugs.openttd.org/task/168#comment315

@DorpsGek
Copy link
Member Author

DorpsGek commented Jun 8, 2006

PeS wrote:

Ahh, I talked with the author of the set. The steam engine was set to dualhead to enable tender front of or after tank engine. When you buy it, it uses some randomization and you have it front of or after. But when you add vagon, you see only the first part (tank or tender), vagon and again tank or tender. See picture http://forum.tycoonez.com/download.php?id=1134 . In TTDPatch it works correctly, eg. tender-tank-coach or tank-tender-coach. Maybe some wrong interpretation in newgrf code?


This comment was imported from FlySpray: https://bugs.openttd.org/task/168#comment316

@DorpsGek
Copy link
Member Author

DorpsGek commented Dec 6, 2006

peter1138 closed the ticket.

Reason for closing: Fixed

Random crashes fixed in r7242. Now there will be a specific sprite-not-found error.


This comment was imported from FlySpray: https://bugs.openttd.org/task/168

@DorpsGek DorpsGek closed this as completed Dec 6, 2006
@DorpsGek DorpsGek added flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) component: NewGRF This issue is related to NewGRFs bug labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: NewGRF This issue is related to NewGRFs flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Development

No branches or pull requests

1 participant