commit f02f934fdc353f6f4c13e1d282239e1238ab6a2d
author roubert@google.com <roubert@google.com@38ededc0-08b8-5190-f2ac-b31f878777ad> Mon Jul 21 19:37:54 2014 +0000
committer roubert@google.com <roubert@google.com@38ededc0-08b8-5190-f2ac-b31f878777ad> Mon Jul 21 19:37:54 2014 +0000
tree 6244375b411799cebed3e6643f0327db5d58110e
parent 8d5e607f6355dea4794867bcc9e74c43d5ba1ca4

Add assert(this != &rule) in Rule::CopyFrom().

It does not make any sense to copy a Rule object onto itself.

R=rouslan@chromium.org
BUG=

Review URL: https://codereview.appspot.com/115150043

git-svn-id: http://libaddressinput.googlecode.com/svn/trunk@313 38ededc0-08b8-5190-f2ac-b31f878777ad

cpp/src/rule.cc

diff --git a/cpp/src/rule.cc b/cpp/src/rule.cc
index a6fd9b5..7ab84e5 100644
--- a/cpp/src/rule.cc
+++ b/cpp/src/rule.cc
@@ -14,6 +14,7 @@
 
 #include "rule.h"
 
+#include <cassert>
 #include <cstddef>
 #include <map>
 #include <string>
@@ -137,6 +138,7 @@
 }
 
 void Rule::CopyFrom(const Rule& rule) {
+  assert(this != &rule);
   id_ = rule.id_;
   format_ = rule.format_;
   latin_format_ = rule.latin_format_;