testcode.git

@@ -0,0 +1,37 @@

+---

+

+Language:        Cpp

+BasedOnStyle:    Google

+AlignOperands:   false

+AllowAllArgumentsOnNextLine: true

+AllowAllParametersOfDeclarationOnNextLine: true

+AllowShortCaseLabelsOnASingleLine: false

+AllowShortFunctionsOnASingleLine: Empty

+AllowShortIfStatementsOnASingleLine: WithoutElse

+AllowShortLoopsOnASingleLine: false

+BinPackArguments: true

+BinPackParameters: true

+

+# PackConstructorInitializers: NextLine  # <-- Preferred, but in v14

+ConstructorInitializerAllOnOneLineOrOnePerLine: true

+AllowAllConstructorInitializersOnNextLine: true

+

+# Support vim navigation with [[ and ]]

+BreakBeforeBraces: Custom

+BraceWrapping:

+  AfterClass: true

+  AfterFunction: true

+  AfterStruct: true

+

+BreakBeforeBinaryOperators: NonAssignment

+BreakConstructorInitializers: AfterColon

+DerivePointerAlignment: false

+SortIncludes:    false

+SpacesBeforeTrailingComments: 1

+AccessModifierOffset: -4

+IndentWidth: 4

+ContinuationIndentWidth: 8

+ConstructorInitializerIndentWidth: 8

+IndentCaseLabels: false

+ColumnLimit: 86

+...

@@ -57,6 +57,23 @@ Or, if it's not in your path, something like...

     /mnt/c/Users/$USER/AppData/Local/Programs/Microsoft\ VS\ Code/bin/code .

+

+### Formatting

+

+Use clang-format. Install with:

+

+    sudo apt install clang-format

+

+Then, after making your edits, use the [vim integration](https://clang.llvm.org/docs/ClangFormat.html#vim-integration) or manually format with:

+

+    git status --porcelain | \

+    awk 'match($1, "M|A") && match($2, "(c|h)(pp)?$"){print $2}' | \

+    xargs clang-format -style=file -i

+

+Or, shorter but less comprehensive:

+

+    git ls-files -m | egrep ".(c|h)(pp)?$" | xargs clang-format -style=file -i

+

 ### Other IDEs

 Look for the presence of their project files. They should work normally.

@@ -17,6 +18,6 @@ public:

 int my_lib_fn();

 int my_lib_fn(int x);

-}

+} // namespace MY_LIB

 #endif

@@ -3,22 +3,20 @@

 using namespace std;

-namespace

-{

+namespace {

-  int my_lib_helper_fn(int x) {

-    cout << "The function " << __func__ << " with parameter " << x << " was called." << endl;

+int my_lib_helper_fn(int x)

+{

+    cout << "The function " << __func__ << " with parameter " << x << " was called."

+         << endl;

     return x + 1;

 }

-}

+} // namespace

 namespace MY_LIB {

-my_lib_rect::my_lib_rect(int width, int height):

-  width_(width),

-  height_(height)

-{}

+my_lib_rect::my_lib_rect(int width, int height) : width_(width), height_(height) {}

 int my_lib_rect::area() const

 {

@@ -34,8 +32,9 @@ int my_lib_fn()

 int my_lib_fn(int x)

 {

     int a = my_lib_helper_fn(x);

-    cout << "The function " << __PRETTY_FUNCTION__ << " with parameter " << x << " was called." << endl;

+    cout << "The function " << __PRETTY_FUNCTION__ << " with parameter " << x

+         << " was called." << endl;

     return x + a;

 }

-}

+} // namespace MY_LIB

@@ -32,7 +32,8 @@ void do_work_on_left(bool have_right, int32_t tick)

     std::lock_guard<std::mutex> lm(left_chopstick);

     std::this_thread::sleep_for(std::chrono::milliseconds(get_rnd()));

     if (have_right) {

-        std::cout << "deadlock::thread " << std::hex << this_thread::get_id() << " " << tick << endl;

+        std::cout << "deadlock::thread " << std::hex << this_thread::get_id() << " "

+                  << tick << endl;

     } else {

         do_work_on_right(true, tick);

     }

@@ -43,7 +44,8 @@ void do_work_on_right(bool have_left, int32_t tick)

     std::lock_guard<std::mutex> rm(right_chopstick);

     std::this_thread::sleep_for(std::chrono::milliseconds(get_rnd()));

     if (have_left) {

-        std::cout << "deadlock::thread " << std::hex << this_thread::get_id() << " " << tick << endl;

+        std::cout << "deadlock::thread " << std::hex << this_thread::get_id() << " "

+                  << tick << endl;

     } else {

         do_work_on_left(true, tick);

     }

@@ -56,7 +58,7 @@ void work(int32_t ticks)

     }

 }

