Unity 8

Code review comment for lp:~nick-dedekind/unity8/lp1236249

Revision history for this message

Michael Terry (mterry) wrote on 2013-10-11:

A couple style comments:

17 +void RootActionState::reset() {

Brace should be on new line.

27 if (m_menu && m_menu->rowCount() > 0) {
28 +
29 + ActionStateParser* oldParser = m_menu->actionStateParser();

We can drop the extra new line.

review: Needs Fixing