Bug 70371 - DRT should always print a newline after #EOF
Summary: DRT should always print a newline after #EOF
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Seidel (no email)
URL:
Keywords:
Depends on:
Blocks: nrwt-wk2
  Show dependency treegraph
 
Reported: 2011-10-18 15:02 PDT by Eric Seidel (no email)
Modified: 2011-10-19 12:13 PDT (History)
2 users (show)

See Also:


Attachments
Patch (6.48 KB, patch)
2011-10-18 15:06 PDT, Eric Seidel (no email)
abarth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2011-10-18 15:02:40 PDT
DRT should always print a newline after #EOF
Comment 1 Eric Seidel (no email) 2011-10-18 15:06:24 PDT
Created attachment 111509 [details]
Patch
Comment 2 Eric Seidel (no email) 2011-10-18 15:13:14 PDT
Important to let this land through the commit-queue to make sure I didn't screw anything up in my testing.
Comment 3 Dirk Pranke 2011-10-18 15:18:49 PDT
this change looks fine. Note that we don't enforce that the #EOF start at the beginning of the line either (specifically, in the case of audio tests where the main output is binary data, not text). Does it make sense to change that as well?
Comment 4 Eric Seidel (no email) 2011-10-18 15:30:15 PDT
The code seems mixed about whether #EOF is on its own line or not.  Since adding a \n before #EOF might add a \n to some test results, I decided to avoid it for now.  You're right that it's possible we're silently ignoring some content in ORWT due to lack of requiring #EOF to be on a line by itself.
Comment 5 Eric Seidel (no email) 2011-10-18 15:32:00 PDT
Comment on attachment 111509 [details]
Patch

re-testing ORWT, I'm seeing failure.  Checking...
Comment 6 Eric Seidel (no email) 2011-10-19 12:13:20 PDT
This is invalid.  I was just confused.  puts always prints a newline after the string.  This change caused us to print double-newlines.

The opposite change is probably correct to remove the \n from all the places where we currently have "#EOF\n".