Wouldn't it be clearer to do `else { RETVAL_NULL(); }` here, even if it's redundant?