Hi Johan, >> --- >> src/sdpd-service.c | 46 >> ++++++++++++++++++++++++++++++---------------- >> 1 files changed, 30 insertions(+), 16 deletions(-) > > So your commit message says two things: "Clean up code..." and "Minor > fix..". Cleanups and fixes should be in separate patches. Don't be > afraid of "too small" or trivial patches. As long as a patch contains a > single logically independent change it's fine. > Just a clarification. I will still have to have the "clean up" patch as a part of this patch set since it relies on the header file with defines (getting rid of magic numbers). Not sure, if you want me to cut the "fix" one to be separate patch submission altogether... Thanks, Inga Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
