On Tue, Aug 16, 2016 at 08:19:08PM +0800, Fam Zheng wrote: > On Tue, 08/16 20:17, Peter Xu wrote: > > > Do we really need error_report_exit when we already have error_fatal? > > > > error_fatal is the name of a global var, not the function. > > I mean most error_report_exit(...) calls can be converted to > error_setg(&error_fatal, ...).
Right. But it's just another way to implement error_report_fatal(). We may still need error_report_fatal() since it's cleaner and shorter than error_setg(&error_fatal, ...). -- peterx