]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
build: revert -Wvla from #15342
authorWillem Jan Withagen <wjw@digiware.nl>
Sun, 4 Jun 2017 19:25:08 +0000 (21:25 +0200)
committerWillem Jan Withagen <wjw@digiware.nl>
Sun, 4 Jun 2017 19:25:08 +0000 (21:25 +0200)
commitbdf524e05be9cb710f4f9e93042d208709b2ee48
tree6e277d7330f103c25badf3dc098eb1b88f6d4fee
parent554cf8394a9ac4f845c1fce03dd1a7f551a414a9
build: revert -Wvla from #15342

 - VLAs are in GCC and Clang, and are there to stay forever,
   if only to be compatible with all the software that is already
   out there.
   - Theoretical debates about VLA being hard to implement are
     long superceded by th actual implentations
 - Before setting this flag is would be required to first start
   work on fixing all the fallout/warnings that will arise from
   setting -Wvla
 - Allocating large variable/stuctures on the stack could be asking
   for trouble, but changes that ceph tools are going to be running
   on small embedded devices are rather slim.

Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>
src/CMakeLists.txt