Skip Menu |
 

This queue is for tickets about the Imager CPAN distribution.

Report information
The Basics
Id: 69243
Status: resolved
Priority: 80/
Queue: Imager

People
Owner: Nobody in particular
Requestors: TONYC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 0.84
Fixed in: (no value)



Subject: clean up the old crufty XS
Date: Sat, 2 Jul 2011 13:17:38 +1000
To: bug-Imager [...] rt.cpan.org
From: tonyc [...] cpan.org
Download (untitled) / with headers
text/plain 246b
In some cases we do our own AV type checks, or check the number of parameters, eg in i_nearest_color() where we could simply specify a suitable type and let XS do the work. A general cleanup pass to make that code more readable would be useful.
Download (untitled) / with headers
text/plain 315b
On Fri Jul 01 23:17:47 2011, TONYC wrote: Show quoted text
> In some cases we do our own AV type checks, or check the number of > parameters, eg in i_nearest_color() where we could simply specify a > suitable type and let XS do the work. > > A general cleanup pass to make that code more readable would be > useful.
Fixed in 0.97


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.