From: Nicholas Clark <nick@ccl4.org>
Date: Sat, 17 Sep 2005 15:32:36 +0000 (+0000)
Subject: add merge Perl_sv_2[inpu]v to TODO
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=e593da1a93b9d7a3f9a5b4d7429f34f9fab60a94;p=p5sagit%2Fp5-mst-13.2.git

add merge Perl_sv_2[inpu]v to TODO

p4raw-id: //depot/perl@25434
---

diff --git a/pod/perltodo.pod b/pod/perltodo.pod
index a4e655c..771740e 100644
--- a/pod/perltodo.pod
+++ b/pod/perltodo.pod
@@ -264,7 +264,6 @@ Clean these up. Move them to mathom.c, and don't compile for blead?
 
 
 
-
 =head1 Tasks that need a knowledge of XS
 
 These tasks would need C knowledge, and roughly the level of knowledge of
@@ -275,6 +274,16 @@ C.
 
 Clean this up. Check everything in core works
 
+=head2 merge Perl_sv_2[inpu]v
+
+There's a lot of code shared between C<Perl_sv_2iv_flags>,
+C<Perl_sv_2uv_flags>, C<Perl_sv_2nv>, and C<Perl_sv_2pv_flags>. It would be
+interesting to see if some of it can be merged into common shared static
+functions. In particular, C<Perl_sv_2uv_flags> started out as a cut&paste
+from C<Perl_sv_2iv_flags> around 5.005_50 time, and it may be possible to
+replace both with a single function that returns a value or union which is
+split out by the macros in F<sv.h>
+
 =head2 UTF8 caching code
 
 The string position/offset cache is not optional. It should be.