k9mail/k-9 — pull request #155 by dnet (2012-07-06 13:22:58+00:00)
Title: Code cleanup + efficient Map iteration

Most of these modifications are trivial, and should not break anything, I successfully built an APK using `ant debug`.

- Unused and/or unread fields got removed.
- An empty else clause got removed (those that had comments in them got spared).
- An override merely calling a method of `super` got removed (those that made a field reachable from a descendant class got spared).
- Several `if` statements of the form `if (foo) { if (bar) { ... } }` got combined into `if ((foo) && (bar)) { ...}` to avoid unnecessary nesting (those that checked debug flags got spared).
- A loop iterating over the `keySet` of a map and then calling `get` for each key got replaced with one iterating over `entrySet` to improve efficiency.