Hi Jay,
I'd keep the display check first, that way you don't have to check it multiple times.
Instead of the explicit checks, can we just use !?
if (!display || !glx_supported) { if (!display) results.error = strdup ("unable to open display"); ...
But the code looks sound.
« Back to merge proposal
Hi Jay,
I'd keep the display check first, that way you don't have to check it multiple
times.
Instead of the explicit checks, can we just use !?
if (!display || !glx_supported) {
if (!display)
results.error = strdup ("unable to open display");
...
But the code looks sound.