RE: AVCTP and AVRCP packet decoder file structure in btmon

[Vikrampal <vikram.pal@samsung.com>]

Thanks Antonio for useful advice!

> -----Original Message-----
> From: Antonio Ospite [mailto:ao2@ao2.it]
> Sent: Tuesday, August 05, 2014 5:22 PM
> To: Vikrampal
> Cc: 'Luiz Augusto von Dentz'; linux-bluetooth@vger.kernel.org; 'Dmitry
> Kasatkin'
> Subject: Re: AVCTP and AVRCP packet decoder file structure in btmon
> 
> On Tue, 05 Aug 2014 12:24:44 +0530
> Vikrampal <vikram.pal@samsung.com> wrote:
> 
> > Sure Luiz!
> >
> > However, I've one more small doubt. If I'm submitting a series of
> > patches Such as 1/4, 2/4, 3/4, 4/4... I guess in such scenarios no
> > need to ensure that the patches separately compiles. What we need to
> > ensure is that all the patches together gets compiled properly. Am I
> mistaken?
> >
> 
> Make sure that sub-sequences of a series produce sensible code.
> 
> The code should $VALIDATE with 1/4 applied.
> The code should $VALIDATE with 1/4 and 2/4 applied.
> The code should $VALIDATE with 1/4, 2/4 and 3/4 applied.
> The code should $VALIDATE with 1/4, 2/4, 3/4 and 4/4 applied.
> 
> The definition of VALIDATE can vary project by project or feature by
feature;
> the minimum requirement should be that compilation always succeeds.
> 
> You can use something like:
> 
> 	git rebase -i --exec "make" some_parent_branch
> 
> to test that compilation passes after each commit.
> 
> Ciao,
>    Antonio
> 
> --
> Antonio Ospite
> http://ao2.it
> 
> A: Because it messes up the order in which people normally read text.
>    See http://en.wikipedia.org/wiki/Posting_style
> Q: Why is top-posting such a bad thing?

Regards,
Vikram