-};

+}; // namespace

 int deadlock()

 {

@@ -113,7 +113,8 @@ int main()

     length_of_array = 6;

     int b[6] = {10, 20, 20, 30, 40, 40};

     length_of_array = remove_duplicates(b, length_of_array);

-    cout << "Removing duplicates yielded " << length_of_array << " items " << b << endl;

+    cout << "Removing duplicates yielded " << length_of_array << " items " << b

+         << endl;

     main_helper_fn();

     UI::widget_cb();

@@ -130,8 +131,10 @@ int main()

         scoped_set_adder another_set_adder("and another set_adder");

         cout << "unique_ptr " << (up ? "passes" : "fails") << " if." << endl;

         delete up.release();

-        cout << "unique_ptr after release " << (up ? "passes" : "fails") << " if." << endl;

-        cout << "unique_ptr.get() after release " << (up.get() ? "passes" : "fails") << " if." << endl;

+        cout << "unique_ptr after release " << (up ? "passes" : "fails") << " if."

+             << endl;

+        cout << "unique_ptr.get() after release " << (up.get() ? "passes" : "fails")

+             << " if." << endl;

         scoped_set_adder::dump();

     }

@@ -144,9 +147,12 @@ int main()

     // cout << "deadlock() returned " << deadlock() << endl;

-    for(unsigned i : {0, 1, 2, 3, 4, 5, 6, 7, 31, 32, 33, 4003, 65535, 65536, 65537}) {

-        cout << "next_power_of_two(" << i << ") = " << round_up_to_next_power_of_two(i) << endl;

-        assert(round_up_to_next_power_of_two(i) == another_round_up_to_next_power_of_two(i));

+    for (unsigned i :

+         {0, 1, 2, 3, 4, 5, 6, 7, 31, 32, 33, 4003, 65535, 65536, 65537}) {

+        cout << "next_power_of_two(" << i

+             << ") = " << round_up_to_next_power_of_two(i) << endl;

+        assert(round_up_to_next_power_of_two(i)

+               == another_round_up_to_next_power_of_two(i));

     }

     cout << "Done." << endl;

     return 0;

@@ -26,15 +25,13 @@ int main_helper_fn()

 class MyClass::Impl

 {

 public:

-    void f() {

+    void f()

+    {

         cout << "The function " << __PRETTY_FUNCTION__ << " was called." << endl;

     };

 };

-MyClass::MyClass()

-  : pimpl{std::make_unique<Impl>()}

-{

-}

+MyClass::MyClass() : pimpl{std::make_unique<Impl>()} {}

 // See http://www.cppsamples.com/common-tasks/pimpl.html

 MyClass::~MyClass() = default;

@@ -7,7 +7,7 @@

 namespace {

 std::mutex g_mutex;

 std::set<scoped_set_adder*> g_items;

-}

+} // namespace

 scoped_set_adder::scoped_set_adder(std::string const& name) : name_(name)

 {

@@ -25,6 +25,7 @@ bool scoped_set_adder::dump()

 {

     std::lock_guard<std::mutex> lock(g_mutex);

     bool ret = (g_items.size() > 0);

-    std::cout << "Scoped sets exist: " << ret << ", count: " << g_items.size() << std::endl;

+    std::cout << "Scoped sets exist: " << ret << ", count: " << g_items.size()

+              << std::endl;

     return ret;

 }

@@ -3,15 +3,14 @@

 #include "thread_manager.hpp"

 // using namespace std;

-using std::thread;

-using std::mutex;

 using std::condition_variable;

-using std::chrono::seconds;

-using std::unique_lock;

-using std::lock_guard;

 using std::cout;

 using std::endl;

-

+using std::lock_guard;

+using std::mutex;

+using std::thread;

+using std::unique_lock;

+using std::chrono::seconds;

 /*

 class Reporter {

@@ -26,10 +25,7 @@ class Worker {

 };

 */

-

-Manager::Manager():

-    abort_reporting_(false),

-    data_to_protect_(0)

+Manager::Manager() : abort_reporting_(false), data_to_protect_(0)

 {

     report_thread_ = thread([this] { WriteReport(); });

 }

@@ -3,14 +3,12 @@

 #include <iostream>

 #include "thread_with_lambda.hpp"

-

-namespace

-{

+namespace {

 std::mutex my_mutex;

 std::thread worker;

-}

+} // namespace

 void thread_with_lambda()

 {

@@ -13,5 +13,4 @@ int widget_cb()

     return 0;

 }

-}

-

+} // namespace UI

@@ -7,6 +7,6 @@ namespace UI {

 int widget_cb();

-}

+} // namespace UI

 #endif

@@ -0,0 +1,5 @@

+#ifndef xxx

+

+#define undefined

+

+#endif