Kamil Dudka 8d9eac
From e87ab5b991b08092a7e07af82b3ec822a8604151 Mon Sep 17 00:00:00 2001
Ondrej Oprala 32b1e5
From: Ondrej Oprala <ooprala@redhat.com>
Ondrej Oprala 32b1e5
Date: Wed, 5 Aug 2015 09:15:09 +0200
Ondrej Oprala 32b1e5
Subject: [PATCH] expand,unexpand: add multibyte support
Ondrej Oprala 32b1e5
MIME-Version: 1.0
Ondrej Oprala 32b1e5
Content-Type: text/plain; charset=UTF-8
Ondrej Oprala 32b1e5
Content-Transfer-Encoding: 8bit
Ondrej Oprala 32b1e5
Ondrej Oprala 32b1e5
* NEWS: Mention the changes.
Ondrej Oprala 32b1e5
* bootstrap.conf: Add mbfile to the list of modules.
Ondrej Oprala 32b1e5
* configure.ac: Properly initialize mbfile.
Ondrej Oprala 32b1e5
* src/expand.c (expand): Iterate over multibyte characters properly.
Ondrej Oprala 32b1e5
* src/unexpand.c (unexpand): Iterate over multibyte characters
Ondrej Oprala 32b1e5
properly.
Ondrej Oprala 32b1e5
* tests/local.mk: Add new tests.
Ondrej Oprala 32b1e5
* tests/{expand,unexpand}/mb.sh: New tests.
Ondrej Oprala 32b1e5
Ondrej Oprala 32b1e5
Co-authored-by: Pádraig Brady <pbrady@redhat.com>
Ondrej Oprala 32b1e5
---
Ondrej Oprala 32b1e5
 bootstrap.conf       |   1 +
Ondrej Oprala 32b1e5
 configure.ac         |   2 +
Kamil Dudka d3849c
 lib/mbfile.c         |   3 +
Kamil Dudka d3849c
 lib/mbfile.h         | 255 +++++++++++++++++++++++++++++++++++++++++++++++++++
Kamil Dudka d3849c
 m4/mbfile.m4         |  14 +++
Kamil Dudka 8d9eac
 src/expand.c         |  43 +++++----
Tom Stellard f4a53e
 src/local.mk         |   4 +-
Kamil Dudka 8d9eac
 src/unexpand.c       |  54 +++++++----
Kamil Dudka d3849c
 tests/expand/mb.sh   |  98 ++++++++++++++++++++
Ondrej Oprala 32b1e5
 tests/local.mk       |   2 +
Kamil Dudka d3849c
 tests/unexpand/mb.sh |  97 ++++++++++++++++++++
Kamil Dudka 8d9eac
 10 files changed, 535 insertions(+), 34 deletions(-)
Kamil Dudka d3849c
 create mode 100644 lib/mbfile.c
Kamil Dudka d3849c
 create mode 100644 lib/mbfile.h
Kamil Dudka d3849c
 create mode 100644 m4/mbfile.m4
Ondrej Oprala 32b1e5
 create mode 100755 tests/expand/mb.sh
Ondrej Oprala 32b1e5
 create mode 100755 tests/unexpand/mb.sh
Ondrej Oprala 32b1e5
Ondrej Oprala 32b1e5
diff --git a/bootstrap.conf b/bootstrap.conf
Kamil Dudka 8d9eac
index 8a0ff31..a1c78b2 100644
Ondrej Oprala 32b1e5
--- a/bootstrap.conf
Ondrej Oprala 32b1e5
+++ b/bootstrap.conf
Kamil Dudka 8d9eac
@@ -152,6 +152,7 @@ gnulib_modules="
Ondrej Oprala 32b1e5
   maintainer-makefile
Ondrej Oprala 32b1e5
   malloc-gnu
Ondrej Oprala 32b1e5
   manywarnings
Ondrej Oprala 32b1e5
+  mbfile
Ondrej Oprala 32b1e5
   mbrlen
Ondrej Oprala 32b1e5
   mbrtowc
Ondrej Oprala 32b1e5
   mbsalign
Ondrej Oprala 32b1e5
diff --git a/configure.ac b/configure.ac
Kamil Dudka 8d9eac
index 1e74b36..24c9725 100644
Ondrej Oprala 32b1e5
--- a/configure.ac
Ondrej Oprala 32b1e5
+++ b/configure.ac
Kamil Dudka 8d9eac
@@ -427,6 +427,8 @@ fi
Ondrej Oprala 32b1e5
 # I'm leaving it here for now.  This whole thing needs to be modernized...
Ondrej Oprala 32b1e5
 gl_WINSIZE_IN_PTEM
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
+gl_MBFILE
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
 gl_HEADER_TIOCGWINSZ_IN_TERMIOS_H
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
 if test $gl_cv_sys_tiocgwinsz_needs_termios_h = no && \
