src: operator[] checks bounds in debug mode

GitHubTracey · MylesBorins · commit 26bd574b3299 · 2017-10-23T17:35:50.000-04:00
PR-URL: #16002 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
diff --git a/src/string_search.h b/src/string_search.h
@@ -44,7 +44,9 @@ class Vector {
 
   // Access individual vector elements - checks bounds in debug mode.
   T& operator[](size_t index) const {
+#ifdef DEBUG
     CHECK(index < length_);
+#endif
     return start_[is_forward_ ? index : (length_ - index - 1)];
   }
 

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,9 @@ class Vector {`
`44`	`44`
`45`	`45`	`// Access individual vector elements - checks bounds in debug mode.`
`46`	`46`	`T& operator[](size_t index) const {`
	`47`	`+#ifdef DEBUG`
`47`	`48`	`CHECK(index < length_);`
	`49`	`+#endif`
`48`	`50`	`return start_[is_forward_ ? index : (length_ - index - 1)];`
`49`	`51`	`}`
`50`	`52`