Bug 37287 – Rename Wince file to WinCE

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Template Framework

[Patrick R. Gansterer]

See patch.

[Patrick R. Gansterer]

Created attachment 52892 [details]
The patch - part 1 (JSC/WTF)

[WebKit Review Bot]

Attachment 52892 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
Last 3072 characters of output:
 is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.h:72:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.h:211:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.h:213:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.cpp:97:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.cpp:121:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
WARNING: Skipping non-existent file: JavaScriptCore/wtf/unicode/wince/UnicodeWince.h
JavaScriptCore/wtf/wince/FastMallocWinCE.h:22:  #ifndef header guard has wrong style, please use: FastMallocWinCE_h  [build/header_guard] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:29:  Alphabetical sorting problem.  [build/include_order] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:34:  n_elements is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:34:  element_size is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:41:  n_elements is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:41:  element_size is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:83:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:86:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:87:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:88:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:89:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:90:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:91:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:92:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:93:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:156:  One space before end of line comments  [whitespace/comments] [5]
WARNING: Skipping non-existent file: JavaScriptCore/wtf/unicode/wince/UnicodeWince.cpp
Total errors found: 25 in 5 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Patrick R. Gansterer]

Created attachment 52902 [details]
The patch - part 2 (WebCore)

It is very big, but there are no real changes in this patch.

[WebKit Review Bot]

Attachment 52902 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
Last 3072 characters of output:
/null] [5]
WebCore/platform/graphics/wince/GraphicsContextWinCE.cpp:1922:  Use 0 instead of NULL.  [readability/null] [5]
WARNING: Skipping non-existent file: WebCore/storage/wince/LocalStorageThreadWince.cpp
WebCore/platform/wince/MIMETypeRegistryWinCE.cpp:31:  Alphabetical sorting problem.  [build/include_order] [4]
WebCore/platform/wince/MIMETypeRegistryWinCE.cpp:61:  Should have a space between // and comment  [whitespace/comments] [4]
WebCore/platform/wince/PasteboardWinCE.cpp:239:  An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
WebCore/platform/wince/PasteboardWinCE.cpp:251:  An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
WebCore/platform/wince/PasteboardWinCE.cpp:273:  cf_html is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
WARNING: Skipping non-existent file: WebCore/platform/wince/SearchPopupMenuWince.cpp
WARNING: Skipping non-existent file: WebCore/storage/wince/DatabaseThreadWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/wince/SharedTimerWince.cpp
WebCore/storage/wince/DatabaseThreadWinCE.h:32:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
WebCore/platform/wince/FileSystemWinCE.cpp:38:  Alphabetical sorting problem.  [build/include_order] [4]
WebCore/platform/wince/FileSystemWinCE.cpp:39:  Alphabetical sorting problem.  [build/include_order] [4]
WebCore/storage/wince/LocalStorageThreadWinCE.h:33:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
WARNING: Skipping non-existent file: WebCore/platform/text/wince/TextBoundariesWince.cpp
WebCore/rendering/RenderThemeWinCE..h:24:  #ifndef header guard has wrong style, please use: RenderThemeWinCE__h  [build/header_guard] [5]
WebCore/rendering/RenderThemeWinCE..h:35:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
WARNING: Skipping non-existent file: WebCore/storage/wince/DatabaseThreadWince.h
WARNING: Skipping non-existent file: WebCore/platform/graphics/wince/PlatformPathWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/graphics/wince/FontCacheWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/graphics/wince/WinceGraphicsExtras.h
WARNING: Skipping non-existent file: WebCore/platform/wince/DragImageWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/graphics/wince/MediaPlayerPrivateWince.h
WARNING: Skipping non-existent file: WebCore/loader/icon/wince/IconDatabaseWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/wince/FileChooserWince.cpp
WebCore/platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp:44:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
WARNING: Skipping non-existent file: WebCore/platform/wince/CursorWince.cpp
Total errors found: 54 in 38 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Patrick R. Gansterer]

Created attachment 52907 [details]
The patch - part 1 (JSC/WTF)

[Patrick R. Gansterer]

Created attachment 52908 [details]
The patch - part 2 (WebCore)

[WebKit Review Bot]

Attachment 52907 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
Last 3072 characters of output:
 is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.h:72:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.h:211:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.h:213:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.cpp:97:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.cpp:121:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
WARNING: Skipping non-existent file: JavaScriptCore/wtf/unicode/wince/UnicodeWince.h
JavaScriptCore/wtf/wince/FastMallocWinCE.h:22:  #ifndef header guard has wrong style, please use: FastMallocWinCE_h  [build/header_guard] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:29:  Alphabetical sorting problem.  [build/include_order] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:34:  n_elements is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:34:  element_size is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:41:  n_elements is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:41:  element_size is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:83:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:86:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:87:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:88:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:89:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:90:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:91:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:92:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:93:  One space before end of line comments  [whitespace/comments] [5]
JavaScriptCore/wtf/wince/FastMallocWinCE.h:156:  One space before end of line comments  [whitespace/comments] [5]
WARNING: Skipping non-existent file: JavaScriptCore/wtf/unicode/wince/UnicodeWince.cpp
Total errors found: 25 in 5 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[WebKit Review Bot]

Attachment 52908 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
Last 3072 characters of output:
/null] [5]
WebCore/platform/graphics/wince/GraphicsContextWinCE.cpp:1922:  Use 0 instead of NULL.  [readability/null] [5]
WARNING: Skipping non-existent file: WebCore/storage/wince/LocalStorageThreadWince.cpp
WebCore/platform/wince/MIMETypeRegistryWinCE.cpp:31:  Alphabetical sorting problem.  [build/include_order] [4]
WebCore/platform/wince/MIMETypeRegistryWinCE.cpp:61:  Should have a space between // and comment  [whitespace/comments] [4]
WebCore/platform/wince/PasteboardWinCE.cpp:239:  An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
WebCore/platform/wince/PasteboardWinCE.cpp:251:  An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
WebCore/platform/wince/PasteboardWinCE.cpp:273:  cf_html is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
WARNING: Skipping non-existent file: WebCore/platform/wince/SearchPopupMenuWince.cpp
WARNING: Skipping non-existent file: WebCore/storage/wince/DatabaseThreadWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/wince/SharedTimerWince.cpp
WebCore/storage/wince/DatabaseThreadWinCE.h:32:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
WebCore/platform/wince/FileSystemWinCE.cpp:38:  Alphabetical sorting problem.  [build/include_order] [4]
WebCore/platform/wince/FileSystemWinCE.cpp:39:  Alphabetical sorting problem.  [build/include_order] [4]
WebCore/storage/wince/LocalStorageThreadWinCE.h:33:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
WARNING: Skipping non-existent file: WebCore/platform/text/wince/TextBoundariesWince.cpp
WebCore/rendering/RenderThemeWinCE..h:24:  #ifndef header guard has wrong style, please use: RenderThemeWinCE__h  [build/header_guard] [5]
WebCore/rendering/RenderThemeWinCE..h:35:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
WARNING: Skipping non-existent file: WebCore/storage/wince/DatabaseThreadWince.h
WARNING: Skipping non-existent file: WebCore/platform/graphics/wince/PlatformPathWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/graphics/wince/FontCacheWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/graphics/wince/WinceGraphicsExtras.h
WARNING: Skipping non-existent file: WebCore/platform/wince/DragImageWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/graphics/wince/MediaPlayerPrivateWince.h
WARNING: Skipping non-existent file: WebCore/loader/icon/wince/IconDatabaseWince.cpp
WARNING: Skipping non-existent file: WebCore/platform/wince/FileChooserWince.cpp
WebCore/platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp:44:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
WARNING: Skipping non-existent file: WebCore/platform/wince/CursorWince.cpp
Total errors found: 54 in 38 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Eric Seidel (no email)]

Style explosion!

[Eric Seidel (no email)]

We have a script for doing these types of renames called "do-webcore-rename".   We might consider using that instead.  Basically we queue up a bunch of renames and do them at once at some later date.

[Patrick R. Gansterer]

