Hi,

I have had a look at the Tcl examples to get rid of the locally
defined values - we now have a namespace containing the definitions
of "symbols" such as PL_POSITION_LEFT and FACETED. I noticed that
quite a lot of the examples use "bare" expressions, like:

     $w cmd plarc 0.0 0.0 [expr 0.1 * $i] [expr 0.1 * $i] 0.0 360.0

This is not the idiomatic way in Tcl: the advice is to always brace
your expressions, so that the above becomes:

     $w cmd plarc 0.0 0.0 [expr {0.1 * $i}] [expr {0.1 * $i}] 0.0 360.0

The reason is that a braced expression needs to be parsed only once.

My intention is to go through these examples and change the
unbraced expressions to braced expressions AND replace the locally
defined constants by the ones in the PLPLOT namespace that we now have.

But should I commit these changes separately? Or does that not matter?

Regards,

Arjen
 

DISCLAIMER: This message is intended exclusively for the addressee(s) and may 
contain confidential and privileged information. If you are not the intended 
recipient please notify the sender immediately and destroy this message. 
Unauthorized use, disclosure or copying of this message is strictly prohibited.
The foundation 'Stichting Deltares', which has its seat at Delft, The 
Netherlands, Commercial Registration Number 41146461, is not liable in any way 
whatsoever for consequences and/or damages resulting from the improper, 
incomplete and untimely dispatch, receipt and/or content of this e-mail.





------------------------------------------------------------------------------
The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE:
Pinpoint memory and threading errors before they happen.
Find and fix more than 250 security defects in the development cycle.
Locate bottlenecks in serial and parallel code that limit performance.
http://p.sf.net/sfu/intel-dev2devfeb
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to