[PATCH] smbclient tarmode rewrite

[Stefan (metze) Metzmacher]

Am 17.02.2014 14:47, schrieb Stefan (metze) Metzmacher:
> Hi David and Andreas,
> 
>>> Thanks for putting your hand up as second reviewer, and of course a huge
>>> thanks again to Aurélien for his hard work and effort during GSoC to
>>> implement (and thoroughly test) this feature - well done!
>>
>> the style of the code is often not easy to read especially the opposite of a 
>> bool for the return code. I've added additional patches to improve readability 
>> there.
>>
>> I also found is_subpath() pretty scary. I would suggest to simply use 
>> strstr_m().
>>
>>  
>> https://git.samba.org/?p=asn/samba.git;a=shortlog;h=refs/heads/libarchive
> 
> As this is a complete rewrite of clitar.c, can you please make sure
> that the new code follows less README.Coding?
> 
> There're a lot of comments and missing braces, too much whitespaces in
> prototypes,
> a lot of globals, uses of 'False' instead of false, typedef is used...

Sorry, I read the diff in the wrong direction, because I tought the versions
with tab indentation would be the new one.

Can you please fix the indentation to use tabs?

Thanks!
metze