[Webkit-unassigned] [Bug 30572] Unification of using null device in perl scripts

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=30572


Csaba Osztrogonac <ossy at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |eric at webkit.org




--- Comment #4 from Csaba Osztrogonac <ossy at webkit.org>  2009-10-22 14:23:45 PDT ---
(In reply to comment #2)
> I believe our style is to put spaces around '.' operators.
Yes, you're right, sorry. I will correct it.

> Also, I'm surprised that sometimes you use local variables to hold $devNull
> and sometimes you make direct calls to File::Spec->devnull().
I told I prefer using File::Spec->devnull() everywhere, but I have no idea how
could I use it directly in `...`. Accordingly I use local variable same as in
svn-create-patch originally. Have you got better idea to solve this problem?

> Is devnull() always a string path?
You can find devNull() here: http://perldoc.perl.org/File/Spec.html
"Returns a string representation of the null device."
It works correctly, we used it previosly in WebKit scripts.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.