Kamil Dudka 8d9eac
diff --git a/lib/mbfile.c b/lib/mbfile.c
Kamil Dudka 8d9eac
new file mode 100644
Kamil Dudka 8d9eac
index 0000000..b0a468e
Kamil Dudka 8d9eac
--- /dev/null
Kamil Dudka 8d9eac
+++ b/lib/mbfile.c
Kamil Dudka 8d9eac
@@ -0,0 +1,3 @@
Kamil Dudka 8d9eac
+#include <config.h>
Kamil Dudka 8d9eac
+#define MBFILE_INLINE _GL_EXTERN_INLINE
Kamil Dudka 8d9eac
+#include "mbfile.h"
Kamil Dudka 8d9eac
diff --git a/lib/mbfile.h b/lib/mbfile.h
Ondrej Oprala 32b1e5
new file mode 100644
Kamil Dudka 8d9eac
index 0000000..11f1b12
Ondrej Oprala 32b1e5
--- /dev/null
Kamil Dudka 8d9eac
+++ b/lib/mbfile.h
Kamil Dudka 8d9eac
@@ -0,0 +1,255 @@
Kamil Dudka 8d9eac
+/* Multibyte character I/O: macros for multi-byte encodings.
Kamil Dudka 8d9eac
+   Copyright (C) 2001, 2005, 2009-2015 Free Software Foundation, Inc.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+   This program is free software: you can redistribute it and/or modify
Ondrej Oprala 32b1e5
+   it under the terms of the GNU General Public License as published by
Kamil Dudka 8d9eac
+   the Free Software Foundation; either version 3 of the License, or
Ondrej Oprala 32b1e5
+   (at your option) any later version.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+   This program is distributed in the hope that it will be useful,
Ondrej Oprala 32b1e5
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
Ondrej Oprala 32b1e5
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
Ondrej Oprala 32b1e5
+   GNU General Public License for more details.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+   You should have received a copy of the GNU General Public License
Ondrej Oprala 32b1e5
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+/* Written by Mitsuru Chinen <mchinen@yamato.ibm.com>
Kamil Dudka 8d9eac
+   and Bruno Haible <bruno@clisp.org>.  */
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+/* The macros in this file implement multi-byte character input from a
Kamil Dudka 8d9eac
+   stream.
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+   mb_file_t
Kamil Dudka 8d9eac
+     is the type for multibyte character input stream, usable for variable
Kamil Dudka 8d9eac
+     declarations.
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+   mbf_char_t
Kamil Dudka 8d9eac
+     is the type for multibyte character or EOF, usable for variable
Kamil Dudka 8d9eac
+     declarations.
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+   mbf_init (mbf, stream)
Kamil Dudka 8d9eac
+     initializes the MB_FILE for reading from stream.
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+   mbf_getc (mbc, mbf)
Kamil Dudka 8d9eac
+     reads the next multibyte character from mbf and stores it in mbc.
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+   mb_iseof (mbc)
Kamil Dudka 8d9eac
+     returns true if mbc represents the EOF value.
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+   Here are the function prototypes of the macros.
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+   extern void          mbf_init (mb_file_t mbf, FILE *stream);
Kamil Dudka 8d9eac
+   extern void          mbf_getc (mbf_char_t mbc, mb_file_t mbf);
Kamil Dudka 8d9eac
+   extern bool          mb_iseof (const mbf_char_t mbc);
Kamil Dudka 8d9eac
+ */
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#ifndef _MBFILE_H
Kamil Dudka 8d9eac
+#define _MBFILE_H 1
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#include <assert.h>
Kamil Dudka 8d9eac
+#include <stdbool.h>
Kamil Dudka 8d9eac
+#include <stdio.h>
Kamil Dudka 8d9eac
+#include <string.h>
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+/* Tru64 with Desktop Toolkit C has a bug: <stdio.h> must be included before
Kamil Dudka 8d9eac
+   <wchar.h>.
Kamil Dudka 8d9eac
+   BSD/OS 4.1 has a bug: <stdio.h> and <time.h> must be included before
Kamil Dudka 8d9eac
+   <wchar.h>.  */
Kamil Dudka 8d9eac
+#include <stdio.h>
Kamil Dudka 8d9eac
+#include <time.h>
Kamil Dudka 8d9eac
+#include <wchar.h>
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#include "mbchar.h"
Kamil Dudka 8d9eac
+
Kamil Dudka 8d9eac
+#ifndef _GL_INLINE_HEADER_BEGIN
Kamil Dudka 8d9eac
+ #error "Please include config.h first."
Kamil Dudka 8d9eac
+#endif
Kamil Dudka 8d9eac
+_GL_INLINE_HEADER_BEGIN
Kamil Dudka 8d9eac
+#ifndef MBFILE_INLINE
Kamil Dudka 8d9eac
+# define MBFILE_INLINE _GL_INLINE
Kamil Dudka 8d9eac
+#endif
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+struct mbfile_multi {
Kamil Dudka 8d9eac
+  FILE *fp;
Kamil Dudka 8d9eac
+  bool eof_seen;
Kamil Dudka 8d9eac
+  bool have_pushback;
Kamil Dudka 8d9eac
+  mbstate_t state;
Kamil Dudka 8d9eac
+  unsigned int bufcount;
Kamil Dudka 8d9eac
+  char buf[MBCHAR_BUF_SIZE];
Kamil Dudka 8d9eac
+  struct mbchar pushback;
Kamil Dudka 8d9eac
+};
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+MBFILE_INLINE void
Kamil Dudka 8d9eac
+mbfile_multi_getc (struct mbchar *mbc, struct mbfile_multi *mbf)
Ondrej Oprala 32b1e5
+{
Kamil Dudka 8d9eac
+  size_t bytes;
Kamil Dudka 8d9eac
+
Kamil Dudka 8d9eac
+  /* If EOF has already been seen, don't use getc.  This matters if
Kamil Dudka 8d9eac
+     mbf->fp is connected to an interactive tty.  */
Kamil Dudka 8d9eac
+  if (mbf->eof_seen)
Kamil Dudka 8d9eac
+    goto eof;
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+  /* Return character pushed back, if there is one.  */
Kamil Dudka 8d9eac
+  if (mbf->have_pushback)
Ondrej Oprala 32b1e5
+    {
Kamil Dudka 8d9eac
+      mb_copy (mbc, &mbf->pushback);
Kamil Dudka 8d9eac
+      mbf->have_pushback = false;
Kamil Dudka 8d9eac
+      return;
Kamil Dudka 8d9eac
+    }
Kamil Dudka 8d9eac
+
Kamil Dudka 8d9eac
+  /* Before using mbrtowc, we need at least one byte.  */
Kamil Dudka 8d9eac
+  if (mbf->bufcount == 0)
Kamil Dudka 8d9eac
+    {
Kamil Dudka 8d9eac
+      int c = getc (mbf->fp);
Kamil Dudka 8d9eac
+      if (c == EOF)
Ondrej Oprala 32b1e5
+        {
Kamil Dudka 8d9eac
+          mbf->eof_seen = true;
Kamil Dudka 8d9eac
+          goto eof;
Ondrej Oprala 32b1e5
+        }
Kamil Dudka 8d9eac
+      mbf->buf[0] = (unsigned char) c;
Kamil Dudka 8d9eac
+      mbf->bufcount++;
Kamil Dudka 8d9eac
+    }
Kamil Dudka 8d9eac
+
Kamil Dudka 8d9eac
+  /* Handle most ASCII characters quickly, without calling mbrtowc().  */
Kamil Dudka 8d9eac
+  if (mbf->bufcount == 1 && mbsinit (&mbf->state) && is_basic (mbf->buf[0]))
Kamil Dudka 8d9eac
+    {
Kamil Dudka 8d9eac
+      /* These characters are part of the basic character set.  ISO C 99
Kamil Dudka 8d9eac
+         guarantees that their wide character code is identical to their
Kamil Dudka 8d9eac
+         char code.  */
Kamil Dudka 8d9eac
+      mbc->wc = mbc->buf[0] = mbf->buf[0];
Kamil Dudka 8d9eac
+      mbc->wc_valid = true;
Kamil Dudka 8d9eac
+      mbc->ptr = &mbc->buf[0];
Kamil Dudka 8d9eac
+      mbc->bytes = 1;
Kamil Dudka 8d9eac
+      mbf->bufcount = 0;
Kamil Dudka 8d9eac
+      return;
Ondrej Oprala 32b1e5
+    }
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+  /* Use mbrtowc on an increasing number of bytes.  Read only as many bytes
Kamil Dudka 8d9eac
+     from mbf->fp as needed.  This is needed to give reasonable interactive
Kamil Dudka 8d9eac
+     behaviour when mbf->fp is connected to an interactive tty.  */
Kamil Dudka 8d9eac
+  for (;;)
Ondrej Oprala 32b1e5
+    {
Kamil Dudka 8d9eac
+      /* We don't know whether the 'mbrtowc' function updates the state when
Kamil Dudka 8d9eac
+         it returns -2, - this is the ISO C 99 and glibc-2.2 behaviour - or
Kamil Dudka 8d9eac
+         not - amended ANSI C, glibc-2.1 and Solaris 2.7 behaviour.  We
Kamil Dudka 8d9eac
+         don't have an autoconf test for this, yet.
Kamil Dudka 8d9eac
+         The new behaviour would allow us to feed the bytes one by one into
Kamil Dudka 8d9eac
+         mbrtowc.  But the old behaviour forces us to feed all bytes since
Kamil Dudka 8d9eac
+         the end of the last character into mbrtowc.  Since we want to retry
Kamil Dudka 8d9eac
+         with more bytes when mbrtowc returns -2, we must backup the state
Kamil Dudka 8d9eac
+         before calling mbrtowc, because implementations with the new
Kamil Dudka 8d9eac
+         behaviour will clobber it.  */
Kamil Dudka 8d9eac
+      mbstate_t backup_state = mbf->state;
Kamil Dudka 8d9eac
+
Kamil Dudka 8d9eac
+      bytes = mbrtowc (&mbc->wc, &mbf->buf[0], mbf->bufcount, &mbf->state);
Kamil Dudka 8d9eac
+
Kamil Dudka 8d9eac
+      if (bytes == (size_t) -1)
Kamil Dudka 8d9eac
+        {
Kamil Dudka 8d9eac
+          /* An invalid multibyte sequence was encountered.  */
Kamil Dudka 8d9eac
+          /* Return a single byte.  */
Kamil Dudka 8d9eac
+          bytes = 1;
Kamil Dudka 8d9eac
+          mbc->wc_valid = false;
Kamil Dudka 8d9eac
+          break;
Kamil Dudka 8d9eac
+        }
Kamil Dudka 8d9eac
+      else if (bytes == (size_t) -2)
Ondrej Oprala 32b1e5
+        {
Kamil Dudka 8d9eac
+          /* An incomplete multibyte character.  */
Kamil Dudka 8d9eac
+          mbf->state = backup_state;
Kamil Dudka 8d9eac
+          if (mbf->bufcount == MBCHAR_BUF_SIZE)
Kamil Dudka 8d9eac
+            {
Kamil Dudka 8d9eac
+              /* An overlong incomplete multibyte sequence was encountered.  */
Kamil Dudka 8d9eac
+              /* Return a single byte.  */
Kamil Dudka 8d9eac
+              bytes = 1;
Kamil Dudka 8d9eac
+              mbc->wc_valid = false;
Kamil Dudka 8d9eac
+              break;
Kamil Dudka 8d9eac
+            }
Kamil Dudka 8d9eac
+          else
Kamil Dudka 8d9eac
+            {
Kamil Dudka 8d9eac
+              /* Read one more byte and retry mbrtowc.  */
Kamil Dudka 8d9eac
+              int c = getc (mbf->fp);
Kamil Dudka 8d9eac
+              if (c == EOF)
Kamil Dudka 8d9eac
+                {
Kamil Dudka 8d9eac
+                  /* An incomplete multibyte character at the end.  */
Kamil Dudka 8d9eac
+                  mbf->eof_seen = true;
Kamil Dudka 8d9eac
+                  bytes = mbf->bufcount;
Kamil Dudka 8d9eac
+                  mbc->wc_valid = false;
Kamil Dudka 8d9eac
+                  break;
Kamil Dudka 8d9eac
+                }
Kamil Dudka 8d9eac
+              mbf->buf[mbf->bufcount] = (unsigned char) c;
Kamil Dudka 8d9eac
+              mbf->bufcount++;
Kamil Dudka 8d9eac
+            }
Ondrej Oprala 32b1e5
+        }
Ondrej Oprala 32b1e5
+      else
Ondrej Oprala 32b1e5
+        {
Kamil Dudka 8d9eac
+          if (bytes == 0)
Kamil Dudka 8d9eac
+            {
Kamil Dudka 8d9eac
+              /* A null wide character was encountered.  */
Kamil Dudka 8d9eac
+              bytes = 1;
Kamil Dudka 8d9eac
+              assert (mbf->buf[0] == '\0');
Kamil Dudka 8d9eac
+              assert (mbc->wc == 0);
Kamil Dudka 8d9eac
+            }
Kamil Dudka 8d9eac
+          mbc->wc_valid = true;
Kamil Dudka 8d9eac
+          break;
Ondrej Oprala 32b1e5
+        }
Ondrej Oprala 32b1e5
+    }
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+  /* Return the multibyte sequence mbf->buf[0..bytes-1].  */
Kamil Dudka 8d9eac
+  mbc->ptr = &mbc->buf[0];
Kamil Dudka 8d9eac
+  memcpy (&mbc->buf[0], &mbf->buf[0], bytes);
Kamil Dudka 8d9eac
+  mbc->bytes = bytes;
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+  mbf->bufcount -= bytes;
Kamil Dudka 8d9eac
+  if (mbf->bufcount > 0)
Kamil Dudka 8d9eac
+    {
Kamil Dudka 8d9eac
+      /* It's not worth calling memmove() for so few bytes.  */
Kamil Dudka 8d9eac
+      unsigned int count = mbf->bufcount;
Kamil Dudka 8d9eac
+      char *p = &mbf->buf[0];
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+      do
Kamil Dudka 8d9eac
+        {
Kamil Dudka 8d9eac
+          *p = *(p + bytes);
Kamil Dudka 8d9eac
+          p++;
Kamil Dudka 8d9eac
+        }
Kamil Dudka 8d9eac
+      while (--count > 0);
Kamil Dudka 8d9eac
+    }
Kamil Dudka 8d9eac
+  return;
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+eof:
Kamil Dudka 8d9eac
+  /* An mbchar_t with bytes == 0 is used to indicate EOF.  */
Kamil Dudka 8d9eac
+  mbc->ptr = NULL;
Kamil Dudka 8d9eac
+  mbc->bytes = 0;
Kamil Dudka 8d9eac
+  mbc->wc_valid = false;
Kamil Dudka 8d9eac
+  return;
Kamil Dudka 8d9eac
+}
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+MBFILE_INLINE void
Kamil Dudka 8d9eac
+mbfile_multi_ungetc (const struct mbchar *mbc, struct mbfile_multi *mbf)
Kamil Dudka 8d9eac
+{
Kamil Dudka 8d9eac
+  mb_copy (&mbf->pushback, mbc);
Kamil Dudka 8d9eac
+  mbf->have_pushback = true;
Kamil Dudka 8d9eac
+}
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+typedef struct mbfile_multi mb_file_t;
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+typedef mbchar_t mbf_char_t;
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#define mbf_init(mbf, stream)                                           \
Kamil Dudka 8d9eac
+  ((mbf).fp = (stream),                                                 \
Kamil Dudka 8d9eac
+   (mbf).eof_seen = false,                                              \
Kamil Dudka 8d9eac
+   (mbf).have_pushback = false,                                         \
Kamil Dudka 8d9eac
+   memset (&(mbf).state, '\0', sizeof (mbstate_t)),                     \
Kamil Dudka 8d9eac
+   (mbf).bufcount = 0)
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#define mbf_getc(mbc, mbf) mbfile_multi_getc (&(mbc), &(mbf))
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#define mbf_ungetc(mbc, mbf) mbfile_multi_ungetc (&(mbc), &(mbf))
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#define mb_iseof(mbc) ((mbc).bytes == 0)
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#ifndef _GL_INLINE_HEADER_BEGIN
Kamil Dudka 8d9eac
+ #error "Please include config.h first."
Kamil Dudka 8d9eac
+#endif
Kamil Dudka 8d9eac
+_GL_INLINE_HEADER_BEGIN
Ondrej Oprala 32b1e5
+
Kamil Dudka 8d9eac
+#endif /* _MBFILE_H */
Kamil Dudka 8d9eac
diff --git a/m4/mbfile.m4 b/m4/mbfile.m4
Kamil Dudka 8d9eac
new file mode 100644
Kamil Dudka 8d9eac
index 0000000..8589902
Kamil Dudka 8d9eac
--- /dev/null
Kamil Dudka 8d9eac
+++ b/m4/mbfile.m4
Kamil Dudka 8d9eac
@@ -0,0 +1,14 @@
Kamil Dudka 8d9eac
+# mbfile.m4 serial 7
Kamil Dudka 8d9eac
+dnl Copyright (C) 2005, 2008-2015 Free Software Foundation, Inc.
Kamil Dudka 8d9eac
+dnl This file is free software; the Free Software Foundation
Kamil Dudka 8d9eac
+dnl gives unlimited permission to copy and/or distribute it,
Kamil Dudka 8d9eac
+dnl with or without modifications, as long as this notice is preserved.
Kamil Dudka 8d9eac
+
Kamil Dudka 8d9eac
+dnl autoconf tests required for use of mbfile.h
Kamil Dudka 8d9eac
+dnl From Bruno Haible.
Kamil Dudka 8d9eac
+
Kamil Dudka 8d9eac
+AC_DEFUN([gl_MBFILE],
Kamil Dudka 8d9eac
+[
Kamil Dudka 8d9eac
+  AC_REQUIRE([AC_TYPE_MBSTATE_T])
Kamil Dudka 8d9eac
+  :
Kamil Dudka 8d9eac
+])
Kamil Dudka 8d9eac
diff --git a/src/expand.c b/src/expand.c
Kamil Dudka 8d9eac
index 9fa2e10..380e020 100644
Kamil Dudka 8d9eac
--- a/src/expand.c
Kamil Dudka 8d9eac
+++ b/src/expand.c
Kamil Dudka 8d9eac
@@ -37,6 +37,9 @@
Kamil Dudka 8d9eac
 #include <stdio.h>
