Code review comment for lp:~jamesodhunt/upstart/bug-1199778
Revision history for this message
| Steve Langasek (vorlon) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
67d34a1
(Get the code!)
Dmitrijs, the only NIH_LIST_EMPTY() assertion that shows up as part of this patch is one that's being *removed*. Are you concerned that the other assertions are latent bugs (incorrect assertions), like this one apparently was? If not, can I suggest that review of those should be postponed to later, outside of landing this crasher fix?