This change makes the axis much more readable. To use it, graphics.pl
must pass the $space parameter. I have attached a patch that in my
installation produces more pleasing output. Try it out and see if you
like it, then we can commit. You are right, the pod needs updating... so
sorry... Th.
> -----Original Message-----
> From: Robert A. Schmied [mailto:ras
AT
acm.org]
> Sent: Friday, March 21, 2008 7:41 PM
> To: devel
AT
geniustrader.org
> Subject: Re: [GT] SVN Commit r582 - trunk/GT/Graphics
>
> >
> > =cut
> > sub build_axis_for_timeframe {
> > - my ($prices, $timeframe, $put_label, $period) = @_;
> > + my ($prices, $timeframe, $put_label, $period, $space) = @_;
>
> this is a newly added capability? i don't find any calls that pass a
> value for $space in the (older) repo (or in my working environment)
>
> the additional parameter isn't included in the pod for this method?
> are the units pixels or ????
>
> i think i like what it might be doing, but it needs something to use
it ...
> maybe an explanatory note say hey look at this neato improvement with
> a before and after picture? or an example of how to use it ...
>
> or did i miss all the above someplace else?
Attachment:
graphic.pl.diff
Description: graphic.pl.diff