Re: DPU-414 terminal

[Thomas S. <t.s...@fz...>]

> Could you please create a new tracker item for the patch and
> upload it there?

yes, ii'l do it.

> I notice a few things on a quick look.
> The various terminal types covered by this driver can share entry points.
> For example, there is no reason to have separate identical routines
> EPSON_reset() STARC_reset() NEC_reset() DPU414_reset().
> All four TERM_TABLEs can simply reference EPSON_reset().

there are some things in 'epson.trm' which should be repaired,
i noticed that for most of the terminals there's a hyphen used in
the name (e.g. nec-cp6) in the help text and in the manual,
but in 'set term' these names are accepted with underscore
only (nec_cp6). obviously because hyphens in terminal names
don't work (i first used 'dpu-414' and had problems with
'set term').

i'll go through 'epson.trm' and try to clean it up a little bit.

>> but the 
>> combination 'draft' + 'large' is forbidden because it drives the 'test'
>> command into an endless(?) loop.
>
> I'd rather fix the bug than forbid an otherwise reasonable combination.
> Anyhow, the bug seems to be more pervasive:
>   set term dpu large normal
>   set output 'test.dpu'
>   load 'histograms.dem'
>also goes into an infinite loop

the same applies for the pbm terminal.

> Can you work out where/why the loop happens?

well, i'll try to find where it happens, up to now i have
no idea where to search.
maybe it's due to clipping of characters which extend
the drawing area?
we'll see...

thomas
-- 
View this message in context: http://www.nabble.com/DPU-414-terminal-tp17371497p17398826.html
Sent from the Gnuplot - Dev mailing list archive at Nabble.com.