On Wed, 1 May 2013 16:32:07 +0200, demerphq <demerphq@gmail.com> wrote: > sub string_as_hex { > my $str= shift; > if (utf8::is_utf8($str)) { > return unpack "H*", Encode::decode_utf8($str); > } else { > return unpack "H*", $str; > } > } pet-peeve: NO ELSE AFTER RETURN/DIE sub string_as_hex { my $str= shift; if (utf8::is_utf8($str)) { return unpack "H*", Encode::decode_utf8($str); } return unpack "H*", $str; } or shorter sub string_as_hex { my $str= shift; utf8::is_utf8 ($str) and return unpack "H*", Encode::decode_utf8 ($str); return unpack "H*", $str; } or even sub string_as_hex { my $str= shift; return unpack "H*", utf8::is_utf8 ($str) ? Encode::decode_utf8 ($str) : $str; } -- H.Merijn Brand http://tux.nl Perl Monger http://amsterdam.pm.org/ using perl5.00307 .. 5.17 porting perl5 on HP-UX, AIX, and openSUSE http://mirrors.develooper.com/hpux/ http://www.test-smoke.org/ http://qa.perl.org http://www.goldmark.org/jeff/stupid-disclaimers/Thread Previous | Thread Next