Code review comment for lp:~mhr3/libzeitgeist/symbols

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Ok, I'll accept your choice on 1). One last nitpick :-)

The namespace for the VAPI should be Zeitgeist.Symbol.* that way it's consistent with how we expose the Zeitgeist.Timestamp.* namespace as well as closer to the Python API for zeitgeist.datamodel.Symbol

review: Needs Fixing

« Back to merge proposal