From b35a2f54bb42a2c67592e8a9ff5e9aad4abc9ec6 Mon Sep 17 00:00:00 2001
From: Paul Beckingham <paul@beckingham.net>
Date: Sat, 31 Dec 2011 11:39:50 -0500
Subject: [PATCH] Documentation

- Removed "suspicion" comment about the autovivifying of map keys.
  Credited Dmitriy properly.
---
 AUTHORS    | 3 ++-
 ChangeLog  | 2 ++
 src/A3.cpp | 2 --
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/AUTHORS b/AUTHORS
index 82bba6ef4..2e2a37554 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -8,6 +8,7 @@ contributions of the following people:
   Cory Donnelly      (Contributing Author)
   Johannes Schlatow  (Contributing Author)
   Dirk Deimeke       (Technical Advisor & Marketing)
+  Wim Schuermann     (Contributing Author)
 
 The following submitted code, packages or analysis, and deserve special thanks:
 
@@ -62,7 +63,6 @@ The following submitted code, packages or analysis, and deserve special thanks:
   Greg Grossmeier
   Barton Meeks
   Martin Klepsch
-  Wim Schuermann
   Ralph Bean
 
 Thanks to the following, who submitted detailed bug reports and excellent
@@ -116,4 +116,5 @@ suggestions:
   Aikido Guy
   Gour-Gadadhara Dasa
   Aaron Jackson
+  Dmitriy Samborskiy
 
diff --git a/ChangeLog b/ChangeLog
index 01a4ee2aa..ab7f02b8e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -219,6 +219,8 @@
  + Fixed bug #860, which prevented lower-case priority values from being
    accepted (thanks to Michelle Crane).
  + Fixed bug #862, which suppressed feedback from the 'denotate' command.
+ + Fixed bug #892, which caused a segfault due to misuse of
+   std::map::operator[] (thanks to Dmitriy Samborskiy).
 
 # Untracked Bugs, biggest first.
  + Fixed bug that required the '%YAML' prologue in a YAML import.
diff --git a/src/A3.cpp b/src/A3.cpp
index eaca05f29..2582027d3 100644
--- a/src/A3.cpp
+++ b/src/A3.cpp
@@ -1290,8 +1290,6 @@ bool A3::is_attr (Nibbler& n, Arg& arg)
         // Most attributes are standard, some are pseudo-attributes, such as
         // 'limit:page', which is not represented by a column object, and
         // therefore not stored.
-
-        // I suspect this is auto-vivifying context.columns["limit"].  Bugger.
         std::map<std::string, Column*>::iterator i = context.columns.find (name);
         if (i != context.columns.end ())
           arg._type = Arg::type_id (i->second->type ());