Things that caught my eye during review:

 - Uses dbus-glib, should be ported to gdbus at some point.
 - cups-pk-helper-mechanism.c is by and large boilerplate code and looks 
correct to me. The polkit result is properly checked, and there are no 
complicated memory operations.
- cups.c is rather large, but also pretty straightforward code to translate the 
dbus interface to the libcups API; it does some rather thorough input value 
validity checks.
- uses gettext properly
- Good maintenance in Debian so far, although the package is only half a year 
old; no bug reports
- No Ubuntu bugs, but not surprising as the package hasn't been used by default
- straightforward packaging

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/808829

Title:
  [MIR] cups-pk-helper

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/cups-pk-helper/+bug/808829/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to