Code review comment for lp:~alan-griffiths/mir/extend-demo-client-display-config
Revision history for this message
| Alan Griffiths (alan-griffiths) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
> I find the use of the conf global strange, but actual use looks fine. The
> current code does no error-checking, so we're not losing anything.
The use of the conf global is necessitated by the way the toolkit API works: it only ever returns the global config. We need to track the session config ourselves.
Nits fixed