diff options
| author | Nicholas Clark <nick@ccl4.org> | 2005-09-17 18:59:37 +0000 |
|---|---|---|
| committer | Nicholas Clark <nick@ccl4.org> | 2005-09-17 18:59:37 +0000 |
| commit | 4b3e5994fbd61011a44dadaca92df3f0f91128e2 (patch) | |
| tree | 5e5675452af3e77fb39d6b60c4e2284c4b047590 /pod/perltodo.pod | |
| parent | ad0fa5c22bd271745e4d548ad08b2115e6549a83 (diff) | |
| download | perl-4b3e5994fbd61011a44dadaca92df3f0f91128e2.tar.gz | |
Add reduce duplication in sv_setsv_flags to TODO
p4raw-id: //depot/perl@25446
Diffstat (limited to 'pod/perltodo.pod')
| -rw-r--r-- | pod/perltodo.pod | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/pod/perltodo.pod b/pod/perltodo.pod index 4fec644ccb..3e925e35c4 100644 --- a/pod/perltodo.pod +++ b/pod/perltodo.pod @@ -320,6 +320,18 @@ necessary to do something like although C<long double> might not be the only type to add to the padding union. +=head2 reduce duplication in sv_setsv_flags + +C<Perl_sv_setsv_flags> has a comment +C</* There's a lot of redundancy below but we're going for speed here */> + +Whilst this was true 10 years ago, the growing disparity between RAM and CPU +speeds mean that the trade offs have changed. In addition, the duplicate code +adds to the maintenance burden. It would be good to see how much of the +redundancy can be pruned, particular in the less common paths. (Profiling +tools at the ready...). For example, why does the test for +"Can't redefine active sort subroutine" need to occur in two places? + |