fstream.tcc (basic_filebuf<>::underflow): Don't call codecvt::in at all if _M_ext_next >= _M_ext_end.

2007-03-27  Paolo Carlini  <pcarlini@suse.de>

	* include/bits/fstream.tcc (basic_filebuf<>::underflow):
	Don't call codecvt::in at all if _M_ext_next >= _M_ext_end.
	* testsuite/27_io/basic_filebuf/underflow/char/3.cc: New.

From-SVN: r123260

3 files changed, 99 insertions, 5 deletions

diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog
index 5bf125a..a9730655 100644
--- a/libstdc++-v3/ChangeLog
+++ b/libstdc++-v3/ChangeLog
@@ -1,3 +1,9 @@
+2007-03-27  Paolo Carlini  <pcarlini@suse.de>
+
+	* include/bits/fstream.tcc (basic_filebuf<>::underflow):
+	Don't call codecvt::in at all if _M_ext_next >= _M_ext_end.
+	* testsuite/27_io/basic_filebuf/underflow/char/3.cc: New.
+
 2007-03-26  Benjamin Kosnik  <bkoz@redhat.com>
 
 	* include/c_compatibility/complex.h: Without
diff --git a/libstdc++-v3/include/bits/fstream.tcc b/libstdc++-v3/include/bits/fstream.tcc
index 5520f9b..54e75db 100644
--- a/libstdc++-v3/include/bits/fstream.tcc
+++ b/libstdc++-v3/include/bits/fstream.tcc
@@ -279,16 +279,19 @@ _GLIBCXX_BEGIN_NAMESPACE(std)
 		      _M_ext_end += __elen;
 		    }
 
-		  char_type* __iend;
-		  __r = _M_codecvt->in(_M_state_cur, _M_ext_next,
-				       _M_ext_end, _M_ext_next, this->eback(),
-				       this->eback() + __buflen, __iend);
+		  char_type* __iend = this->eback();
+		  if (_M_ext_next < _M_ext_end)
+		    __r = _M_codecvt->in(_M_state_cur, _M_ext_next,
+					 _M_ext_end, _M_ext_next,
+					 this->eback(),
+					 this->eback() + __buflen, __iend);
 		  if (__r == codecvt_base::noconv)
 		    {
 		      size_t __avail = _M_ext_end - _M_ext_buf;
 		      __ilen = std::min(__avail, __buflen);
 		      traits_type::copy(this->eback(),
-					reinterpret_cast<char_type*>(_M_ext_buf), __ilen);
+					reinterpret_cast<char_type*>
+					(_M_ext_buf), __ilen);
 		      _M_ext_next = _M_ext_buf + __ilen;
 		    }
 		  else
diff --git a/libstdc++-v3/testsuite/27_io/basic_filebuf/underflow/char/3.cc b/libstdc++-v3/testsuite/27_io/basic_filebuf/underflow/char/3.cc
new file mode 100644
index 0000000..d76a52e
--- /dev/null
+++ b/libstdc++-v3/testsuite/27_io/basic_filebuf/underflow/char/3.cc
@@ -0,0 +1,85 @@
+// Copyright (C) 2007 Free Software Foundation, Inc.
+//
+// This file is part of the GNU ISO C++ Library.  This library is free
+// software; you can redistribute it and/or modify it under the
+// terms of the GNU General Public License as published by the
+// Free Software Foundation; either version 2, or (at your option)
+// any later version.
+
+// This library is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+
+// You should have received a copy of the GNU General Public License along
+// with this library; see the file COPYING.  If not, write to the Free
+// Software Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301,
+// USA.
+
+// 27.8.1.4 Overridden virtual functions
+
+// { dg-require-fileio "" }
+
+#include <fstream>
+#include <cctype>
+#include <locale>
+#include <testsuite_hooks.h>
+
+class Mycvtcc
+: public std::codecvt<char, char, std::mbstate_t>
+{
+protected:
+  virtual result
+  do_in(state_type&,
+	const extern_type* from, const extern_type* from_end,
+	const extern_type*& from_next,
+	intern_type* to, intern_type* to_limit,
+	intern_type*& to_next) const
+  {
+    from_next = from, to_next = to;
+    
+    if (from_next == from_end || to_next == to_limit)
+      return partial;
+    
+    if (std::islower(*from_next))
+      *to_next = std::toupper(*from_next);
+    else
+      *to_next = *from_next;
+    ++from_next, ++to_next;
+    return ok;
+  }
+
+  virtual bool
+  do_always_noconv() const throw()
+  { return false; }
+};
+
+// See Novell Bug 255122
+void test01()
+{
+  bool test __attribute__((unused)) = true;
+  using namespace std;
+
+  const char* name = "tmp_underflow_3.tst";
+  filebuf fbuf, fbufx;
+
+  fbuf.open(name, ios_base::out | ios_base::trunc);
+  VERIFY( fbuf.sputc('a') == 'a' );
+  VERIFY( fbuf.sputc('b') == 'b' );
+  VERIFY( fbuf.sputc('\n') == '\n' );
+  fbuf.close();
+
+  fbufx.pubimbue(locale(locale::classic(), new Mycvtcc));
+  fbufx.open(name, ios_base::in);
+  VERIFY( fbufx.sbumpc() == 'A' );
+  VERIFY( fbufx.sbumpc() == 'B' );
+  VERIFY( fbufx.sbumpc() == '\n' );
+  VERIFY( fbufx.sbumpc() == filebuf::traits_type::eof() );
+  fbufx.close();
+}
+
+int main()
+{
+  test01();
+  return 0;
+}