Skip to content

Dict object improvements. Performance. - #214

Merged
andreasgal merged 4 commits into
mozilla:masterfrom
saebekassebil:master
Jul 8, 2011
Merged

andreasgal merged 4 commits into
mozilla:masterfrom
saebekassebil:master

Conversation

@saebekassebil

Copy link
Copy Markdown
Contributor

So this is the simple changes i made to the Dict object which seems to speed up thing x2 - x5 and improve the code-quality a bit.

Hope this is any use for you guys!

@joneschrisg

Copy link
Copy Markdown
Contributor

This looks fine to me. Andreas?

@saebekassebil

Copy link
Copy Markdown
Contributor Author

Woops, not fine. Minor typo, let me fix that.

@saebekassebil

Copy link
Copy Markdown
Contributor Author

Fixed with latest commit 42cdf6b

@andreasgal

Copy link
Copy Markdown
Contributor

Shaon is working on fixing the glyph issues. Can you change the pull request back to 42cdf6b so you don't collide with him? I want to pull the perf work.

@andreasgal

Copy link
Copy Markdown
Contributor

I will merge and backout the last changeset manually. Thanks for the patch!

andreasgal added a commit that referenced this pull request Jul 8, 2011
Dict object improvements. Performance.
@andreasgal
andreasgal merged commit df79f10 into mozilla:master Jul 8, 2011
@saebekassebil

Copy link
Copy Markdown
Contributor Author

Yea, sorry I was only commiting to my fork, didn't mean to include the later commits..?

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.

3 participants