Bug 3945 - [PATCH] Safe merges of comments and other trivialities from KDE's kjs
Summary: [PATCH] Safe merges of comments and other trivialities from KDE's kjs
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 420+
Hardware: All All
: P2 Trivial
Assignee: Maciej Stachowiak
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-10 13:47 PDT by Martijn Klingens
Modified: 2005-07-14 11:41 PDT (History)
0 users

See Also:


Attachments
GCC namespace fix (9.45 KB, patch)
2005-07-10 13:47 PDT, Martijn Klingens
mjs: review+
Details | Formatted Diff | Diff
Comment merges (57.93 KB, patch)
2005-07-10 13:48 PDT, Martijn Klingens
mjs: review-
Details | Formatted Diff | Diff
Comment patch without removing 'int index' :) (57.91 KB, patch)
2005-07-13 14:23 PDT, Martijn Klingens
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martijn Klingens 2005-07-10 13:47:11 PDT
Hiya, 
 
I'll attach two patches to this bugzilla item that together reduce the diff 
between JSC's kjs and khtml's kjs with over a 1000 lines. 
 
The first part merges namespace fixes for more recent GCC versions (namespaces 
should end with just } and not }; and recent GCC is more picky about that). 
 
The second diff merges many comments in the code that apply to both trees. 
Think about the change of the FSF's postal address and changes and typo fixes 
in the API docs. I've been deliberately conservative with merging and aimed 
for big reductions first before going over the diff more closely to extract 
smaller hunks. Comments that I left out on purpose include copyright changes 
on either side (those should be merged with the code that the copyright refers 
to and not on theirselves) and some docs for classes that are no longer 
sharing the same API in both trees. 
 
Please commit to JSC as it makes the life of those comparing the trees a lot 
easier... 
 
Proposed changelog entries for the diffs: 
 
1. Merge GCC namespace termination (s/};/}/) from KHTML's kjs 
 
2. Merge API docs and the FSF postall address change from KHTML's kjs
Comment 1 Martijn Klingens 2005-07-10 13:47:54 PDT
Created attachment 2902 [details]
GCC namespace fix
Comment 2 Martijn Klingens 2005-07-10 13:48:44 PDT
Created attachment 2903 [details]
Comment merges
Comment 3 Martijn Klingens 2005-07-11 12:00:47 PDT
I just noticed a small issue: 
 
http://bugzilla.opendarwin.org/attachment.cgi?id=2903&action=diff#property_map.h_sec4 
 
That one removed line is NOT correct, when applying the patch please readd 
this single line. 
 
Comment 4 Maciej Stachowiak 2005-07-13 11:44:59 PDT
r=me on the namespace patch
Comment 5 Maciej Stachowiak 2005-07-13 11:48:26 PDT
Comment on attachment 2903 [details]
Comment merges

please repost this patch without the accidental removal of index
Comment 6 Maciej Stachowiak 2005-07-13 11:52:12 PDT
Note, only the namespace fix is approved, not the comment fix (unless committer fees like undoing the 
accidental line removal).
Comment 7 Martijn Klingens 2005-07-13 14:23:52 PDT
Created attachment 2948 [details]
Comment patch without removing 'int index' :)

Hmm, if I should have known you'd be picky on the patch and have recreated the
patch right away... so much for being lazy :)

Anyway, new patch as replacement, once it's in I'll try to merge more trivial
stuff.

Thanks so far. Note though that you'll probably need to commit for me, I don't
think I have such rights myself.
Comment 8 Geoffrey Garen 2005-07-14 11:36:20 PDT
Comment on attachment 2948 [details]
Comment patch without removing 'int index' :)

I'm marking comments.diff review+, since it fixes the int problem.