[rt #69158] *_has_chars() now returns PL_sv_yes/PL_sv_no
authorTony Cook <tony@develop-help.com>
Fri, 14 Oct 2011 12:33:48 +0000 (23:33 +1100)
committerTony Cook <tony@develop-help.com>
Fri, 14 Oct 2011 12:33:48 +0000 (23:33 +1100)
which should be faster than making lots of IV SVs.

Changes
FT2/Changes
FT2/FT2.xs
Imager.xs
T1/Changes
T1/T1.xs

diff --git a/Changes b/Changes
index 57576c2..1e12836 100644 (file)
--- a/Changes
+++ b/Changes
@@ -41,6 +41,11 @@ Bug fixes:
  - Imager::Font->new now produces better error messages for the T1
    engine.
 
+ - the font has_chars() method now returns perl's true and false
+   values in list context rather than integers, which should be more
+   efficient.
+   https://rt.cpan.org/Ticket/Display.html?id=69158
+
 Imager 0.85_01 - 10 Oct 2011
 ==============
 
index 3fb1d73..89d3a5e 100644 (file)
@@ -1,3 +1,10 @@
+Imager-Font-FT2 0.83
+====================
+
+ - has_chars() now returns perl's true and false values in list
+   context rather than integers, which should be more efficient.
+   https://rt.cpan.org/Ticket/Display.html?id=69158
+
 Imager-Font-FT2 0.82
 ====================
 
index 97c037d..ac8396e 100644 (file)
@@ -228,7 +228,7 @@ i_ft2_has_chars(handle, text_sv, utf8)
         if (GIMME_V == G_ARRAY) {
           EXTEND(SP, count);
           for (i = 0; i < count; ++i) {
-            PUSHs(sv_2mortal(newSViv(work[i])));
+            PUSHs(boolSV(work[i]));
           }
         }
         else {
index 5624115..b0474ac 100644 (file)
--- a/Imager.xs
+++ b/Imager.xs
@@ -2209,7 +2209,7 @@ i_tt_has_chars(handle, text_sv, utf8)
         if (GIMME_V == G_ARRAY) {
           EXTEND(SP, count);
           for (i = 0; i < count; ++i) {
-            PUSHs(sv_2mortal(newSViv(work[i])));
+            PUSHs(boolSV(work[i]));
           }
         }
         else {
index 9180239..5e6e5d9 100644 (file)
@@ -10,6 +10,10 @@ Imager::Font::T1 1.015
  - Imager::Font->new now produces better error messages for the T1
    engine.
 
+ - has_chars() now returns perl's true and false values in list
+   context rather than integers, which should be more efficient.
+   https://rt.cpan.org/Ticket/Display.html?id=69158
+
 Imager::Font::T1 1.014
 ======================
 
index 659b01d..f587fb0 100644 (file)
--- a/T1/T1.xs
+++ b/T1/T1.xs
@@ -134,7 +134,7 @@ i_t1_has_chars(handle, text_sv, utf8 = 0)
         if (GIMME_V == G_ARRAY) {
           EXTEND(SP, count);
           for (i = 0; i < count; ++i) {
-            PUSHs(sv_2mortal(newSViv(work[i])));
+            PUSHs(boolSV(work[i]));
           }
         }
         else {