Kamil Dudka 8d9eac
 #include <getopt.h>
Kamil Dudka 8d9eac
 #include <sys/types.h>
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#include <mbfile.h>
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
 #include "system.h"
Kamil Dudka 8d9eac
 #include "die.h"
Ondrej Oprala 32b1e5
 #include "xstrndup.h"
Kamil Dudka 8d9eac
@@ -100,19 +103,19 @@ expand (void)
Ondrej Oprala 32b1e5
 {
Ondrej Oprala 32b1e5
   /* Input stream.  */
Ondrej Oprala 32b1e5
   FILE *fp = next_file (NULL);
Ondrej Oprala 32b1e5
+  mb_file_t mbf;
Ondrej Oprala 32b1e5
+  mbf_char_t c;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
   if (!fp)
Ondrej Oprala 32b1e5
     return;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
+  mbf_init (mbf, fp);
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
   while (true)
Ondrej Oprala 32b1e5
     {
Ondrej Oprala 32b1e5
-      /* Input character, or EOF.  */
Ondrej Oprala 32b1e5
-      int c;
Ondrej Oprala 32b1e5
-
Ondrej Oprala 32b1e5
       /* If true, perform translations.  */
Ondrej Oprala 32b1e5
       bool convert = true;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
-
Ondrej Oprala 32b1e5
       /* The following variables have valid values only when CONVERT
Ondrej Oprala 32b1e5
          is true:  */
Ondrej Oprala 32b1e5
 
Kamil Dudka 8d9eac
@@ -122,17 +125,23 @@ expand (void)
Ondrej Oprala 32b1e5
       /* Index in TAB_LIST of next tab stop to examine.  */
Ondrej Oprala 32b1e5
       size_t tab_index = 0;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
-
Ondrej Oprala 32b1e5
       /* Convert a line of text.  */
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
       do
Ondrej Oprala 32b1e5
         {
Ondrej Oprala 32b1e5
-          while ((c = getc (fp)) < 0 && (fp = next_file (fp)))
Ondrej Oprala 32b1e5
-            continue;
Ondrej Oprala 32b1e5
+          do {
Ondrej Oprala 32b1e5
+            mbf_getc (c, mbf);
Ondrej Oprala 32b1e5
+            if (mb_iseof (c))
Ondrej Oprala 32b1e5
+              {
Ondrej Oprala 32b1e5
+                mbf_init (mbf, fp = next_file (fp));
Ondrej Oprala 32b1e5
+                continue;
Ondrej Oprala 32b1e5
+              }
Ondrej Oprala 32b1e5
+            }
Ondrej Oprala 32b1e5
+          while (false);
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
           if (convert)
Ondrej Oprala 32b1e5
             {
Ondrej Oprala 32b1e5
-              if (c == '\t')
Ondrej Oprala 32b1e5
+              if (mb_iseq (c, '\t'))
Ondrej Oprala 32b1e5
                 {
Ondrej Oprala 32b1e5
                   /* Column the next input tab stop is on.  */
Ondrej Oprala 32b1e5
                   uintmax_t next_tab_column;
Kamil Dudka 8d9eac
@@ -151,32 +160,34 @@ expand (void)
Ondrej Oprala 32b1e5
                     if (putchar (' ') < 0)
Kamil Dudka 8d9eac
                       die (EXIT_FAILURE, errno, _("write error"));
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
-                  c = ' ';
Ondrej Oprala 32b1e5
+                  mb_setascii (&c, ' ');
Ondrej Oprala 32b1e5
                 }
Ondrej Oprala 32b1e5
-              else if (c == '\b')
Ondrej Oprala 32b1e5
+              else if (mb_iseq (c, '\b'))
Ondrej Oprala 32b1e5
                 {
Ondrej Oprala 32b1e5
                   /* Go back one column, and force recalculation of the
Ondrej Oprala 32b1e5
                      next tab stop.  */
Ondrej Oprala 32b1e5
                   column -= !!column;
Ondrej Oprala 32b1e5
                   tab_index -= !!tab_index;
Ondrej Oprala 32b1e5
                 }
Ondrej Oprala 32b1e5
-              else
Ondrej Oprala 32b1e5
+              /* A leading control character could make us trip over.  */
Ondrej Oprala 32b1e5
+              else if (!mb_iscntrl (c))
Ondrej Oprala 32b1e5
                 {
Ondrej Oprala 32b1e5
-                  column++;
Ondrej Oprala 32b1e5
+                  column += mb_width (c);
Ondrej Oprala 32b1e5
                   if (!column)
Kamil Dudka 8d9eac
                     die (EXIT_FAILURE, 0, _("input line is too long"));
Ondrej Oprala 32b1e5
                 }
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
-              convert &= convert_entire_line || !! isblank (c);
Ondrej Oprala 32b1e5
+              convert &= convert_entire_line || mb_isblank (c);
Ondrej Oprala 32b1e5
             }
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
-          if (c < 0)
Ondrej Oprala 32b1e5
+          if (mb_iseof (c))
Ondrej Oprala 32b1e5
             return;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
-          if (putchar (c) < 0)
Ondrej Oprala 32b1e5
+          mb_putc (c, stdout);
Ondrej Oprala 32b1e5
+          if (ferror (stdout))
Kamil Dudka 8d9eac
             die (EXIT_FAILURE, errno, _("write error"));
Ondrej Oprala 32b1e5
         }
Ondrej Oprala 32b1e5
-      while (c != '\n');
Ondrej Oprala 32b1e5
+      while (!mb_iseq (c, '\n'));
Ondrej Oprala 32b1e5
     }
Ondrej Oprala 32b1e5
 }
