matplotlib-checkins

SF.net SVN: matplotlib:[8592] branches/v1_0_maint/src

[<md...@us...>]

Revision: 8592
          http://matplotlib.svn.sourceforge.net/matplotlib/?rev=8592&view=rev
Author:   mdboom
Date:     2010-07-29 14:09:11 +0000 (Thu, 29 Jul 2010)

Log Message:
-----------
Remove points_to_pixels_snapto -- since it isn't used anywhere and is now actually incomplete wrt the new stroke-width-aware snapping.

Modified Paths:
--------------
    branches/v1_0_maint/src/_backend_agg.cpp
    branches/v1_0_maint/src/_backend_agg.h

Modified: branches/v1_0_maint/src/_backend_agg.cpp
===================================================================
--- branches/v1_0_maint/src/_backend_agg.cpp	2010-07-28 19:21:59 UTC (rev 8591)
+++ branches/v1_0_maint/src/_backend_agg.cpp	2010-07-29 14:09:11 UTC (rev 8592)
@@ -2245,18 +2245,6 @@
 
 
 double
-RendererAgg::points_to_pixels_snapto(const Py::Object& points)
-{
-    // convert a value in points to pixels depending on renderer dpi and
-    // screen pixels per inch
-    // snap return pixels to grid
-    _VERBOSE("RendererAgg::points_to_pixels_snapto");
-    double p = Py::Float(points) ;
-    return (int)(p*dpi / 72.0) + 0.5;
-}
-
-
-double
 RendererAgg::points_to_pixels(const Py::Object& points)
 {
     _VERBOSE("RendererAgg::points_to_pixels");

Modified: branches/v1_0_maint/src/_backend_agg.h
===================================================================
--- branches/v1_0_maint/src/_backend_agg.h	2010-07-28 19:21:59 UTC (rev 8591)
+++ branches/v1_0_maint/src/_backend_agg.h	2010-07-29 14:09:11 UTC (rev 8592)
@@ -230,7 +230,6 @@
 
 protected:
     double points_to_pixels(const Py::Object& points);
-    double points_to_pixels_snapto(const Py::Object& points);
     agg::rgba rgb_to_color(const Py::SeqBase<Py::Object>& rgb, double alpha);
     facepair_t _get_rgba_face(const Py::Object& rgbFace, double alpha);
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

SF.net SVN: matplotlib:[8708] branches/v1_0_maint/src

[<ef...@us...>]

Revision: 8708
          http://matplotlib.svn.sourceforge.net/matplotlib/?rev=8708&view=rev
Author:   efiring
Date:     2010-09-17 06:41:34 +0000 (Fri, 17 Sep 2010)

Log Message:
-----------
including Python.h and png.h: a second try, special case for linux

Modified Paths:
--------------
    branches/v1_0_maint/src/_backend_agg.cpp
    branches/v1_0_maint/src/_png.cpp

Modified: branches/v1_0_maint/src/_backend_agg.cpp
===================================================================
--- branches/v1_0_maint/src/_backend_agg.cpp	2010-09-17 02:52:46 UTC (rev 8707)
+++ branches/v1_0_maint/src/_backend_agg.cpp	2010-09-17 06:41:34 UTC (rev 8708)
@@ -4,9 +4,6 @@
 /* Python API mandates Python.h is included *first* */
 #include "Python.h"
 
-#define PNG_SKIP_SETJMP_CHECK
-#include <png.h>
-
 #include "ft2font.h"
 #include "_image.h"
 #include "_backend_agg.h"

Modified: branches/v1_0_maint/src/_png.cpp
===================================================================
--- branches/v1_0_maint/src/_png.cpp	2010-09-17 02:52:46 UTC (rev 8707)
+++ branches/v1_0_maint/src/_png.cpp	2010-09-17 06:41:34 UTC (rev 8708)
@@ -1,8 +1,26 @@
+
+/* For linux, png.h must be imported before Python.h because
+   png.h needs to be the one to define setjmp.
+   Undefining _POSIX_C_SOURCE and _XOPEN_SOURCE stops a couple
+   of harmless warnings.
+*/
+
+#ifdef __linux__
+#   include <png.h>
+#   ifdef _POSIX_C_SOURCE
+#       undef _POSIX_C_SOURCE
+#   endif
+#   ifdef _XOPEN_SOURCE
+#       undef _XOPEN_SOURCE
+#   endif
+#   include "Python.h"
+#else
+
 /* Python API mandates Python.h is included *first* */
-#include "Python.h"
+#   include "Python.h"
 
-#define PNG_SKIP_SETJMP_CHECK
-#include <png.h>
+#   include <png.h>
+#endif
 
 // TODO: Un CXX-ify this module
 #include "CXX/Extensions.hxx"


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.