Performance #190

2015-08-07T14:49:41-04:00

yankejustin commented

2015-08-07 14:49:41 -04:00

(Migrated from github.com)

Performance Changes

- Reduced some string concatenation - Reduced a few decisions that are made very often in the program - Removed an unnecessary method call

Code Quality Improvements

- Make sure the MemoryStream used when adding a Texture Pack is disposed of; even if an exception is thrown after its initialization. - Ensure old and unused texture fonts are disposed of.

Other Notes

I know it doesn't seem like much, but a few more of these and it will be noticeably faster. :)

<h1>Performance Changes</h1> - Reduced some string concatenation - Reduced a few decisions that are made very often in the program - Removed an unnecessary method call <h1>Code Quality Improvements</h1> - Make sure the <code>MemoryStream</code> used when adding a Texture Pack is disposed of; even if an exception is thrown after its initialization. - Ensure old and unused texture fonts are disposed of. <h1>Other Notes</h1> I know it doesn't seem like much, but a few more of these and it will be noticeably faster. :)

ddevault commented

2015-08-07 14:50:52 -04:00

(Migrated from github.com)

Thanks! FYI, the client is being rewritten under the opentk branch, so don't spend too much time optimizing the master branch's client. Will merge this once the travis build passes.

ddevault commented

2015-08-07 15:05:09 -04:00

(Migrated from github.com)

🎉

:tada:

This repo is archived. You cannot comment on pull requests.

1 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: AstralTransRocketries/TrueCraft#190