[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:49:10 -0500 |
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"
jwe