[all-commits] [llvm/llvm-project] d95597: [libc++][string] Remove potential non-trailing 0-l...

Louis Dionne via All-commits all-commits at lists.llvm.org
Mon Sep 16 13:44:01 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d95597dc06c510ad7fbf00a43583c54d38f79aa7
      https://github.com/llvm/llvm-project/commit/d95597dc06c510ad7fbf00a43583c54d38f79aa7
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2024-09-16 (Mon, 16 Sep 2024)

  Changed paths:
    M libcxx/include/string

  Log Message:
  -----------
  [libc++][string] Remove potential non-trailing 0-length array (#108867)

It is a violation of the standard to use 0 length arrays, especially
when not at the end of a structure (not a FAM GNU extension). Compiler
generally accept it, but it's probably better to have a conforming
implementation.

This is a re-application of #105865 which was reverted in 72cfc74
because it broke the data formatters. A LLDB patch has since been landed
that should make this a non-issue.

Co-authored-by: serge-sans-paille <sguelton at mozilla.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list