search

Chris-plus-alphanumericgibberish / dNAO

Merge of dnethack onto the nethack.alt.org sources.
Other
20 stars 29 forks source link

Allow merging stacks with unknown but identical traits #2258

Closed RikerW closed 4 months ago

RikerW commented 4 months ago

This only changes behavior for 3 fields - known, bknown (BUC), rknown (rustproof). stolen is already handled and already merges or auto-IDs state, and dknown (color) has not been changed. i don't know what that would affect, but i think the idea is that dknown is set if you can see it, so if you don't have dknown you don't know ANYTHING. this is also keeping in line with vanilla/etc. re dknown

This does potentially affect behavior for unknown oprops, since I believe those use ->known. I consider this intentional.

Chris-plus-alphanumericgibberish commented 4 months ago

I put in the rknown and dknown change, but I'm leaving known and bknown as they currently are.