[Box Backup-dev] Merges

Chris Wilson boxbackup-dev@fluffy.co.uk
Sun, 20 Aug 2006 11:16:27 +0100 (BST)


Hi Ben,

On Sun, 20 Aug 2006, Ben Summers wrote:

>>  Well spotted, thanks! Fixed in r799. I'll wait for Ben to tell me what he 
>>  wants the struct to be called before asking you to review again.
>
> Well cmd_info_t is not exactly in the project coding style. 
> QueryCommandSpecification might fit in better and describe what it is more 
> accurately.

OK, fixed.

> I hope everyone else thinks coding standards are important.

Yes, I do, although I disagree about certain standards in certain projects 
I will follow them anyway to avoid causing trouble for others :-) I hope 
you will forgive me, and correct me, if I occasionally let something slip 
through.

> I read a essay recently which suggested that the compilers should 
> enforce coding standards. I think that's a good plan. It's not that any 
> coding standard is correct or better or The One True Way, it's just that 
> it's important for everyone on a project to use the same one.

I think it would be a good idea, but it would have to be optional, or else 
nothing would compile any more :-)

Cheers, Chris.
-- 
_ ___ __     _
  / __/ / ,__(_)_  | Chris Wilson <0000 at qwirx.com> - Cambs UK |
/ (_/ ,\/ _/ /_ \ | Security/C/C++/Java/Perl/SQL/HTML Developer |
\ _/_/_/_//_/___/ | We are GNU-free your mind-and your software |