diff options
| author | thakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-15 17:34:59 +0000 |
|---|---|---|
| committer | thakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-15 17:34:59 +0000 |
| commit | a866cc21b77be5881eef0039db0b5928587f0b54 (patch) | |
| tree | 83c0375c7d2b1c48adb92cf41f981f5f84273d94 /app/clipboard | |
| parent | e1df35e117fd5e243874bf9db9224b7112e51b34 (diff) | |
| download | chromium_src-a866cc21b77be5881eef0039db0b5928587f0b54.zip chromium_src-a866cc21b77be5881eef0039db0b5928587f0b54.tar.gz chromium_src-a866cc21b77be5881eef0039db0b5928587f0b54.tar.bz2 | |
Mac/clang: Possibly contentious changes.
As discussed on irc, we're getting rid of const NSObjects, and we're making all properties nonatomic.
const: All of cocoa takes nonconst NSObjects, and passing a const NSString to a function that takes NSString is a const violation. gcc doesn't complain about this, but clang intentionally does. constness is usually done via immutable base classes and mutable subclasses in cocoa anyway, so getting rid of const isn't that bad. Also, we don't really have a choice.
nonatomic: Some of our classes have custom setters that are not @synchronized. If the @property for that is not non-atomic, then the interface claims that the method is synchronized but the implementation actually isn't. That's a bug. gcc happens not to complain about this, but clang does. Since we shouldn't need atomic properties anywhere, the simple rule is now to just make all properties nonatomic.
Review URL: http://codereview.chromium.org/2769014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49808 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'app/clipboard')
| -rw-r--r-- | app/clipboard/clipboard_mac.mm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/clipboard/clipboard_mac.mm b/app/clipboard/clipboard_mac.mm index 7ae3a1b..0fa5d8f 100644 --- a/app/clipboard/clipboard_mac.mm +++ b/app/clipboard/clipboard_mac.mm @@ -19,11 +19,11 @@ namespace { // Would be nice if this were in UTCoreTypes.h, but it isn't -const NSString* kUTTypeURLName = @"public.url-name"; +NSString* const kUTTypeURLName = @"public.url-name"; // Tells us if WebKit was the last to write to the pasteboard. There's no // actual data associated with this type. -const NSString *kWebSmartPastePboardType = @"NeXT smart paste pasteboard type"; +NSString* const kWebSmartPastePboardType = @"NeXT smart paste pasteboard type"; NSPasteboard* GetPasteboard() { // The pasteboard should not be nil in a UI session, but this handy DCHECK |