[PATCHSET] Add support for fall through attribute

[Andreas Schneider]

On Tuesday, 16 January 2018 19:31:44 CET Andrew Bartlett wrote:
> On Tue, 2018-01-16 at 15:11 +0100, Andreas Schneider via samba-
> 
> technical wrote:
> > On Thursday, 21 December 2017 20:00:41 CET Andreas Schneider via samba-
> > 
> > technical wrote:
> > > I'm not sure. Either you need to remove the p++ and FALL_THROUGH or we
> > > have
> > > to add a break.
> > > 
> > > However I think we should make the TLDAP test work which currently
> > > fails.
> > 
> > I've talked to Simo, he confirmed that we need to break in this switch
> > statement or we stop parsing in the middle. I've also enabled the TLDAP
> > test.
> Did the TLDAP test cover this, that is did it break previously?  It
> isn't clear from a skim of the patch.

Our git webinterface can't show more context. You need to look at the patch 
with more context:

https://git.samba.org/?p=asn/samba.git;a=blob;f=source3/lib/
tldap.c;h=33a852446b9a253526c70afb82d9ed41086a2de7;hb=refs/heads/master-fallthrough#l1267

Line 1299 has been added.

If someone can give me an example of a string that function parses we can 
write a unit test with cmocka. However I have no clue about tldap and I don't 
think the TLDAP test we have is the right test to test parsing functions!


	Andreas


-- 
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             asn at samba.org
www.samba.org