Ondrej Oprala 32b1e5
 
Tom Stellard f4a53e
diff --git a/src/local.mk b/src/local.mk
Tom Stellard f4a53e
index 72db9c704..ef3bfa469 100644
Tom Stellard f4a53e
--- a/src/local.mk
Tom Stellard f4a53e
+++ b/src/local.mk
Tom Stellard f4a53e
@@ -415,8 +415,8 @@ src_basenc_CPPFLAGS = -DBASE_TYPE=42 $(AM_CPPFLAGS)
Tom Stellard f4a53e
 
Tom Stellard f4a53e
 src_ginstall_CPPFLAGS = -DENABLE_MATCHPATHCON=1 $(AM_CPPFLAGS)
Tom Stellard f4a53e
 
Tom Stellard f4a53e
-src_expand_SOURCES = src/expand.c src/expand-common.c
Tom Stellard f4a53e
-src_unexpand_SOURCES = src/unexpand.c src/expand-common.c
Tom Stellard f4a53e
+src_expand_SOURCES = src/expand.c src/expand-common.c lib/mbfile.c
Tom Stellard f4a53e
+src_unexpand_SOURCES = src/unexpand.c src/expand-common.c lib/mbfile.c
Tom Stellard f4a53e
 
Tom Stellard f4a53e
 # Ensure we don't link against libcoreutils.a as that lib is
