From 4ccf40038e773478265632f67eda45a2561f4586 Mon Sep 17 00:00:00 2001 From: Dane Springmeyer Date: Mon, 13 Aug 2012 11:27:02 -0700 Subject: [PATCH] add preferred assignment syntax for zero initialization --- docs/contributing.markdown | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/docs/contributing.markdown b/docs/contributing.markdown index 6bc272f18b..c6938e3e56 100644 --- a/docs/contributing.markdown +++ b/docs/contributing.markdown @@ -92,7 +92,7 @@ If you see bits of code around that do not follow these please don't hesitate to #### Use C++ style casts static_cast(value); // yes - + (int)value; // no #### Use const keyword after the type @@ -109,22 +109,29 @@ If you see bits of code around that do not follow these please don't hesitate to #### Shared pointers should be created with [boost::make_shared](http://www.boost.org/doc/libs/1_47_0/libs/smart_ptr/make_shared.html) where possible +#### Use assignment operator for zero initialized numbers + + double num = 0; // please + + double num(0); // no + + #### Function definitions should not be separated from their arguments: void foo(int a) // please - + void foo (int a) // no #### Separate arguments by a single space: void foo(int a, float b) // please - + void foo(int a,float b) // no #### Space between operators: if (a == b) // please - + if(a==b) // no #### Braces should always be used: @@ -136,7 +143,7 @@ If you see bits of code around that do not follow these please don't hesitate to if (!file) throw mapnik::datasource_exception("not found"); // no - + #### Braces should be on a separate line: