dondiego

It's fixed i think, but now the menu opens and i don't remember that happening in vanilla. In D_ProcessEvents the menu ate the event: ``` while ((ev = D_PopEvent()) !=...

I've pushed another commit to prevent opening the menu while chatting. https://github.com/drfrag666/RUDE/commit/38302caaa4b30770088775282ddb3d47999b83f5

In the end my original commit was wrong but the last one fixed the problem.

Well the question is what vanilla did. The marker is 0xFFFFF and for a DWORD that is 65535 and for a WORD -1, does it matter? How can we check/see...

Then i think it's better to continue.

And you can't continue since the blockmap is broken and it would crash anyway, there are no more lines. Either it's chopped or it's just a bad marker and you...

For me it still crashes with your change and RelWithDebInfo, same value for *list as above. ``` #0 0x441969 PIT_AddLineIntercepts(ld=0x31b85a4) (C:\DEV\rude\src\doom\p_maputl.c:592) #1 0x441501 P_BlockLinesIterator(x=x@entry=29, y=y@entry=23, func=func@entry=0x441930 ) (C:\DEV\rude\src\doom\p_maputl.c:497) #2 0x44219f...

I think it's fixed, valid values for blocks are between 0 and 32767 (short type). https://github.com/drfrag666/chocolate-doom/commit/c5910991754f951f5dcc5035379e8ac1643723aa Edit: fixed to use the short limit macro. https://github.com/drfrag666/chocolate-doom/commit/846f7553d00de8ff1928e9d84634bbec45661136

It was wrong and broke maps with extended nodes. USHRT_MAX didn't work and seems @jeffdoggett was right and we should put numlines there. https://github.com/drfrag666/RUDE/commit/e9dedebb0b11d3105401f3b089948f7b35601a62

There wasn't such a check before. Same as in vanilla probably. If there wasn't a -1 then the game would hang and then crash so i think the blockmap is...