Closed
Bug 46577
Opened 24 years ago
Closed 24 years ago
Perl errors in TestOutSinks.pl
Categories
(Core :: DOM: HTML Parser, defect, P3)
Core
DOM: HTML Parser
Tracking
()
VERIFIED
FIXED
M18
People
(Reporter: rufus-mozilla, Assigned: akkzilla)
References
()
Details
Well, it's kinda obvious that whoever wrote this was thinking C instead of Perl. Here are the errors: Line 46: print "xxxxxxxx Status = " + $status + "\n"; should be: print "xxxxxxxx Status = " . $status . "\n"; also Line 60: $status = system("# TestOutput -i text/html -o text/plain -f 32 -c OutTestData/plainwrap.out OutTestData/plain.html"); should be: # $status = system("TestOutput -i text/html -o text/plain -f 32 -c OutTestData/plainwrap.out OutTestData/plain.html");
Assignee | ||
Comment 2•24 years ago
|
||
Yes, this is mine. I was thinking JS -- that would do something even more unexpected in C. :-) Thanks for noticing and offering the fix!
Status: NEW → ASSIGNED
Updated•24 years ago
|
Keywords: correctness
Target Milestone: --- → M19
Comment 3•24 years ago
|
||
putting to m19, this is a testscript fix
Assignee | ||
Comment 4•24 years ago
|
||
Fixed -- thanks again!
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•24 years ago
|
||
Sorry for the spam! But apparently all these closed bugs need to have their target milestones changed since M19 and M20 are going away. Since they're allready closed, I'm choosing M18.
Target Milestone: M19 → M18
Marking verified as per above developer comments.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•