Thread (25 messages) flat view 25 messages, 3 authors, 2012-08-14

Re: [PATCH BlueZ] mgmt: print error message when start_discovery fails

From: Joao Paulo Rechi Vita <hidden>
Date: 2012-08-14 17:53:01

On Tue, Aug 14, 2012 at 7:28 AM, Johan Hedberg [off-list ref] wrote:
Hi,

On Tue, Aug 14, 2012, Johan Hedberg wrote:
quoted
On Fri, Aug 03, 2012, João Paulo Rechi Vita wrote:
quoted
If we fail to communicate with the MGMT socket is better to print the
error message on the mgmtops plugin, where it really happened, instead
of leaving this job to its users.
---
 src/adapter.c | 6 +-----
 src/mgmt.c    | 7 +++++--
 2 files changed, 6 insertions(+), 7 deletions(-)
How do these two patches differ from patches 02/14 and 07/14 of the
patch set that was sent on July 27th? Why were these two patches resent
separately? A cover letter would have been quite useful here.
Nevermind. I see that they reference the original thread and are
probably updates based on received comments. Anyway, it would have been
clearer if you'd have added "v2" to the subject and kept the xx/yy
numbering.
Sorry, I thought that using "--in-reply-to <message_id>" was enough
for you to figure out what was going without problems. I've only
re-generated these two when re-sending, that's why they got a
different numbering. I'll be more caution next time.

-- 
João Paulo Rechi Vita
Openbossa Labs - INdT
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help