(In reply to comment #9)
> Style explosion!
What to do against it? Are style patches welcome?

(In reply to comment #10)
> We have a script for doing these types of renames called "do-webcore-rename".  
> We might consider using that instead.  Basically we queue up a bunch of renames
> and do them at once at some later date.

What should I do now? Post a patch for "do-webcore-rename"?

[WebKit Review Bot]

Attachment 52907 [details] did not build on win:
Build output: http://webkit-commit-queue.appspot.com/results/1574389

[Adam Barth]

> Build output: http://webkit-commit-queue.appspot.com/results/1574389

Sorry, looks like a bug in the win-ews.

[Patrick R. Gansterer]

(In reply to comment #13)
> > Build output: http://webkit-commit-queue.appspot.com/results/1574389
> 
> Sorry, looks like a bug in the win-ews.
I think this a a general problem with subversion on windows, because windows has no case sensitive filesystem. :-/

[Adam Barth]

> I think this a a general problem with subversion on windows, because windows
> has no case sensitive filesystem. :-/

Does that mean this patch is going to cause a problem for all our windows developers?

[Patrick R. Gansterer]

(In reply to comment #15)
> > I think this a a general problem with subversion on windows, because windows
> > has no case sensitive filesystem. :-/
> 
> Does that mean this patch is going to cause a problem for all our windows
> developers?
They won't be able to apply it on their working copy. If someone  apply it on a case sensitive filesystem an commit it, there won't be a problem when checking out on windows.

[Kent Tamura]

(In reply to comment #16)
> (In reply to comment #15)
> > > I think this a a general problem with subversion on windows, because windows
> > > has no case sensitive filesystem. :-/
> > 
> > Does that mean this patch is going to cause a problem for all our windows
> > developers?
> They won't be able to apply it on their working copy. If someone  apply it on a case sensitive filesystem an commit it, there won't be a problem when checking out on windows.

Is it same for Mac?
How about git pull/merge on Windows or Mac?

[Patrick R. Gansterer]

(In reply to comment #17)
> Is it same for Mac?
If Mac as a case-sensitve filesystem ther is no problem.

> How about git pull/merge on Windows or Mac?
It should work with git.
The problem is caused by the ".svn" folder where the "metadata" for each file is stored. Because renaming is a "delete and add" action in SVN it needs a file to store the delete action and one for the add action, but Windows can't store them because of its caseinsensitve filesystem.

[WebKit Commit Bot]

Comment on attachment 52907 [details]
The patch - part 1 (JSC/WTF)

Rejecting patch 52907 from commit-queue.

Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Darin Adler', u'--force']" exit_code: 2
Parsed 11 diffs from patch file(s).
cp: JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.cpp and JavaScriptCore/wtf/unicode/wince/UnicodeWince.cpp are identical (not copied).
Failed to copy JavaScriptCore/wtf/unicode/wince/UnicodeWince.cpp JavaScriptCore/wtf/unicode/wince/UnicodeWinCE.cpp. at /Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply line 416.

Full output: http://webkit-commit-queue.appspot.com/results/3295777

[WebKit Commit Bot]

Comment on attachment 52908 [details]
The patch - part 2 (WebCore)

Rejecting patch 52908 from commit-queue.

Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Darin Adler', u'--force']" exit_code: 2
Parsed 100 diffs from patch file(s).
cp: WebCore/loader/icon/wince/IconDatabaseWinCE.cpp and WebCore/loader/icon/wince/IconDatabaseWince.cpp are identical (not copied).
Failed to copy WebCore/loader/icon/wince/IconDatabaseWince.cpp WebCore/loader/icon/wince/IconDatabaseWinCE.cpp. at /Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply line 416.

Full output: http://webkit-commit-queue.appspot.com/results/3280747

[Patrick R. Gansterer]

I will update the patch to match the current trunk.

[Patrick R. Gansterer]

(In reply to comment #21)
> I will update the patch to match the current trunk.
Sorry, seams to be a problem of the commit queue, not of the patch?

[Eric Seidel (no email)]

The commit queue is running on a case insensitive (but case preserving) file system (HFS+).  It can't land case-renaming patches, no.

[Patrick R. Gansterer]

(In reply to comment #23)
> The commit queue is running on a case insensitive (but case preserving) file system (HFS+).  It can't land case-renaming patches, no.
Who will commit the patch? I'm not a committer by now. ;-)
I can split every patch into a "remove" and a "add again" part, if we can land it with the commit queue then.

[Kwang Yul Seo]

I don't see much value in renaming Wince. It becomes very hard to track commit logs once WinCE files are removed and committed again.

[Patrick R. Gansterer]

(In reply to comment #25)
> I don't see much value in renaming Wince.
It was requested by Darin at bug 32169.

> It becomes very hard to track commit logs once WinCE files are removed and committed again.
If we do a svn-add from a previous revision, this shouldn't be no problem.

[Patrick R. Gansterer]

Comment on attachment 52907 [details]
The patch - part 1 (JSC/WTF)

Clearing flags on attachment: 52907

Manually committed r67788: <http://trac.webkit.org/changeset/67788>

[Patrick R. Gansterer]

Comment on attachment 52908 [details]
The patch - part 2 (WebCore)

Clearing flags on attachment: 52908

Manually committed r67788: <http://trac.webkit.org/changeset/67788>

[Patrick R. Gansterer]

All reviewed patches have been landed.  Closing bug.