In article <20040531152854.79_d0a3b@localhost>, Rafael Garcia-Suarez <rgarciasuarez@mandrakesoft.com> writes: > sub shortmess { # Short-circuit &longmess if called via multiple packages > - { local $@; require Carp::Heavy; } # XXX fix require to not clear $@? > + { > + local $@; > + # XXX fix require to not clear $@? > + require Carp::Heavy unless $INC{"Carp/Heavy.pm"}; > + } Please add a comment for that. A future casual reader of that code will be totally confused by the seemingly pointless unless.Thread Previous