Tom Stellard f4a53e
 # not compiled with -fPIC which causes issues on 64 bit at least
Ondrej Oprala 32b1e5
diff --git a/src/unexpand.c b/src/unexpand.c
Kamil Dudka 8d9eac
index 7801274..569a7ee 100644
Ondrej Oprala 32b1e5
--- a/src/unexpand.c
Ondrej Oprala 32b1e5
+++ b/src/unexpand.c
Kamil Dudka 8d9eac
@@ -38,6 +38,9 @@
Ondrej Oprala 32b1e5
 #include <stdio.h>
Ondrej Oprala 32b1e5
 #include <getopt.h>
Ondrej Oprala 32b1e5
 #include <sys/types.h>
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#include <mbfile.h>
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
 #include "system.h"
Kamil Dudka 8d9eac
 #include "die.h"
Ondrej Oprala 32b1e5
 #include "xstrndup.h"
Kamil Dudka 8d9eac
@@ -107,11 +110,12 @@ unexpand (void)
Ondrej Oprala 32b1e5
 {
Ondrej Oprala 32b1e5
   /* Input stream.  */
Ondrej Oprala 32b1e5
   FILE *fp = next_file (NULL);
Ondrej Oprala 32b1e5
+  mb_file_t mbf;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
   /* The array of pending blanks.  In non-POSIX locales, blanks can
Ondrej Oprala 32b1e5
      include characters other than spaces, so the blanks must be
Ondrej Oprala 32b1e5
      stored, not merely counted.  */
Ondrej Oprala 32b1e5
-  char *pending_blank;
Ondrej Oprala 32b1e5
+  mbf_char_t *pending_blank;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
   if (!fp)
Ondrej Oprala 32b1e5
     return;
Kamil Dudka 8d9eac
@@ -119,12 +123,14 @@ unexpand (void)
Ondrej Oprala 32b1e5
   /* The worst case is a non-blank character, then one blank, then a
Ondrej Oprala 32b1e5
      tab stop, then MAX_COLUMN_WIDTH - 1 blanks, then a non-blank; so
Ondrej Oprala 32b1e5
      allocate MAX_COLUMN_WIDTH bytes to store the blanks.  */
Ondrej Oprala 32b1e5
-  pending_blank = xmalloc (max_column_width);
Ondrej Oprala 32b1e5
+  pending_blank = xmalloc (max_column_width * sizeof (mbf_char_t));
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+  mbf_init (mbf, fp);
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
   while (true)
Ondrej Oprala 32b1e5
     {
Ondrej Oprala 32b1e5
       /* Input character, or EOF.  */
Ondrej Oprala 32b1e5
-      int c;
Ondrej Oprala 32b1e5
+      mbf_char_t c;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
       /* If true, perform translations.  */
Ondrej Oprala 32b1e5
       bool convert = true;
Kamil Dudka 8d9eac
@@ -158,12 +164,19 @@ unexpand (void)
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
       do
Ondrej Oprala 32b1e5
         {
Ondrej Oprala 32b1e5
-          while ((c = getc (fp)) < 0 && (fp = next_file (fp)))
Ondrej Oprala 32b1e5
-            continue;
Ondrej Oprala 32b1e5
+          do {
Ondrej Oprala 32b1e5
+            mbf_getc (c, mbf);
Ondrej Oprala 32b1e5
+            if (mb_iseof (c))
Ondrej Oprala 32b1e5
+              {
Ondrej Oprala 32b1e5
+                mbf_init (mbf, fp = next_file (fp));
Ondrej Oprala 32b1e5
+                continue;
Ondrej Oprala 32b1e5
+              }
Ondrej Oprala 32b1e5
+            }
Ondrej Oprala 32b1e5
+          while (false);
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
           if (convert)
Ondrej Oprala 32b1e5
             {
Ondrej Oprala 32b1e5
-              bool blank = !! isblank (c);
Ondrej Oprala 32b1e5
+              bool blank = mb_isblank (c);
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
               if (blank)
Ondrej Oprala 32b1e5
                 {
Kamil Dudka 8d9eac
@@ -180,16 +193,16 @@ unexpand (void)
Ondrej Oprala 32b1e5
                       if (next_tab_column < column)
Kamil Dudka 8d9eac
                         die (EXIT_FAILURE, 0, _("input line is too long"));
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
-                      if (c == '\t')
Ondrej Oprala 32b1e5
+                      if (mb_iseq (c, '\t'))
Ondrej Oprala 32b1e5
                         {
Ondrej Oprala 32b1e5
                           column = next_tab_column;
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
                           if (pending)
Ondrej Oprala 32b1e5
-                            pending_blank[0] = '\t';
Ondrej Oprala 32b1e5
+                            mb_setascii (&pending_blank[0], '\t');
Ondrej Oprala 32b1e5
                         }
Ondrej Oprala 32b1e5
                       else
Ondrej Oprala 32b1e5
                         {
Ondrej Oprala 32b1e5
-                          column++;
Ondrej Oprala 32b1e5
+                          column += mb_width (c);
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
                           if (! (prev_blank && column == next_tab_column))
Ondrej Oprala 32b1e5
                             {
Kamil Dudka 8d9eac
@@ -197,13 +210,14 @@ unexpand (void)
Ondrej Oprala 32b1e5
                                  will be replaced by tabs.  */
Ondrej Oprala 32b1e5
                               if (column == next_tab_column)
Ondrej Oprala 32b1e5
                                 one_blank_before_tab_stop = true;
Ondrej Oprala 32b1e5
-                              pending_blank[pending++] = c;
Ondrej Oprala 32b1e5
+                              mb_copy (&pending_blank[pending++], &c);
Ondrej Oprala 32b1e5
                               prev_blank = true;
Ondrej Oprala 32b1e5
                               continue;
Ondrej Oprala 32b1e5
                             }
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
                           /* Replace the pending blanks by a tab or two.  */
Ondrej Oprala 32b1e5
-                          pending_blank[0] = c = '\t';
Ondrej Oprala 32b1e5
+                          mb_setascii (&c, '\t');
Ondrej Oprala 32b1e5
+                          mb_setascii (&pending_blank[0], '\t');
Ondrej Oprala 32b1e5
                         }
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
                       /* Discard pending blanks, unless it was a single
Kamil Dudka 8d9eac
@@ -211,7 +225,7 @@ unexpand (void)
Ondrej Oprala 32b1e5
                       pending = one_blank_before_tab_stop;
Ondrej Oprala 32b1e5
                     }
Ondrej Oprala 32b1e5
                 }
Ondrej Oprala 32b1e5
-              else if (c == '\b')
Ondrej Oprala 32b1e5
+              else if (mb_iseq (c, '\b'))
Ondrej Oprala 32b1e5
                 {
Ondrej Oprala 32b1e5
                   /* Go back one column, and force recalculation of the
Ondrej Oprala 32b1e5
                      next tab stop.  */
Kamil Dudka 8d9eac
@@ -221,7 +235,7 @@ unexpand (void)
Ondrej Oprala 32b1e5
                 }
Ondrej Oprala 32b1e5
               else
Ondrej Oprala 32b1e5
                 {
Ondrej Oprala 32b1e5
-                  column++;
Ondrej Oprala 32b1e5
+                  column += mb_width (c);
Ondrej Oprala 32b1e5
                   if (!column)
Kamil Dudka 8d9eac
                     die (EXIT_FAILURE, 0, _("input line is too long"));
Ondrej Oprala 32b1e5
                 }
Kamil Dudka 8d9eac
@@ -229,8 +243,11 @@ unexpand (void)
Ondrej Oprala 32b1e5
               if (pending)
Ondrej Oprala 32b1e5
                 {
Ondrej Oprala 32b1e5
                   if (pending > 1 && one_blank_before_tab_stop)
Ondrej Oprala 32b1e5
-                    pending_blank[0] = '\t';
Ondrej Oprala 32b1e5
-                  if (fwrite (pending_blank, 1, pending, stdout) != pending)
Ondrej Oprala 32b1e5
+                    mb_setascii (&pending_blank[0], '\t');
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+                  for (int n = 0; n < pending; ++n)
Ondrej Oprala 32b1e5
+                    mb_putc (pending_blank[n], stdout);
Ondrej Oprala 32b1e5
+                  if (ferror (stdout))
Kamil Dudka 8d9eac
                     die (EXIT_FAILURE, errno, _("write error"));
Ondrej Oprala 32b1e5
                   pending = 0;
Ondrej Oprala 32b1e5
                   one_blank_before_tab_stop = false;
Kamil Dudka 8d9eac
@@ -240,16 +257,17 @@ unexpand (void)
Ondrej Oprala 32b1e5
               convert &= convert_entire_line || blank;
Ondrej Oprala 32b1e5
             }
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
-          if (c < 0)
Ondrej Oprala 32b1e5
+          if (mb_iseof (c))
Ondrej Oprala 32b1e5
             {
Ondrej Oprala 32b1e5
               free (pending_blank);
Ondrej Oprala 32b1e5
               return;
Ondrej Oprala 32b1e5
             }
Kamil Dudka 8d9eac
 
Ondrej Oprala 32b1e5
-          if (putchar (c) < 0)
Ondrej Oprala 32b1e5
+          mb_putc (c, stdout);
Ondrej Oprala 32b1e5
+          if (ferror (stdout))
Kamil Dudka 8d9eac
             die (EXIT_FAILURE, errno, _("write error"));
Ondrej Oprala 32b1e5
         }
Ondrej Oprala 32b1e5
-      while (c != '\n');
Ondrej Oprala 32b1e5
+      while (!mb_iseq (c, '\n'));
Ondrej Oprala 32b1e5
     }
Ondrej Oprala 32b1e5
 }
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
diff --git a/tests/expand/mb.sh b/tests/expand/mb.sh
Ondrej Oprala 32b1e5
new file mode 100755
Ondrej Oprala 32b1e5
index 0000000..7971e18
Ondrej Oprala 32b1e5
--- /dev/null
Ondrej Oprala 32b1e5
+++ b/tests/expand/mb.sh
Ondrej Oprala 32b1e5
@@ -0,0 +1,98 @@
Ondrej Oprala 32b1e5
+#!/bin/sh
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+# Copyright (C) 2012-2015 Free Software Foundation, Inc.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+# This program is free software: you can redistribute it and/or modify
Ondrej Oprala 32b1e5
+# it under the terms of the GNU General Public License as published by
Ondrej Oprala 32b1e5
+# the Free Software Foundation, either version 3 of the License, or
Ondrej Oprala 32b1e5
+# (at your option) any later version.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+# This program is distributed in the hope that it will be useful,
Ondrej Oprala 32b1e5
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
Ondrej Oprala 32b1e5
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
Ondrej Oprala 32b1e5
+# GNU General Public License for more details.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+# You should have received a copy of the GNU General Public License
Ondrej Oprala 32b1e5
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+. "${srcdir=.}/tests/init.sh"; path_prepend_ ./src
Ondrej Oprala 32b1e5
+print_ver_ expand
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+export LC_ALL=en_US.UTF-8
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#input containing multibyte characters
Ondrej Oprala 32b1e5
+cat <<\EOF > in || framework_failure_
Ondrej Oprala 32b1e5
+1234567812345678123456781
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+a	b	c	d
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+ä	ö	ü	ß
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+EOF
Ondrej Oprala 32b1e5
+env printf '   äöü\t.    öüä.   \tä xx\n' >> in || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+cat <<\EOF > exp || framework_failure_
Ondrej Oprala 32b1e5
+1234567812345678123456781
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+a       b       c       d
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+ä       ö       ü       ß
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+   äöü  .    öüä.       ä xx
Ondrej Oprala 32b1e5
+EOF
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+expand < in > out || fail=1
Ondrej Oprala 32b1e5
+compare exp out > /dev/null 2>&1 || fail=1
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#test characters with display widths != 1
Ondrej Oprala 32b1e5
+env printf '12345678
Ondrej Oprala 32b1e5
+e\t|ascii(1)
Ondrej Oprala 32b1e5
+\u00E9\t|composed(1)
Ondrej Oprala 32b1e5
+e\u0301\t|decomposed(1)
Ondrej Oprala 32b1e5
+\u3000\t|ideo-space(2)
Ondrej Oprala 32b1e5
+\uFF0D\t|full-hypen(2)
Ondrej Oprala 32b1e5
+' > in || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+env printf '12345678
Ondrej Oprala 32b1e5
+e       |ascii(1)
Ondrej Oprala 32b1e5
+\u00E9       |composed(1)
Ondrej Oprala 32b1e5
+e\u0301       |decomposed(1)
Ondrej Oprala 32b1e5
+\u3000      |ideo-space(2)
Ondrej Oprala 32b1e5
+\uFF0D      |full-hypen(2)
Ondrej Oprala 32b1e5
+' > exp || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+expand < in > out || fail=1
Ondrej Oprala 32b1e5
+compare exp out > /dev/null 2>&1 || fail=1
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#shouldn't fail with "input line too long"
Ondrej Oprala 32b1e5
+#when a line starts with a control character
Ondrej Oprala 32b1e5
+env printf '\n' > in || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+expand < in > out || fail=1
Ondrej Oprala 32b1e5
+compare in out > /dev/null 2>&1 || fail=1
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#non-Unicode characters interspersed between Unicode ones
Ondrej Oprala 32b1e5
+env printf '12345678
Ondrej Oprala 32b1e5
+\t\xFF|
Ondrej Oprala 32b1e5
+\xFF\t|
Ondrej Oprala 32b1e5
+\t\xFFä|
Ondrej Oprala 32b1e5
+ä\xFF\t|
Ondrej Oprala 32b1e5
+\tä\xFF|
Ondrej Oprala 32b1e5
+\xFF\tä|
Ondrej Oprala 32b1e5
+äbcdef\xFF\t|
Ondrej Oprala 32b1e5
+' > in || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+env printf '12345678
Ondrej Oprala 32b1e5
+        \xFF|
Ondrej Oprala 32b1e5
+\xFF       |
Ondrej Oprala 32b1e5
+        \xFFä|
Ondrej Oprala 32b1e5
+ä\xFF      |
Ondrej Oprala 32b1e5
+        ä\xFF|
Ondrej Oprala 32b1e5
+\xFF       ä|
Ondrej Oprala 32b1e5
+äbcdef\xFF |
Ondrej Oprala 32b1e5
+' > exp || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+expand < in > out || fail=1
Ondrej Oprala 32b1e5
+compare exp out > /dev/null 2>&1 || fail=1
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+exit $fail
Ondrej Oprala 32b1e5
diff --git a/tests/local.mk b/tests/local.mk
Kamil Dudka 8d9eac
index 192f776..8053397 100644
Ondrej Oprala 32b1e5
--- a/tests/local.mk
Ondrej Oprala 32b1e5
+++ b/tests/local.mk
Kamil Dudka 8d9eac
@@ -544,6 +544,7 @@ all_tests =					\
Ondrej Oprala 32b1e5
   tests/du/threshold.sh				\
Ondrej Oprala 32b1e5
   tests/du/trailing-slash.sh			\
Ondrej Oprala 32b1e5
   tests/du/two-args.sh				\
Ondrej Oprala 32b1e5
+  tests/expand/mb.sh				\
Ondrej Oprala 32b1e5
   tests/id/gnu-zero-uids.sh			\
Ondrej Oprala 32b1e5
   tests/id/no-context.sh			\
Ondrej Oprala 32b1e5
   tests/id/context.sh				\
Kamil Dudka 8d9eac
@@ -684,6 +685,7 @@ all_tests =					\
Ondrej Oprala 32b1e5
   tests/touch/read-only.sh			\
Ondrej Oprala 32b1e5
   tests/touch/relative.sh			\
Ondrej Oprala 32b1e5
   tests/touch/trailing-slash.sh			\
Ondrej Oprala 32b1e5
+  tests/unexpand/mb.sh				\
Ondrej Oprala 32b1e5
   $(all_root_tests)
Ondrej Oprala 32b1e5
 
Ondrej Oprala 32b1e5
 # See tests/factor/create-test.sh.
Ondrej Oprala 32b1e5
diff --git a/tests/unexpand/mb.sh b/tests/unexpand/mb.sh
Ondrej Oprala 32b1e5
new file mode 100755
Ondrej Oprala 32b1e5
index 0000000..60d4c1a
Ondrej Oprala 32b1e5
--- /dev/null
Ondrej Oprala 32b1e5
+++ b/tests/unexpand/mb.sh
Ondrej Oprala 32b1e5
@@ -0,0 +1,97 @@
Ondrej Oprala 32b1e5
+#!/bin/sh
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+# Copyright (C) 2012-2015 Free Software Foundation, Inc.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+# This program is free software: you can redistribute it and/or modify
Ondrej Oprala 32b1e5
+# it under the terms of the GNU General Public License as published by
Ondrej Oprala 32b1e5
+# the Free Software Foundation, either version 3 of the License, or
Ondrej Oprala 32b1e5
+# (at your option) any later version.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+# This program is distributed in the hope that it will be useful,
Ondrej Oprala 32b1e5
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
Ondrej Oprala 32b1e5
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
Ondrej Oprala 32b1e5
+# GNU General Public License for more details.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+# You should have received a copy of the GNU General Public License
Ondrej Oprala 32b1e5
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+. "${srcdir=.}/tests/init.sh"; path_prepend_ ./src
Ondrej Oprala 32b1e5
+print_ver_ unexpand
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+export LC_ALL=en_US.UTF-8
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#input containing multibyte characters
Ondrej Oprala 32b1e5
+cat > in <<\EOF
Ondrej Oprala 32b1e5
+1234567812345678123456781
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+a       b       c       d
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+ä       ö       ü       ß
Ondrej Oprala 32b1e5
+.       .       .       .
Ondrej Oprala 32b1e5
+   äöü  .    öüä.       ä xx
Ondrej Oprala 32b1e5
+EOF
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+cat > exp <<\EOF
Ondrej Oprala 32b1e5
+1234567812345678123456781
Ondrej Oprala 32b1e5
+.	.	.	.
Ondrej Oprala 32b1e5
+a	b	c	d
Ondrej Oprala 32b1e5
+.	.	.	.
Ondrej Oprala 32b1e5
+ä	ö	ü	ß
Ondrej Oprala 32b1e5
+.	.	.	.
Ondrej Oprala 32b1e5
+   äöü	.    öüä.	ä xx
Ondrej Oprala 32b1e5
+EOF
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+unexpand -a < in > out || fail=1
Ondrej Oprala 32b1e5
+compare exp out > /dev/null 2>&1 || fail=1
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#test characters with a display width larger than 1
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+env printf '12345678
Ondrej Oprala 32b1e5
+e       |ascii(1)
Ondrej Oprala 32b1e5
+\u00E9       |composed(1)
Ondrej Oprala 32b1e5
+e\u0301       |decomposed(1)
Ondrej Oprala 32b1e5
+\u3000      |ideo-space(2)
Ondrej Oprala 32b1e5
+\uFF0D      |full-hypen(2)
Ondrej Oprala 32b1e5
+' > in || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+env printf '12345678
Ondrej Oprala 32b1e5
+e\t|ascii(1)
Ondrej Oprala 32b1e5
+\u00E9\t|composed(1)
Ondrej Oprala 32b1e5
+e\u0301\t|decomposed(1)
Ondrej Oprala 32b1e5
+\u3000\t|ideo-space(2)
Ondrej Oprala 32b1e5
+\uFF0D\t|full-hypen(2)
Ondrej Oprala 32b1e5
+' > exp || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+unexpand -a < in > out || fail=1
Ondrej Oprala 32b1e5
+compare exp out > /dev/null 2>&1 || fail=1
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#test input where a blank of width > 1 is not being substituted
Ondrej Oprala 32b1e5
+in="$(LC_ALL=en_US.UTF-8 printf ' \u3000  ö       ü       ß')"
Ondrej Oprala 32b1e5
+exp='    ö	     ü	     ß'
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+unexpand -a < in > out || fail=1
Ondrej Oprala 32b1e5
+compare exp out > /dev/null 2>&1 || fail=1
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+#non-Unicode characters interspersed between Unicode ones
Ondrej Oprala 32b1e5
+env printf '12345678
Ondrej Oprala 32b1e5
+        \xFF|
Ondrej Oprala 32b1e5
+\xFF       |
Ondrej Oprala 32b1e5
+        \xFFä|
Ondrej Oprala 32b1e5
+ä\xFF      |
Ondrej Oprala 32b1e5
+        ä\xFF|
Ondrej Oprala 32b1e5
+\xFF       ä|
Ondrej Oprala 32b1e5
+äbcdef\xFF |
Ondrej Oprala 32b1e5
+' > in || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+env printf '12345678
Ondrej Oprala 32b1e5
+\t\xFF|
Ondrej Oprala 32b1e5
+\xFF\t|
Ondrej Oprala 32b1e5
+\t\xFFä|
Ondrej Oprala 32b1e5
+ä\xFF\t|
Ondrej Oprala 32b1e5
+\tä\xFF|
Ondrej Oprala 32b1e5
+\xFF\tä|
Ondrej Oprala 32b1e5
+äbcdef\xFF\t|
Ondrej Oprala 32b1e5
+' > exp || framework_failure_
Ondrej Oprala 32b1e5
+
Ondrej Oprala 32b1e5
+unexpand -a < in > out || fail=1
Ondrej Oprala 32b1e5
+compare exp out > /dev/null 2>&1 || fail=1
Kamil Dudka d3849c
-- 
Kamil Dudka 8d9eac
2.7.4
Kamil Dudka d3849c