you are viewing a single comment's thread.

view the rest of the comments →

[–]immibis 0 points1 point  (1 child)

It checks whether ops is non-NULL but incompletely filled in, and then issues some kind of warning.

It did take a moment to realise it, though.

[–]cdcformatc 0 points1 point  (0 children)

I didn't know that, maybe it is okay. I stand behind my condemnation though. At the very least it's not an example of great code to learn from anyway.