[webkit-reviews] review denied: [Bug 30572] Unification of using null device in perl scripts : [Attachment 41507] proposed patch

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has denied Csaba Osztrogonac <ossy at webkit.org>'s
request for review:
Bug 30572: Unification of using null device in perl scripts
https://bugs.webkit.org/show_bug.cgi?id=30572

Attachment 41507: proposed patch
https://bugs.webkit.org/attachment.cgi?id=41507&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
I believe our style is to put spaces around '.' operators.

Also, I'm surprised that sometimes you use local variables to hold $devNull and
sometimes you make direct calls to File::Spec->devnull().

Is devnull() always a string path?