"set term wxt font" still has a problem

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

"set term wxt font" still has a problem

Jun T.
With the CVS HEAD (since the 'fix' of Bug#1713), I can't set
the font and font size simultaneously for wxt terminal.
The following sets the font but the font size is ignored:

gnuplot>set term wxt font 'Times,20'

In wxt.trm, line 183,

            if (s[sep] == ',')

is not satisfied if sep>0, because s[sep] has been
already set to null at line 181.

A possible simple patch is attached.


------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
_______________________________________________
gnuplot-beta mailing list
[hidden email]
Membership management via: https://lists.sourceforge.net/lists/listinfo/gnuplot-beta

wxt-set-font.patch (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: "set term wxt font" still has a problem

sfeam

On Tuesday, 26 January, 2016 18:36:07 Jun T. wrote:

> With the CVS HEAD (since the 'fix' of Bug#1713), I can't set

> the font and font size simultaneously for wxt terminal.

> The following sets the font but the font size is ignored:

>

> gnuplot>set term wxt font 'Times,20'

>

> In wxt.trm, line 183,

>

> if (s[sep] == ',')

>

> is not satisfied if sep>0, because s[sep] has been

> already set to null at line 181.

>

> A possible simple patch is attached.

 

 

Yes. Thank you for correcting my error in the earlier fix.

 

Ethan


------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
_______________________________________________
gnuplot-beta mailing list
[hidden email]
Membership management via: https://lists.sourceforge.net/lists/listinfo/gnuplot-beta