Skip Menu |

This queue is for tickets about the Syntax-Highlight-Shell CPAN distribution.

Report information
The Basics
Id: 7983
Status: resolved
Priority: 0/
Queue: Syntax-Highlight-Shell

People
Owner: SAPER [...] cpan.org
Requestors: allenday [...] ucla.edu
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.01
Fixed in: (no value)



Subject: <pre></pre> output even when disabled
Syntax::Highlighter::Shell->new(pre=>0); ... still outputs <pre>...</pre> tags. on closer inspection the constructor argument handline tests settting of arguments based on their truth status rather than if they are defined. it would be better to do: $self->{myslot} = $arg{myslot} if defined($arg{myslot}); than: $self->{myslot} = $arg{myslot} if $arg{myslot}; looking forward to seeing this module in action!
You're perfectly right. I always get caught in this old Perl trick. This is corrected for the next release. Thanks for reporting this bug.