[Webkit-unassigned] [Bug 29024] Make JavaScriptCore compile on platforms with case-insensitive file system

[bugzilla-daemon at webkit.org]

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


Mark Rowe (bdash) <mrowe at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #39175|review?                     |review-
               Flag|                            |




--- Comment #2 from Mark Rowe (bdash) <mrowe at apple.com>  2009-09-08 01:55:22 PDT ---
(From update of attachment 39175)
When renaming a file like this you should update all references to the file so
that you don't break the build.

A few other points:
1) Includes should be kept in alphabetical order.
2) The JS prefix is typically reserved for objects that represent an object
within the JavaScript environment (JSArray, JSString, etc).  Maybe this case is
ok for the same reason that JSType is ok.
3) Classes live inside files of the same name.  Since you're renaming the file
I suspect that you should rename the class to match.
4) You should explicit about which platform this is intended to fix.  It's
quite clear that JavaScriptCore compiles on Mac OS X and Windows, both of which
use case-insensitive file systems.  If roll out this change in the future and
only test on those platforms, we'll break whatever platform it is you're trying
to fix here.

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