Fix that deals with unusable custom variables.

Started by Thomas Hallgrenover 21 years ago2 messagespatches
Jump to latest
#1Thomas Hallgren
thhal@mailblocks.com

This is a bit embarrassing but the patch I submitted that enables custom
variable classes (8-10 months ago) was somewhat incomplete. The
DefineCustomIntVariable and DefineCustomRealVariable functions doesn't
have parameters that make it possible to set the min and max values of
the variables. Consequently, custom variables of int and real type are
completely useless. This patch adds the additional parameters minValue
and maxValue to those functions.

No one but me have used this so far (or someone would have noticed) so I
think it would be safe to backport this for 8.0.2.

Regards,
Thomas Hallgren

Attachments:

gucbug.difftext/plain; name=gucbug.diffDownload+12-0
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Thomas Hallgren (#1)
Re: Fix that deals with unusable custom variables.

Thomas Hallgren <thhal@mailblocks.com> writes:

This is a bit embarrassing but the patch I submitted that enables custom
variable classes (8-10 months ago) was somewhat incomplete. The
DefineCustomIntVariable and DefineCustomRealVariable functions doesn't
have parameters that make it possible to set the min and max values of
the variables. Consequently, custom variables of int and real type are
completely useless. This patch adds the additional parameters minValue
and maxValue to those functions.

No one but me have used this so far (or someone would have noticed) so I
think it would be safe to backport this for 8.0.2.

Applied to HEAD and 8.0.

regards, tom lane