On Thursday, December 23, 2010 at 09:28:20 (+0100) José Luis García Pallero writes: > Attached I send a patch for svn plfill() that uses malloc/free in case > of n > PL_MAXPOLY-1. I've used a behavior similar to the function > plP_plfclp() in plfill.c. plP_plfclp tests if the number of point is > greater than PL_MAXPOLY and uses static or dynamic array in > consequence. I've done the same for c_plfill() and c_plfill3(), so for > polygons with small number of vertices no overload is done for the > using of malloc/free. This only introduces two if's sentences: for > test if malloc must be used and for tests if free must be used. And > deletes a if sentence: the check if ( n < PL_MAXPOLY ) in the test for > determining if the last point is the same as the initial in tle > polyline defining the polygon.
Have you surveyed the rest of the code base for other places that need changing? I see drivers/xwin.c and src/plbuf.c for sure. Else code will go boom. -- Maurice LeBrun ------------------------------------------------------------------------------ Learn how Oracle Real Application Clusters (RAC) One Node allows customers to consolidate database storage, standardize their database environment, and, should the need arise, upgrade to a full multi-node Oracle RAC database without downtime or disruption http://p.sf.net/sfu/oracle-sfdevnl _______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
