-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simplifying color setup #1320
Simplifying color setup #1320
Conversation
Use the _fontColor object's CGColor property to setup the context rather than making a new CGColor object and then deleting it. NSMutableAttributedStringFixPlatformSpecificAttributes will also use the _fontColor CGColor so this change doesn't create an object that wasn't going to be created anyway, and we get some reuse.
I can't find a way to merge it to other branch (develop-v3.5). Can somebody advice how to merge PR into other target branch. Or can @richardgroves change target branch himself? |
Will take a look at re-targeting the PR to the 3.5 branch |
Ok - ~1400 comments asking for GitHub to implement changing the branch - isaacs/github#18 - but it still can't be done. Will repurpose the ~5 line change onto the new branch. |
Well, I can merge it manually too. it's not about my lazyness, it's about keeping things git (is it a phrase at all?). |
It's OK - doing a new PR and will reference this one for completeness. |
Repeat of PR in cocos2d#1320 that got lost due branch changes and GitHub limitations.
Redone in #1365 |
Repeat of PR in #1320 that got lost due branch changes and GitHub limitations.
Use the _fontColor object's CGColor property to setup the context rather than making a new CGColor object and then deleting it. NSMutableAttributedStringFixPlatformSpecificAttributes will also use the _fontColor CGColor so this change doesn't create an object that wasn't going to be created anyway, and we get some reuse.