2009/10/30 Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>:
> Pavel Stehule wrote:
>> 2009/10/30 Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>:
>>> Pavel Stehule wrote:
>>>> else
>>>> {
>>>> ereport(ERROR,
>>>> (errcode(ERRCODE_INVALID_FUNCTION_DEFINITION),
>>>> errmsg("function result type must be specified")));
>>>> /* Alternative possibility: default to RETURNS VOID */
>>>>
>>>> /* WHY FOLOWING LINES? */
>>>> prorettype = VOIDOID;
>>>> returnsSet = false;
>>>> }
>>> To keep the compiler quiet about using the variables uninitialized. The
>>> compiler doesn't know that ereport(ERROR) never returns.
>>
>> Should be similar code little bit commented?
>
> *shrug*, maybe, often we do put a "/* keep compiler quiet */" comment on
> such places.
>
> On closer look, the "Alternative possibility: default to RETURNS VOID"
> comment suggests that besides keeping the compiler quiet, those lines
> demonstrate an alternative behavior that was considered.
I prefere "keep compiler quiet". It is cleaner - and it is used more times.
Pavel
>
> --
> Heikki Linnakangas
> EnterpriseDB http://www.enterprisedb.com
>