[libcxx-commits] [libcxx] 521c611 - [libc++][NFC] Run clang-format on strstream.cpp

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Fri Sep 8 05:50:24 PDT 2023


Author: Louis Dionne
Date: 2023-09-08T08:50:19-04:00
New Revision: 521c6117f66c9cb1fcdd998ff2ae95d7faa0fa64

URL: https://github.com/llvm/llvm-project/commit/521c6117f66c9cb1fcdd998ff2ae95d7faa0fa64
DIFF: https://github.com/llvm/llvm-project/commit/521c6117f66c9cb1fcdd998ff2ae95d7faa0fa64.diff

LOG: [libc++][NFC] Run clang-format on strstream.cpp

I'm about to land https://reviews.llvm.org/D112956 which touches many
lines in that file anyway, so we might as well clang-format it first.

Added: 
    

Modified: 
    libcxx/src/strstream.cpp
    libcxx/utils/data/ignore_format.txt

Removed: 
    


################################################################################
diff  --git a/libcxx/src/strstream.cpp b/libcxx/src/strstream.cpp
index bc56cafa8f80337..a4c949ccb196c46 100644
--- a/libcxx/src/strstream.cpp
+++ b/libcxx/src/strstream.cpp
@@ -20,320 +20,239 @@ _LIBCPP_PUSH_MACROS
 _LIBCPP_BEGIN_NAMESPACE_STD
 
 strstreambuf::strstreambuf(streamsize __alsize)
-    : __strmode_(__dynamic),
-      __alsize_(__alsize),
-      __palloc_(nullptr),
-      __pfree_(nullptr)
-{
-}
+    : __strmode_(__dynamic), __alsize_(__alsize), __palloc_(nullptr), __pfree_(nullptr) {}
 
 strstreambuf::strstreambuf(void* (*__palloc)(size_t), void (*__pfree)(void*))
-    : __strmode_(__dynamic),
-      __alsize_(__default_alsize),
-      __palloc_(__palloc),
-      __pfree_(__pfree)
-{
-}
+    : __strmode_(__dynamic), __alsize_(__default_alsize), __palloc_(__palloc), __pfree_(__pfree) {}
 
