Re: z_floor and z_ceiling (was Re: various observations)

[Daniel J S. <dan...@ie...>]

Hans-Bernhard Br=F6ker wrote:
> Daniel J Sebald wrote:
>=20
>> I can disallow that...  Other things to note about "ticslevel":  There=
=20
>> hasn't been a bug report for the bad behavior of the z-axis line so=20
>> that might mean it isn't used too often. =20
>=20
>=20
> It might equally well mean that other users don't consider it all that=20
> buggy.  Reading minds of people you don't even know exist is not really=
=20
> possibly.
>=20
>> "unset xyplane", wouldn't we expect that to remove the xyplane
>=20
>  > altogether?
>=20
> No.  I would expect that to be refused by the parser, given that it's=20
> not listed in 'help xyplane'.  Whatever 'unset xyplane' does besides=20
> causing an error message, is an undocumented feature, i.e. a bug.
>=20
>  > For example "unset xtics" removes the xtics.
>=20
> And is documented to do so.

I will disallow "unset xyplane" for now in the patch, unless you think "u=
nset xyplane" should be documentated instead.


>> What may have been nice from the start would be to force=20
>> set/show/reset to come as a structure with three functions.  (Or maybe=
=20
>> four, set/unset/show/reset.) =20
>=20
>=20
> Absolutely, except that there should actually be five, but they should=20
> be 'reset/set/unset/show/save').  If I had the time to do it these days=
,=20
> I would have.  If you want to try that, you have all my blessings.

Maybe post 4.2.  That change will be fairly easy (because the compiler wi=
ll complain about any problem with structure entries), but it is sort of =
a destablizing change for a while.  (Plus a patch like that could only go=
 a couple days without hunks being rejected.)

So, that would get rid of all the S_TICLEVELS, S_XAXIS, etc. (which will =
be nice).  The struct would be something like

{
"xyp$lane",
reset_xyplane,
set_xyplane,
unset_xyplane,
show_xyplane,
save_xyplane
};

And then maybe a short little routine to dump the string of set options, =
not exceeding 80 chars, to remove having to manually plut the option name=
 in that text string.

Dan