3 ms·
At first brush those comments seem overly verbose and obvious: /* Do we have any named arguments? */ ... /* If so, we must apply reorder_function_a
by d23 14y ago
At first brush those comments seem overly verbose and obvious:
/* Do we have any named arguments? */
...
/* If so, we must apply reorder_function_arguments */
if (has_named_args)
{
args = reorder_function_arguments(args, func_tuple);
- noselasd 14y agoThough I do appreciate comments such as this: /* Fetch the function body */ tmp = SysCacheGetAttr(PROCOID, func_tuple, num_pg_proc_prosrc, &isNull); Newcomers have be be quite familiar with the internals to immediately see that the line fetches the function body.