Skip to content

Commit 7533342

Browse files
committed
Some include what you use header suggestions arround the ACES code
Signed-off-by: Kevin Wheatley <kevin.wheatley@framestore.com>
1 parent b5569dc commit 7533342

File tree

7 files changed

+29
-0
lines changed

7 files changed

+29
-0
lines changed

src/OpenColorIO/ops/range/RangeOp.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,11 @@
1010
#include "ops/range/RangeOpCPU.h"
1111
#include "ops/range/RangeOpGPU.h"
1212
#include "ops/range/RangeOp.h"
13+
#include "ops/range/RangeOpData.h"
1314
#include "transforms/RangeTransform.h"
1415
#include "ops/matrix/MatrixOp.h"
16+
#include "ops/matrix/MatrixOpData.h"
17+
1518

1619
namespace OCIO_NAMESPACE
1720
{

src/OpenColorIO/ops/range/RangeOpCPU.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,15 @@
33

44

55
#include <algorithm>
6+
#include <memory>
67

78
#include <OpenColorIO/OpenColorIO.h>
89

10+
#include "Op.h"
911
#include "MathUtils.h"
1012
#include "ops/range/RangeOpCPU.h"
13+
#include "ops/range/RangeOpData.h"
14+
1115

1216
namespace OCIO_NAMESPACE
1317
{

src/OpenColorIO/transforms/BuiltinTransform.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33

44

55
#include <sstream>
6+
#include <ostream>
67

78
#include <OpenColorIO/OpenColorIO.h>
89

src/OpenColorIO/transforms/BuiltinTransform.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66
#define INCLUDED_OCIO_BUILTINTRANSFORM_H
77

88

9+
#include <cstddef>
10+
911
#include <OpenColorIO/OpenColorIO.h>
1012

1113

src/OpenColorIO/transforms/builtins/ACES.cpp

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,27 @@
22
// Copyright Contributors to the OpenColorIO Project.
33

44

5+
#include <cstddef>
56
#include <cmath>
67
#include <memory>
78
#include <algorithm>
9+
#include <vector>
10+
#include <string>
811

912
#include <OpenColorIO/OpenColorIO.h>
1013

14+
#include "Op.h"
1115
#include "ops/fixedfunction/FixedFunctionOp.h"
16+
#include "ops/fixedfunction/FixedFunctionOpData.h"
1217
#include "ops/gradingrgbcurve/GradingRGBCurveOp.h"
18+
#include "ops/gradingrgbcurve/GradingRGBCurveOpData.h"
1319
#include "ops/log/LogOp.h"
20+
#include "ops/log/LogOpData.h"
1421
#include "ops/matrix/MatrixOp.h"
22+
#include "ops/matrix/MatrixOpData.h"
1523
#include "ops/range/RangeOp.h"
24+
#include "ops/range/RangeOpData.h"
25+
#include "transforms/builtins/ColorMatrixHelpers.h"
1626
#include "transforms/builtins/ACES.h"
1727
#include "transforms/builtins/BuiltinTransformRegistry.h"
1828
#include "transforms/builtins/OpHelpers.h"

tests/cpu/ops/range/RangeOp_tests.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,12 @@
55
#include <string>
66
#include <memory>
77

8+
#include <OpenColorIO/OpenColorIO.h>
9+
10+
#include "Op.h"
811
#include "ops/range/RangeOp.cpp"
12+
#include "fileformats/FormatMetadata.h"
13+
#include "ops/matrix/MatrixOpData.h"
914

1015
#include "testutils/UnitTest.h"
1116

tests/cpu/transforms/BuiltinTransform_tests.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
// Copyright Contributors to the OpenColorIO Project.
33

44

5+
#include <cstddef>
56
#include <algorithm>
67
#include <cmath>
78
#include <map>
@@ -18,12 +19,15 @@
1819
#include "ops/lut3d/Lut3DOp.h"
1920

2021
#include "transforms/builtins/ColorMatrixHelpers.h"
22+
#include "ops/matrix/MatrixOpData.h"
2123
#include "transforms/builtins/OpHelpers.h"
2224
#include "testutils/UnitTest.h"
2325
#include "UnitTestUtils.h"
2426

27+
2528
namespace OCIO = OCIO_NAMESPACE;
2629

30+
2731
OCIO_ADD_TEST(BuiltinTransform, creation)
2832
{
2933
// Tests around the creation of a built-in transform instance.

0 commit comments

Comments
 (0)