Tim Ruehsen <[email protected]> writes: > On Thursday 02 October 2014 13:07:09 Giuseppe Scrivano wrote: >> Tim Ruehsen <[email protected]> writes: >> > -my $the_test = HTTPTest->new (name => >> > "Test--spider-r-HTTP-Content-Disposition", +my $the_test = HTTPTest->new >> > ( >> > >> > input => \%urls, >> > cmdline => $cmdline, >> > errcode => $expected_error_code, >> >> I think it is a good idea and that we should just do it. The only minor >> issue that I see is "my $the_test = HTTPTest->new (" on a line by >> itself, we should drop the new line and have "input =>" on the same >> line, like: >> >> my $the_test = HTTPTest->new (input => \%urls, >> >> this pattern repeats for all the tests. > > Ok, finally found a sed pattern: > sed -i ':x; /HTTPTest->new ($/ { N; s/\n[[:space:]]*//; tx }' *.px
thanks for sharing it. It is always good to learn new sed tricks. > Here is the amended patch. I am going to push it soon. Regards, Giuseppe
