Skip to content

Various performance improvements.#60

Merged
AliSoftware merged 5 commits intoAliSoftware:masterfrom
jparise:speed
Aug 15, 2012
Merged

Various performance improvements.#60
AliSoftware merged 5 commits intoAliSoftware:masterfrom
jparise:speed

Conversation

@jparise
Copy link
Copy Markdown
Contributor

@jparise jparise commented Jan 20, 2012

  • Defer customLinks allocation.
  • Avoid copying strings in attributedTextWithLinks.
  • Cache the result of the link color selector check.

Please see the individual commits for descriptions.

Previously, an NSMutableArray was always being allocated, even if the
label ultimately had no custom links. This defers the array allocation
until it's actually needed.
We don't need to make a local mutable (and autoreleased) copy of our
attributed string in attributedTextWithLinks. Instead, we can operate
directly on our _attributedText ivar.

This change also adds some early-out conditions that let us avoid the
bulk of the processing code when we don't have any links to set.
Previously, we were checking for the selector on every enumeration
block callback.
AliSoftware added a commit that referenced this pull request Aug 15, 2012
Various performance improvements.
@AliSoftware AliSoftware merged commit ac58cb1 into AliSoftware:master Aug 15, 2012
stegiorgi pushed a commit to wedoit-io/OHAttributedLabel that referenced this pull request Jan 21, 2014
- MBProgressHUD now requires LLVM 3+

- instance variables moved to the class implementation or removed (using synthesized instance vars)
- using KVO instead of custom accessors
- completely rewritten layout and UI buildup code
- more concise arc conditionals
- added MBProgressHUDModeText (+ example)
- various documentation fixes
- migrated to @autoreleasepool
- general source code cleanup and restructuring
- removed deprecated delegate callback
- version bump to 5.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants