xNetHack icon indicating copy to clipboard operation
xNetHack copied to clipboard

Ceilings on outdoor levels

Open entrez opened this issue 3 years ago • 5 comments

Opening this PR just to document the ceiling stuff I did a while ago. Needs an extra field in struct rm so not for immediate use.

  • Add some missing ceiling_exists calls
  • Lay groundwork for des.add_ceiling()
  • Check for ceiling existing at a specific spot...
  • Improve(?) on ceiling-required trap generation
  • Require ceiling directly above for scroll of earth
  • Test the ceiling thing
  • Add ceilings to outdoors levels
  • Use separate bitmask for ceiling
  • Add roof to Sam-loca courtyard
  • Add another missing ceiling_exists call
  • Add wizmode option to highlight ceilings

entrez avatar Aug 19 '22 17:08 entrez

This will also need some changes after next vanilla merge, to replace xchar with coordxy.

entrez avatar Aug 19 '22 18:08 entrez

On the 8.0-savebreaking branch soon there is an "unused" bitfield in struct rm that is intended to have bits allocated away from it for new purposes such as this.

copperwater avatar Sep 04 '22 21:09 copperwater

Rebased onto 8.0-savebreaking

entrez avatar Sep 06 '22 20:09 entrez

(poorly, apparently...)

entrez avatar Sep 06 '22 20:09 entrez

Rebased onto master, so the coordxy stuff should be taken care of. Could probably take a look at it properly now and potentially adopt it (optionally after dropping the wizceiling option and squashing everything).

entrez avatar Jun 09 '23 18:06 entrez