-void
-strstreambuf::__init(char* __gnext, streamsize __n, char* __pbeg)
-{
-    if (__n == 0)
-        __n = static_cast<streamsize>(strlen(__gnext));
-    else if (__n < 0)
-        __n = INT_MAX;
-    if (__pbeg == nullptr)
-        setg(__gnext, __gnext, __gnext + __n);
-    else
-    {
-        setg(__gnext, __gnext, __pbeg);
-        setp(__pbeg, __pbeg + __n);
-    }
+void strstreambuf::__init(char* __gnext, streamsize __n, char* __pbeg) {
+  if (__n == 0)
+    __n = static_cast<streamsize>(strlen(__gnext));
+  else if (__n < 0)
+    __n = INT_MAX;
+  if (__pbeg == nullptr)
+    setg(__gnext, __gnext, __gnext + __n);
+  else {
+    setg(__gnext, __gnext, __pbeg);
+    setp(__pbeg, __pbeg + __n);
+  }
 }
 
 strstreambuf::strstreambuf(char* __gnext, streamsize __n, char* __pbeg)
-    : __strmode_(),
-      __alsize_(__default_alsize),
-      __palloc_(nullptr),
-      __pfree_(nullptr)
-{
-    __init(__gnext, __n, __pbeg);
+    : __strmode_(), __alsize_(__default_alsize), __palloc_(nullptr), __pfree_(nullptr) {
+  __init(__gnext, __n, __pbeg);
 }
 
 strstreambuf::strstreambuf(const char* __gnext, streamsize __n)
-    : __strmode_(__constant),
-      __alsize_(__default_alsize),
-      __palloc_(nullptr),
-      __pfree_(nullptr)
-{
-    __init(const_cast<char *>(__gnext), __n, nullptr);
+    : __strmode_(__constant), __alsize_(__default_alsize), __palloc_(nullptr), __pfree_(nullptr) {
+  __init(const_cast<char*>(__gnext), __n, nullptr);
 }
 
 strstreambuf::strstreambuf(signed char* __gnext, streamsize __n, signed char* __pbeg)
-    : __strmode_(),
-      __alsize_(__default_alsize),
-      __palloc_(nullptr),
-      __pfree_(nullptr)
-{
-    __init(const_cast<char *>(reinterpret_cast<const char*>(__gnext)), __n, reinterpret_cast<char*>(__pbeg));
+    : __strmode_(), __alsize_(__default_alsize), __palloc_(nullptr), __pfree_(nullptr) {
+  __init(const_cast<char*>(reinterpret_cast<const char*>(__gnext)), __n, reinterpret_cast<char*>(__pbeg));
 }
 
 strstreambuf::strstreambuf(const signed char* __gnext, streamsize __n)
-    : __strmode_(__constant),
-      __alsize_(__default_alsize),
-      __palloc_(nullptr),
-      __pfree_(nullptr)
-{
-    __init(const_cast<char *>(reinterpret_cast<const char*>(__gnext)), __n, nullptr);
+    : __strmode_(__constant), __alsize_(__default_alsize), __palloc_(nullptr), __pfree_(nullptr) {
+  __init(const_cast<char*>(reinterpret_cast<const char*>(__gnext)), __n, nullptr);
 }
 
 strstreambuf::strstreambuf(unsigned char* __gnext, streamsize __n, unsigned char* __pbeg)
-    : __strmode_(),
-      __alsize_(__default_alsize),
-      __palloc_(nullptr),
-      __pfree_(nullptr)
-{
-    __init(const_cast<char *>(reinterpret_cast<const char*>(__gnext)), __n, reinterpret_cast<char*>(__pbeg));
+    : __strmode_(), __alsize_(__default_alsize), __palloc_(nullptr), __pfree_(nullptr) {
+  __init(const_cast<char*>(reinterpret_cast<const char*>(__gnext)), __n, reinterpret_cast<char*>(__pbeg));
 }
 
 strstreambuf::strstreambuf(const unsigned char* __gnext, streamsize __n)
-    : __strmode_(__constant),
-      __alsize_(__default_alsize),
-      __palloc_(nullptr),
-      __pfree_(nullptr)
-{
-    __init(const_cast<char *>(reinterpret_cast<const char*>(__gnext)), __n, nullptr);
+    : __strmode_(__constant), __alsize_(__default_alsize), __palloc_(nullptr), __pfree_(nullptr) {
+  __init(const_cast<char*>(reinterpret_cast<const char*>(__gnext)), __n, nullptr);
 }
 
-strstreambuf::~strstreambuf()
-{
-    if (eback() && (__strmode_ & __allocated) != 0 && (__strmode_ & __frozen) == 0)
-    {
-        if (__pfree_)
-            __pfree_(eback());
-        else
-            delete [] eback();
-    }
+strstreambuf::~strstreambuf() {
+  if (eback() && (__strmode_ & __allocated) != 0 && (__strmode_ & __frozen) == 0) {
+    if (__pfree_)
+      __pfree_(eback());
+    else
+      delete[] eback();
+  }
 }
 
-void
-strstreambuf::swap(strstreambuf& __rhs)
-{
-    streambuf::swap(__rhs);
-    _VSTD::swap(__strmode_, __rhs.__strmode_);
-    _VSTD::swap(__alsize_, __rhs.__alsize_);
-    _VSTD::swap(__palloc_, __rhs.__palloc_);
-    _VSTD::swap(__pfree_, __rhs.__pfree_);
+void strstreambuf::swap(strstreambuf& __rhs) {
+  streambuf::swap(__rhs);
+  _VSTD::swap(__strmode_, __rhs.__strmode_);
+  _VSTD::swap(__alsize_, __rhs.__alsize_);
+  _VSTD::swap(__palloc_, __rhs.__palloc_);
+  _VSTD::swap(__pfree_, __rhs.__pfree_);
 }
 
-void
-strstreambuf::freeze(bool __freezefl)
-{
-    if (__strmode_ & __dynamic)
-    {
-        if (__freezefl)
-            __strmode_ |= __frozen;
-        else
-            __strmode_ &= ~__frozen;
-    }
+void strstreambuf::freeze(bool __freezefl) {
+  if (__strmode_ & __dynamic) {
+    if (__freezefl)
+      __strmode_ |= __frozen;
+    else
+      __strmode_ &= ~__frozen;
+  }
 }
 
-char*
-strstreambuf::str()
-{
-    if (__strmode_ & __dynamic)
-        __strmode_ |= __frozen;
-    return eback();
+char* strstreambuf::str() {
+  if (__strmode_ & __dynamic)
+    __strmode_ |= __frozen;
+  return eback();
 }
 
-int
-strstreambuf::pcount() const
-{
-    return static_cast<int>(pptr() - pbase());
-}
+int strstreambuf::pcount() const { return static_cast<int>(pptr() - pbase()); }
 
-strstreambuf::int_type
-strstreambuf::overflow(int_type __c)
-{
-    if (__c == EOF)
-        return int_type(0);
-    if (pptr() == epptr())
-    {
-        if ((__strmode_ & __dynamic) == 0 || (__strmode_ & __frozen) != 0)
-            return int_type(EOF);
-        size_t old_size = static_cast<size_t> ((epptr() ? epptr() : egptr()) - eback());
-        size_t new_size = max<size_t>(static_cast<size_t>(__alsize_), 2*old_size);
-        if (new_size == 0)
-            new_size = __default_alsize;
-        char* buf = nullptr;
-        if (__palloc_)
-            buf = static_cast<char*>(__palloc_(new_size));
-        else
-            buf = new char[new_size];
-        if (buf == nullptr)
-            return int_type(EOF);
-        if (old_size != 0) {
-            _LIBCPP_ASSERT_UNCATEGORIZED(eback(), "overflow copying from NULL");
-            memcpy(buf, eback(), static_cast<size_t>(old_size));
-        }
-        ptr
diff _t ninp = gptr()  - eback();
-        ptr
diff _t einp = egptr() - eback();
-        ptr
diff _t nout = pptr()  - pbase();
-        if (__strmode_ & __allocated)
-        {
-            if (__pfree_)
-                __pfree_(eback());
-            else
-                delete [] eback();
-        }
-        setg(buf, buf + ninp, buf + einp);
-        setp(buf + einp, buf + new_size);
-        __pbump(nout);
-        __strmode_ |= __allocated;
+strstreambuf::int_type strstreambuf::overflow(int_type __c) {
+  if (__c == EOF)
+    return int_type(0);
+  if (pptr() == epptr()) {
+    if ((__strmode_ & __dynamic) == 0 || (__strmode_ & __frozen) != 0)
+      return int_type(EOF);
+    size_t old_size = static_cast<size_t>((epptr() ? epptr() : egptr()) - eback());
+    size_t new_size = max<size_t>(static_cast<size_t>(__alsize_), 2 * old_size);
+    if (new_size == 0)
+      new_size = __default_alsize;
+    char* buf = nullptr;
+    if (__palloc_)
+      buf = static_cast<char*>(__palloc_(new_size));
+    else
+      buf = new char[new_size];
+    if (buf == nullptr)
+      return int_type(EOF);
+    if (old_size != 0) {
+      _LIBCPP_ASSERT_UNCATEGORIZED(eback(), "overflow copying from NULL");
+      memcpy(buf, eback(), static_cast<size_t>(old_size));
+    }
+    ptr
diff _t ninp = gptr() - eback();
+    ptr
diff _t einp = egptr() - eback();
+    ptr
diff _t nout = pptr() - pbase();
+    if (__strmode_ & __allocated) {
+      if (__pfree_)
+        __pfree_(eback());
+      else
+        delete[] eback();
     }
-    *pptr() = static_cast<char>(__c);
-    pbump(1);
-    return int_type(static_cast<unsigned char>(__c));
+    setg(buf, buf + ninp, buf + einp);
+    setp(buf + einp, buf + new_size);
+    __pbump(nout);
+    __strmode_ |= __allocated;
+  }
+  *pptr() = static_cast<char>(__c);
+  pbump(1);
+  return int_type(static_cast<unsigned char>(__c));
 }
 
-strstreambuf::int_type
-strstreambuf::pbackfail(int_type __c)
-{
-    if (eback() == gptr())
-        return EOF;
-    if (__c == EOF)
-    {
-        gbump(-1);
-        return int_type(0);
-    }
-    if (__strmode_ & __constant)
-    {
-        if (gptr()[-1] == static_cast<char>(__c))
-        {
-            gbump(-1);
-            return __c;
-        }
-        return EOF;
-    }
+strstreambuf::int_type strstreambuf::pbackfail(int_type __c) {
+  if (eback() == gptr())
+    return EOF;
+  if (__c == EOF) {
     gbump(-1);
-    *gptr() = static_cast<char>(__c);
-    return __c;
+    return int_type(0);
+  }
+  if (__strmode_ & __constant) {
+    if (gptr()[-1] == static_cast<char>(__c)) {
+      gbump(-1);
+      return __c;
+    }
+    return EOF;
+  }
+  gbump(-1);
+  *gptr() = static_cast<char>(__c);
+  return __c;
 }
 
-strstreambuf::int_type
-strstreambuf::underflow()
-{
-    if (gptr() == egptr())
-    {
-        if (egptr() >= pptr())
-            return EOF;
-        setg(eback(), gptr(), pptr());
-    }
-    return int_type(static_cast<unsigned char>(*gptr()));
+strstreambuf::int_type strstreambuf::underflow() {
+  if (gptr() == egptr()) {
+    if (egptr() >= pptr())
+      return EOF;
+    setg(eback(), gptr(), pptr());
+  }
+  return int_type(static_cast<unsigned char>(*gptr()));
 }
 
-strstreambuf::pos_type
-strstreambuf::seekoff(off_type __off, ios_base::seekdir __way, ios_base::openmode __which)
-{
-    off_type __p(-1);
-    bool pos_in = (__which & ios::in) != 0;
-    bool pos_out = (__which & ios::out) != 0;
-    bool legal = false;
-    switch (__way)
-    {
+strstreambuf::pos_type strstreambuf::seekoff(off_type __off, ios_base::seekdir __way, ios_base::openmode __which) {
+  off_type __p(-1);
+  bool pos_in  = (__which & ios::in) != 0;
+  bool pos_out = (__which & ios::out) != 0;
+  bool legal   = false;
+  switch (__way) {
+  case ios::beg:
+  case ios::end:
+    if (pos_in || pos_out)
+      legal = true;
+    break;
+  case ios::cur:
+    if (pos_in != pos_out)
+      legal = true;
+    break;
+  }
+  if (pos_in && gptr() == nullptr)
+    legal = false;
+  if (pos_out && pptr() == nullptr)
+    legal = false;
+  if (legal) {
+    off_type newoff;
+    char* seekhigh = epptr() ? epptr() : egptr();
+    switch (__way) {
     case ios::beg:
-    case ios::end:
-        if (pos_in || pos_out)
-            legal = true;
-        break;
+      newoff = 0;
+      break;
     case ios::cur:
-        if (pos_in != pos_out)
-            legal = true;
-        break;
+      newoff = (pos_in ? gptr() : pptr()) - eback();
+      break;
+    case ios::end:
+      newoff = seekhigh - eback();
+      break;
+    default:
+      __libcpp_unreachable();
     }
-    if (pos_in && gptr() == nullptr)
-        legal = false;
-    if (pos_out && pptr() == nullptr)
-        legal = false;
-    if (legal)
-    {
-        off_type newoff;
-        char* seekhigh = epptr() ? epptr() : egptr();
-        switch (__way)
-        {
-        case ios::beg:
-            newoff = 0;
-            break;
-        case ios::cur:
-            newoff = (pos_in ? gptr() : pptr()) - eback();
-            break;
-        case ios::end:
-            newoff = seekhigh - eback();
-            break;
-        default:
-            __libcpp_unreachable();
-        }
-        newoff += __off;
-        if (0 <= newoff && newoff <= seekhigh - eback())
-        {
-            char* newpos = eback() + newoff;
-            if (pos_in)
-                setg(eback(), newpos, _VSTD::max(newpos, egptr()));
-            if (pos_out)
-            {
-                // min(pbase, newpos), newpos, epptr()
-                __off = epptr() - newpos;
-                setp(min(pbase(), newpos), epptr());
-                __pbump((epptr() - pbase()) - __off);
-            }
-            __p = newoff;
-        }
+    newoff += __off;
+    if (0 <= newoff && newoff <= seekhigh - eback()) {
+      char* newpos = eback() + newoff;
+      if (pos_in)
+        setg(eback(), newpos, _VSTD::max(newpos, egptr()));
+      if (pos_out) {
+        // min(pbase, newpos), newpos, epptr()
+        __off = epptr() - newpos;
+        setp(min(pbase(), newpos), epptr());
+        __pbump((epptr() - pbase()) - __off);
+      }
+      __p = newoff;
     }
-    return pos_type(__p);
+  }
+  return pos_type(__p);
 }
 
-strstreambuf::pos_type
-strstreambuf::seekpos(pos_type __sp, ios_base::openmode __which)
-{
-    off_type __p(-1);
-    bool pos_in = (__which & ios::in) != 0;
-    bool pos_out = (__which & ios::out) != 0;
-    if (pos_in || pos_out)
-    {
-        if (!((pos_in && gptr() == nullptr) || (pos_out && pptr() == nullptr)))
-        {
-            off_type newoff = __sp;
-            char* seekhigh = epptr() ? epptr() : egptr();
-            if (0 <= newoff && newoff <= seekhigh - eback())
-            {
-                char* newpos = eback() + newoff;
-                if (pos_in)
-                    setg(eback(), newpos, _VSTD::max(newpos, egptr()));
-                if (pos_out)
-                {
-                    // min(pbase, newpos), newpos, epptr()
-                    off_type temp = epptr() - newpos;
-                    setp(min(pbase(), newpos), epptr());
-                    __pbump((epptr() - pbase()) - temp);
-                }
-                __p = newoff;
-            }
+strstreambuf::pos_type strstreambuf::seekpos(pos_type __sp, ios_base::openmode __which) {
+  off_type __p(-1);
+  bool pos_in  = (__which & ios::in) != 0;
+  bool pos_out = (__which & ios::out) != 0;
+  if (pos_in || pos_out) {
+    if (!((pos_in && gptr() == nullptr) || (pos_out && pptr() == nullptr))) {
+      off_type newoff = __sp;
+      char* seekhigh  = epptr() ? epptr() : egptr();
+      if (0 <= newoff && newoff <= seekhigh - eback()) {
+        char* newpos = eback() + newoff;
+        if (pos_in)
+          setg(eback(), newpos, _VSTD::max(newpos, egptr()));
+        if (pos_out) {
+          // min(pbase, newpos), newpos, epptr()
+          off_type temp = epptr() - newpos;
+          setp(min(pbase(), newpos), epptr());
+          __pbump((epptr() - pbase()) - temp);
         }
+        __p = newoff;
+      }
     }
-    return pos_type(__p);
+  }
+  return pos_type(__p);
 }
 
-istrstream::~istrstream()
-{
-}
+istrstream::~istrstream() {}
 
-ostrstream::~ostrstream()
-{
-}
+ostrstream::~ostrstream() {}
 
-strstream::~strstream()
-{
-}
+strstream::~strstream() {}
 
 _LIBCPP_END_NAMESPACE_STD
 

diff  --git a/libcxx/utils/data/ignore_format.txt b/libcxx/utils/data/ignore_format.txt
index 3caf9d45c783ec4..5d84b3843ef18d3 100644
--- a/libcxx/utils/data/ignore_format.txt
+++ b/libcxx/utils/data/ignore_format.txt
@@ -537,7 +537,6 @@ libcxx/src/random_shuffle.cpp
 libcxx/src/regex.cpp
 libcxx/src/std_stream.h
 libcxx/src/string.cpp
-libcxx/src/strstream.cpp
 libcxx/src/support/ibm/mbsnrtowcs.cpp
 libcxx/src/support/ibm/wcsnrtombs.cpp
 libcxx/src/support/ibm/xlocale_zos.cpp


        


More information about the libcxx-commits mailing list