Beefy Boxes and Bandwidth Generously Provided by pair Networks
good chemistry is complicated,
and a little bit messy -LW

Re: Script to convert HBA WWNs to lowercase and add ":"

by Not_a_Number (Prior)
on Dec 18, 2011 at 19:21 UTC ( #944193=note: print w/replies, xml ) Need Help??

in reply to Script to convert HBA WWNs to lowercase and add ":"[Updated]

IMO, your script would be more useful if it included some input validation. After the line:

chomp ($wwn=<STDIN>);

I would suggest adding:

die "Input must be 16 characters\n" unless 16 == length $wwn; die "Invalid hex digit '$1'\n" if $wwn =~ /([^[:xdigit:]])/;

Update: changed allowed input length from 10 (where did I get that number from?) to 16, as per WWN spec.

Replies are listed 'Best First'.
Re^2: Script to convert HBA WWNs to lowercase and add ":"
by perl514 (Pilgrim) on Dec 28, 2011 at 17:58 UTC

    Hi Not_a_Number,

    I have put up an updated version of the script. Its got all the checks that you suggested and some more. :)

    Perlpetually Indebted To PerlMonks

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://944193]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others exploiting the Monastery: (10)
As of 2018-07-16 10:57 GMT
Find Nodes?
    Voting Booth?
    It has been suggested to rename Perl 6 in order to boost its marketing potential. Which name would you prefer?

    Results (333 votes). Check out past polls.