[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [changeset] print() options not working
From: |
John W. Eaton |
Subject: |
Re: [changeset] print() options not working |
Date: |
Tue, 27 Jan 2009 22:53:24 -0500 |
On 27-Jan-2009, John W. Eaton wrote:
| On 3-Jan-2009, Ben Abbott wrote:
|
| |
| | On Jan 3, 2009, at 1:34 AM, Ben Abbott wrote:
| |
| | >
| | > On Jan 3, 2009, at 1:21 AM, Daniel J Sebald wrote:
| | >
| | >> I've noticed that the options for print() are not working
| | >> correctly. Attached is a short patch with a fix. Here are a
| | >> couple lines to illustrate the problem:
| | >>
| | >> function [ enhanced ] = gnuplot_set_term (plot_stream, h, term, file)
| | >> ...
| | >> if (nargin > 4 && ischar (opts_str))
| | >> ## Options must go last
| | >> term_str = sprintf ("%s %s", term_str, opts_str);
| | >>
| | >> I believe it should be > 2, not > 4. term_str is derived from term.
| | >>
| | >> Here is some code to try as before/after:
| | >>
| | >> x = [0:0.01:1]';
| | >> Y = [x sin(x) cos(x)];
| | >> plot(x,Y)
| | >> legend('x', 'sin(x)', 'cos(x)')
| | >> print('-deps', '-solid', 'test.eps')
| | >>
| | >> Dan
| | >>
| | >
| | > Dan, I've been working in this area quite a bit over the last
| | > several weeks. Perhaps this bug is my doing?
| | >
| | > In any event, I can take a look tomorrow.
| | >
| | > Ben
| |
| | I've attached a changeset attributed to you. I also fixed a few
| | function definitions that included brackets that are not consistent
| | with Octave's coding standard.
|
| What happened to the patch? The attachment I received only included
| this shell script:
|
| #!/usr/bin/env bash
| export OCTAVE_ARCHIVE=~/Development/mercurial/octave
| octave_changeset -u "Daniel Sebald <address@hidden>" -d "gnuplot_drawnow.m:
Bug fix for print options" -o "changeset-gnuplot_drawnow2.patch"
gnuplot_drawnow.m
| mate "changeset-gnuplot_drawnow2.patch"
Never mind, I see now that this change has apparently been made.
jwe