From b3c9ee3f332dadc32e5c0e8ae57e2ccfbec9fc6f Mon Sep 17 00:00:00 2001 From: Chris Backhouse Date: Thu, 9 Sep 2021 14:07:01 +0100 Subject: [PATCH] Complain about bad autos in examples --- sbn/codingconv/CodingConventions.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sbn/codingconv/CodingConventions.md b/sbn/codingconv/CodingConventions.md index 952f79aab..ea236d174 100644 --- a/sbn/codingconv/CodingConventions.md +++ b/sbn/codingconv/CodingConventions.md @@ -437,6 +437,8 @@ It should be used judiciously (and sparsely). **[D]** In general, the use of `auto` is **discouraged** +TODO - agree, but we have quite a few dubious `auto`s in the examples in this very document. Let's practice what we preach + **[S]** The `auto` keyword can be safely used when the underlying type is _obvious_ from the code in the same line or the previous one. In doubt, spell the type out instead.