From 72ef210e1b15f0fa3a749c52176d86ba583d7b79 Mon Sep 17 00:00:00 2001
From: Maxime Coste <frrrwww@gmail.com>
Date: Tue, 23 Apr 2013 18:46:18 +0200
Subject: [PATCH] Buffer: add is_valid(BufferCoord) method

---
 src/buffer.cc              | 7 +++++++
 src/buffer.hh              | 2 ++
 src/buffer_iterator.inl.hh | 7 +------
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/buffer.cc b/src/buffer.cc
index 932b5532b..93360b748 100644
--- a/src/buffer.cc
+++ b/src/buffer.cc
@@ -446,4 +446,11 @@ void Buffer::notify_saved()
     }
 }
 
+bool Buffer::is_valid(const BufferCoord& c) const
+{
+   return (c.line < line_count() and c.column < m_lines[c.line].length()) or
+          (c.line == line_count() - 1 and c.column == m_lines.back().length()) or
+          (c.line == line_count() and c.column == 0);
+}
+
 }
diff --git a/src/buffer.hh b/src/buffer.hh
index b14822944..bca300783 100644
--- a/src/buffer.hh
+++ b/src/buffer.hh
@@ -123,6 +123,8 @@ public:
     String         string(const BufferIterator& begin,
                           const BufferIterator& end) const;
 
+    bool is_valid(const BufferCoord& c) const;
+
     BufferIterator begin() const;
     BufferIterator end() const;
     ByteCount      character_count() const;
diff --git a/src/buffer_iterator.inl.hh b/src/buffer_iterator.inl.hh
index e5b7c95a2..ac4554824 100644
--- a/src/buffer_iterator.inl.hh
+++ b/src/buffer_iterator.inl.hh
@@ -20,12 +20,7 @@ inline const Buffer& BufferIterator::buffer() const
 
 inline bool BufferIterator::is_valid() const
 {
-    return m_buffer and
-           ((line() < m_buffer->line_count() and
-             column() < m_buffer->m_lines[line()].length()) or
-            ((line() == m_buffer->line_count() and column() == 0)) or
-             (line() == m_buffer->line_count() - 1 and
-              column() == m_buffer->m_lines.back().length()));
+    return m_buffer and m_buffer->is_valid(m_coord);
 }
 
 inline void BufferIterator::clamp(bool avoid_eol)