Re: [PATCH] chardev: give some context on chardev-add error

2021-08-04 Thread Marc-André Lureau
Hi On Tue, Aug 3, 2021 at 6:39 PM Daniel P. Berrangé wrote: > On Tue, Aug 03, 2021 at 04:02:29PM +0400, marcandre.lur...@redhat.com > wrote: > > From: Marc-André Lureau > > > > Related to: > > https://bugzilla.redhat.com/show_bug.cgi?id=1984721 > > It is preferrable to describe the problem & ap

Re: [PATCH] chardev: give some context on chardev-add error

2021-08-03 Thread Daniel P . Berrangé
On Tue, Aug 03, 2021 at 04:02:29PM +0400, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Related to: > https://bugzilla.redhat.com/show_bug.cgi?id=1984721 It is preferrable to describe the problem & approach in the comit message, rather than leaving people to read through count

[PATCH] chardev: give some context on chardev-add error

2021-08-03 Thread marcandre . lureau
From: Marc-André Lureau Related to: https://bugzilla.redhat.com/show_bug.cgi?id=1984721 Signed-off-by: Marc-André Lureau --- chardev/char.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/chardev/char.c b/chardev/char.c index d959eec522..f59a61774b 100644 --