Page MenuHome
Differential D3568

Build: require C11/C++11 for all operating systems in master.
ClosedPublic
Actions

Authored by Brecht Van Lommel (brecht) on Jul 29 2018, 4:18 PM.

Details

Reviewers
Sergey Sharybin (sergey)
Bastien Montagne (mont29)
Campbell Barton (campbellbarton)
Commits
rC951e3e5d2414: Build: require C11/C++11 for all operating systems in master
rBS885cc4cf9a1d: Build: require C11/C++11 for all operating systems in master.
rB885cc4cf9a1d: Build: require C11/C++11 for all operating systems in master.
Summary

This is in preparation of upgrading our library dependencies, some of which
need C++11. We already use C++11 in blender2.8 and for Windows and macOS, so
this really just affects Linux.

Diff Detail

Repository
rB Blender
Branch
require-cxx11
Build Status
Buildable 1861
Build 1861: arc lint + arc unit

Event Timeline

Brecht Van Lommel (brecht) created this revision.Jul 29 2018, 4:18 PM
Campbell Barton (campbellbarton) accepted this revision.Jul 30 2018, 8:30 AM
This revision is now accepted and ready to land.Jul 30 2018, 8:30 AM
Sergey Sharybin (sergey) requested changes to this revision.Jul 30 2018, 10:08 AM
Sergey Sharybin (sergey) added inline comments.
source/blender/alembic/intern/abc_customdata.cc
29

I would keep uv_index_map as a typedef, otherwise it's ridiculous to type std::unordered_map<uint64_t, int>every time.

This revision now requires changes to proceed.Jul 30 2018, 10:08 AM
Brecht Van Lommel (brecht) updated this revision to Diff 11478.Jul 30 2018, 11:46 AM

Address comment.

Brecht Van Lommel (brecht) marked an inline comment as done.Jul 30 2018, 11:47 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 30 2018, 6:37 PM
Closed by commit rB885cc4cf9a1d: Build: require C11/C++11 for all operating systems in master. (authored by Brecht Van Lommel (brecht)). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
100 lines
build_files/
build_environment/
41 lines
cmake/
158 lines
platform/
2 lines
3 lines
intern/
cycles/
25 lines
util/
10 lines
9 lines
18 lines
27 lines
32 lines
22 lines
24 lines
8 lines
4 lines
source/
blender/
alembic/
intern/
11 lines
blenlib/
6 lines
depsgraph/
22 lines
util/
17 lines
73 lines

Diff 11476

View Options

CMakeLists.txt

Loading...
View Options

build_files/build_environment/install_deps.sh

Loading...
View Options

build_files/cmake/macros.cmake

Loading...
View Options

build_files/cmake/platform/platform_apple_xcode.cmake

Loading...
View Options

build_files/cmake/platform/platform_win32.cmake

Loading...
View Options

intern/cycles/CMakeLists.txt

Loading...
View Options

intern/cycles/util/util_defines.h

Loading...
View Options

intern/cycles/util/util_foreach.h

Loading...
View Options

intern/cycles/util/util_function.h

Loading...
View Options

intern/cycles/util/util_map.h

Loading...
View Options

intern/cycles/util/util_set.h

Loading...
View Options

intern/cycles/util/util_static_assert.h

Loading...
View Options

intern/cycles/util/util_thread.h

Loading...
View Options

intern/cycles/util/util_thread.cpp

Loading...
View Options

intern/cycles/util/util_vector.h

Loading...
View Options

source/blender/alembic/intern/abc_customdata.cc

Loading...
View Options

source/blender/blenlib/BLI_compiler_compat.h

Loading...
View Options

source/blender/depsgraph/CMakeLists.txt

Loading...
View Options

source/blender/depsgraph/util/deg_util_foreach.h

Loading...
View Options

source/blender/depsgraph/util/deg_util_function.h

Loading...