On Fri, Jul 10, 2009 at 1:18 AM, Hezekiah M.
Carty<hezekiahca...@users.sourceforge.net> wrote:
> I would like to ask for comments on the patch before I commit it

That's a great idea.  I have already looked for a way to label the x
axis in Bode plots with k, M, T SI prefixes.

A couple of suggestions:

1. Options in comment of c_plbox were sorted by alphabet, so please
put the new one in appropriate place.

2. It could be clearer to make a typedef for a callback function's type.

3. It could make sense to pass an opaque void pointer to user-supplied
data to the callback.  Although I don't see a use for this currently,
it is always good practice to do so when writing code that uses
callbacks.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <griboz...@gmail.com>*/

------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to