[DiscordArchive] <@209419353035243520> Summoning additional brain
[DiscordArchive] <@209419353035243520> Summoning additional brain
Archived author: walkline • Posted: 2025-02-12T08:15:55.541000+00:00
Original source
<@209419353035243520> Summoning additional brain
Do you have any idea how a deleted creature could remain in the grid container?
dangling.txt
Archived author: Takenbacon • Posted: 2025-02-12T08:19:28.669000+00:00
Original source
it shouldn't, unless something is pretty well fucked
Archived author: Takenbacon • Posted: 2025-02-12T08:19:32.936000+00:00
Original source
grids are pretty solid
Archived author: walkline • Posted: 2025-02-12T08:21:12.449000+00:00
Original source
Maybe in some edge cases we are adding the deleted object to the grid again.
Archived author: Takenbacon • Posted: 2025-02-12T08:21:56.210000+00:00
Original source
m_tempspellTarget is the problem
Archived author: Takenbacon • Posted: 2025-02-12T08:22:40.626000+00:00
Original source
shouldn't ever, ever, ever store direct pointers to an object unless you damn well know what you're doing (and in this case, they don't know what they're doing..)
Archived author: Takenbacon • Posted: 2025-02-12T08:23:28.725000+00:00
Original source
change it to a lookup, looks like that object is most likely being deleted between Pet::Updates
Archived author: Takenbacon • Posted: 2025-02-12T08:28:18.504000+00:00
Original source
i also see a potential memory leak while you're there
![[Image: image.png?ex=690c5c39&is=690b0ab9&hm=757...ee5cde0d4&]](https://cdn.discordapp.com/attachments/284323424032129024/1339151180730990602/image.png?ex=690c5c39&is=690b0ab9&hm=757c6a8e570524b5d6989c4c2128adb5929e17a0e28e7753b82c595ee5cde0d4&)
Archived author: Takenbacon • Posted: 2025-02-12T08:28:41.765000+00:00
Original source
spell not being deleted if !unit_taget fails
![[Image: image.png?ex=690c5c39&is=690b0ab9&hm=757...ee5cde0d4&]](https://cdn.discordapp.com/attachments/284323424032129024/1339151180730990602/image.png?ex=690c5c39&is=690b0ab9&hm=757c6a8e570524b5d6989c4c2128adb5929e17a0e28e7753b82c595ee5cde0d4&)
Archived author: Takenbacon • Posted: 2025-02-12T08:32:19.611000+00:00
Original source
glad you're using address sanitizer though ❤️ you wouldn't believe the amount of problems it'll discover