Perl / perl5

🐪 The Perl programming language
https://dev.perl.org/perl5/
Other
1.95k stars 554 forks source link

sv.c: Remove redundant `PERL_ABS` in S_hextract. #22624

Closed t-a-k closed 3 weeks ago

t-a-k commented 3 weeks ago

It seems redundant to take the absolute value of the first argument of frexp() here, because the sign of the first argument will only affect its return value which is discarded in this case.

This change should not change the behavior of perl (will just slightly save the code size and CPU cycles), so I think that: