Code review comment for lp:~posulliv/drizzle/cleanup-replace-typelib
Revision history for this message
| Padraig O'Sullivan (posulliv) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
Yep, good point. I updated the branch and was able to remove the function object like you mentioned. Ran the test suite also and its all good so I pushed the changes to the branch.
Thanks guys for the comments! Btw, I created a blueprint for the task of replacing TYPELIB and linked it to the code-cleanup-c++ blueprint. I also added the TYPELIB blueprint to the low-hanging fruit milestone in case anyone else wants to work on it before I get started with it.
-Padraig