Code review comment for lp:~zeitgeist/zeitgeist/bb-extensions-conf

Revision history for this message
Michal Hruby (mhr3) wrote :

1) I'd get rid of the get_name method, we can just use get_type().name()
2) if the extension needed something special, store/retrieve could be virtual

Other than that it looks fine, feel free to merge!

review: Approve

« Back to merge proposal