[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gnuplot-info-beta
Subject:    Re: NEW BUG: filled arrowheads not working
From:       Hans-Bernhard Broeker <broeker () physik ! rwth-aachen ! de>
Date:       2004-03-25 14:45:22
[Download RAW message or body]

On Thu, 25 Mar 2004, Hans-Bernhard Broeker wrote:

> Aargh.  The reason for that is of course, that do_arrow() will become
> term->arrow(), so it *must* match the prototype for that in struct
> TERMENTRY.  OTOH, I don't see any real need to touch that argument of
> do_arrow().  

But in the meantime, I did find it.  The head argument of term->arrow() is
indeed used to pass a literal '2'.  This really is bad, for at least two
reasons.

1) It clearly breaks the entire idea behind having a TBOOLEAN.

2) That '2' should be a named constant.  We don't want magic numbers.

-- 
Hans-Bernhard Broeker (broeker@physik.rwth-aachen.de)
Even if all the snow were burnt, ashes would remain.





-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
gnuplot-beta mailing list
gnuplot-beta@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/gnuplot-beta
[prev in list] [next in list] [prev in thread] [next in thread]