16
-red
3y

Shitcode

Comments
  • 3
    @jespersh Yeaaaaah ! :( Should’ve mentioned that.
  • 2
    at least put it in array.
  • 4
    @jespersh You use a struct for items where the position changes their meaning, like in x/y/z coordinates. But then you also need to give them different names.

    Just enumerating them like in this case rather suggests that they should be an array, unless it's an improbable case of an eight-dimensional space where these are the coordinates.
  • 6
    @jespersh @Fast-Nop I currently use this to access them:

    @highlight
    *(&s.xvm0 + (i * (&s.xvm1 - &s.xvm0)))
  • 2
  • 2
  • 3
    @jespersh Yeah sorry about the lack of context. I was annoyed and wanted to quickly rant.

    I did not get a satisfactory answer so far for why it was written this way. And this code is so old and the original Dev is no longer at the company.
  • 6
    @-red I think you could improve clarity with the "offsetof" macro.
  • 4
    @Fast-Nop Well that’s handy. Didn’t know about it. Thanks for this.
Add Comment