Summary: | mutable pointers from const methods, timeContainer() | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jeongeun Kim <je_julie.kim> | ||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ap, commit-queue, dbates, d-r, fmalita, gyuyoung.kim, pdr, rniwa, schenney, sergio | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Jeongeun Kim
2014-07-09 03:53:33 PDT
Created attachment 234625 [details]
Patch
After I discussed const method with Daniel Bates on IRC, I've checked timeContainer() const method. I think that API doesn't need to be const method. Please take a look. Comment on attachment 234625 [details] Patch Clearing flags on attachment: 234625 Committed r171144: <http://trac.webkit.org/changeset/171144> All reviewed patches have been landed. Closing bug. This commit has added svn:executable bit to several files. I removed it in <http://trac.webkit.org/r171147>. Any idea how it got there? It was not in the patch, and the patch was landed by commit queue. Did someone break commit queue recently? Actually, it was in the patch:
> old mode 100644
> new mode 100755
Sorry for the inconvenience. I checked it with ap on IRC. I'll check patch with that mode information next time. Thanks, |