Conversation

Notices

1. Embed this notice
   aeva (aeva@mastodon.gamedev.place)'s status on Sunday, 14-Jan-2024 21:34:04 JST aeva

   • egg boy color

   I'm running this code in a debug build.

   The value of "CellHasInterpreter" before the branch is false. For some reason it is just running headlong into the assertions and crashing. If I were to remove the assertions, it'll instead run headlong into dereferencing "Interpreter", which is a nullptr.

   Anyways here's the disassembly if anyone is interested in giving me a second opinion: https://gist.github.com/Aeva/36fe522762566dd4a0898a83eb4bb96b

   EDIT: the award goes to @eggboycolor for spotting the stray semicolon

   • Embed this notice
     egg boy color (eggboycolor@mastodon.social)'s status on Sunday, 14-Jan-2024 21:34:03 JST egg boy color

     in reply to

     @aeva If you haven't found already: It looks like there's a semicolon after the if, before the newline and opening brace, so the if statement running on an empty statement, and then entering the block afterwards. These sorts of typo bugs always suck to debug since there's often no warning until you crank the warning levels esp in Visual Studio, and often they don't warn about "allowed but not actually desired" code :/

     clacke likes this.
   • Embed this notice
     aeva (aeva@mastodon.gamedev.place)'s status on Sunday, 14-Jan-2024 21:34:08 JST aeva

     in reply to

     • egg boy color

     @eggboycolor one of these days I'm gonna turn into one of those all warnings are errors people 🙃

     clacke likes this.
   • Embed this notice
     aeva (aeva@mastodon.gamedev.place)'s status on Sunday, 14-Jan-2024 21:34:10 JST aeva

     in reply to

     • egg boy color

     @eggboycolor marvelous! that was it! thank you so much

     clacke likes this.
   • Embed this notice
     clacke (clacke@libranet.de)'s status on Sunday, 14-Jan-2024 21:34:12 JST clacke

     in reply to

     • egg boy color

     @eggboycolor @aeva An argument against having the curly after the newline. 😁

     *ducks*

   • Embed this notice
     aeva (aeva@mastodon.gamedev.place)'s status on Monday, 15-Jan-2024 16:21:39 JST aeva

     in reply to

     • clacke
     • egg boy color

     @clacke @eggboycolor yeah great idea I'll just go reformat 100000000 lines of code right now

     clacke likes this.
   • Embed this notice
     Simon Richter (gyrosgeier@hachyderm.io)'s status on Monday, 15-Jan-2024 16:21:43 JST Simon Richter

     in reply to

     • egg boy color

     @aeva @eggboycolor you could do that gradually, start with -Werror=empty-body :>

     clacke likes this.