Closed Bug 525956 Opened 12 years ago Closed 12 years ago

White spaces at EOL in nsDisplaylist.cpp


(Core :: Layout, defect)

Not set





(Reporter: MikeK, Assigned: MikeK)



(1 file)

/layout/base/nsDisplayList.cpp contains white spaces at EOL.

These white spaces are preventing me getting a larger patch reviewed.
Attachment #409741 - Flags: review?(roc)
We really don't need a separate bug report for each file.
(In reply to comment #1)
> We really don't need a separate bug report for each file.

Ok - that will make my life much more easy next time - I tried asking this to the reviewer of my main patch several times, but never got an answer, so had to make a choice...  I assume that you still want one patch per file?
If the larger patch you refer to is bug 422540 then these whitespace changes aren't preventing your larger patch from being reviewed. 

My request in that bug was just to not include whitespace changes in the patch containing the GStreamer changes to make it easier to review. 

Providing a GStreamer patch without the whitespace changes is not dependent on getting these other ones reviewed as far as I can see.
No longer blocks: 422540
I'm closing this bug as its debated if it is an issue that should be dealt with, and if this is an acceptable way to deal with it.
Closed: 12 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.