diff options
| author | Nicholas Clark <nick@ccl4.org> | 2009-08-21 09:40:03 +0100 |
|---|---|---|
| committer | Nicholas Clark <nick@ccl4.org> | 2009-08-21 11:04:57 +0100 |
| commit | a18d9f20d1de8843127a2b32b9d608c7d2a1b7b9 (patch) | |
| tree | 57afdf7c3d4f20bf96c37b834274e25721ac0337 /ext/mro | |
| parent | b3ed409dee4fd31952bdcb9fa77d0e0c4e4516ab (diff) | |
| download | perl-a18d9f20d1de8843127a2b32b9d608c7d2a1b7b9.tar.gz | |
In mro_get_linear_isa_c3() optimise even if AvARRAY(isa_lin) is NULL.
It can only be NULL if AvFILLp(isa_lin) is -1, and the code in the fast path
block will work cleanly in that case too.
Diffstat (limited to 'ext/mro')
| -rw-r--r-- | ext/mro/mro.xs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ext/mro/mro.xs b/ext/mro/mro.xs index d9451b6b8f..92be620787 100644 --- a/ext/mro/mro.xs +++ b/ext/mro/mro.xs @@ -91,7 +91,7 @@ S_mro_get_linear_isa_c3(pTHX_ HV* stash, U32 level) AV* const isa_lin = S_mro_get_linear_isa_c3(aTHX_ isa_item_stash, level + 1); - if(items == 0 && AvFILLp(seqs) == -1 && AvARRAY(isa_lin)) { + if(items == 0 && AvFILLp(seqs) == -1) { /* Only one parent class. For this case, the C3 linearisation is this class followed by the parent's inearisation, so don't bother with the expensive |