This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
add merge Perl_sv_2[inpu]v to TODO
authorNicholas Clark <nick@ccl4.org>
Sat, 17 Sep 2005 15:32:36 +0000 (15:32 +0000)
committerNicholas Clark <nick@ccl4.org>
Sat, 17 Sep 2005 15:32:36 +0000 (15:32 +0000)
p4raw-id: //depot/perl@25434

pod/perltodo.pod

index a4e655c..771740e 100644 (file)
@@ -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.