Re: SCS2CSS

[chromatic]

in reply to SCS2CSS

Care for a couple small suggestions?

my $argc = @ARGV;

if($argc != 1) {
 print "Usage: \$ $0 <scsfile>\n";
 exit;
}

my $scs = $ARGV[0];

This could be more succinct as:

my $scs = shift or die "Usage: $0 <scsfile>\n";

Don't forget to check the success of your open calls.

You might want to escape the periods and anchor the ends of your regular expression. File::Basename might also come in handy, but may be severe overkill:

(my $css = $scs) =~ s/\.scs\z/.css/;

A heredoc would make the print statements somewhat more manageable. Roll on Perl 6 and intendable heredocs!

[staeryatz]

Thanks for the tips. :)

I never would have thought of putting the argument vector processing all in one line like that. I kinda just did it the 'C' way. Kudos to you.

Ah, the heredoc operator...I don't know why I haven't thought of that one myself (I use it all the time for cgi). But the strangest thing of all, is why did I put my print staements in parenthesis?

The mysteries of programming on \0 caffeine. This could be an X-File...

Update: Okay, I finally changed it and tested it. It looks alot better now, thanks.