Skip to content

Commit 3e3f66a

Browse files
committed
Renderable : Remove render() method
This was used to render via the ancient `IECoreScene::Renderer` class, which has been superseded by a new Renderer class in GafferScene. The new Renderer is just passed objects directly, and doesn't have a method for each object type.
1 parent 1f4b87c commit 3e3f66a

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+1
-364
lines changed

Changes

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ Fixes
1212
Breaking Changes
1313
----------------
1414

15+
- Renderable : Removed `render()` method.
1516
- IECoreHoudini : Removed.
1617
- IECoreMaya : Removed.
1718
- IECoreNuke : Removed.

include/IECoreScene/AttributeState.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,6 @@ class IECORESCENE_API AttributeState : public StateRenderable
6363
/// (it calls readable() or writable() for you).
6464
IECore::CompoundDataPtr attributesData();
6565

66-
/// Calls renderer->setAttribute() for each name, data pair in attributes()
67-
void render( Renderer *renderer ) const override;
68-
6966
private:
7067

7168
IECore::CompoundDataPtr m_attributes;

include/IECoreScene/Camera.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -242,9 +242,6 @@ class IECORESCENE_API Camera : public PreWorldRenderable
242242
/// horizontal field of view ( in degrees )
243243
void setFocalLengthFromFieldOfView( float horizontalFOV );
244244

245-
void render( Renderer *renderer ) const override;
246-
247-
248245
private:
249246

250247
Imath::Box2f defaultApertureRect() const;

include/IECoreScene/ClippingPlane.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@ class IECORESCENE_API ClippingPlane : public PreWorldRenderable
5050

5151
IE_CORE_DECLAREEXTENSIONOBJECT( ClippingPlane, ClippingPlaneTypeId, PreWorldRenderable );
5252

53-
void render( Renderer *renderer ) const override;
54-
5553
};
5654

5755
IE_CORE_DECLAREPTR( ClippingPlane );

include/IECoreScene/CoordinateSystem.h

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -68,12 +68,6 @@ class IECORESCENE_API CoordinateSystem : public StateRenderable
6868
/// Sets the Transform applied to the coordinate system.
6969
void setTransform( TransformPtr transform );
7070

71-
/// Calls renderer->coordinateSystem( name ). If a transform
72-
/// has been applied then also scopes this in a
73-
/// transformBegin/transformEnd with the appropriate
74-
/// transformation.
75-
void render( Renderer *renderer ) const override;
76-
7771
private:
7872

7973
std::string m_name;

include/IECoreScene/CurvesPrimitive.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,6 @@ class IECORESCENE_API CurvesPrimitive : public Primitive
6464
bool periodic() const;
6565
void setTopology( IECore::ConstIntVectorDataPtr verticesPerCurve, const IECore::CubicBasisf &basis, bool periodic );
6666

67-
void render( Renderer *renderer ) const override;
6867
/// Follows the RenderMan specification for variable sizes.
6968
size_t variableSize( PrimitiveVariable::Interpolation interpolation ) const override;
7069
/// Returns the variable size for a single curve.

include/IECoreScene/DiskPrimitive.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@ class IECORESCENE_API DiskPrimitive : public Primitive
6767
size_t variableSize( PrimitiveVariable::Interpolation interpolation ) const override;
6868

6969
Imath::Box3f bound() const override;
70-
void render( Renderer *renderer ) const override;
7170

7271
void topologyHash( IECore::MurmurHash &h ) const override;
7372

include/IECoreScene/ExternalProcedural.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,6 @@ class IECORESCENE_API ExternalProcedural : public VisibleRenderable
6565
IECore::CompoundData *parameters();
6666
const IECore::CompoundData *parameters() const;
6767

68-
void render( Renderer *renderer ) const override;
6968
Imath::Box3f bound() const override;
7069

7170
private :

include/IECoreScene/Group.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -121,18 +121,6 @@ class IECORESCENE_API Group : public VisibleRenderable
121121
Group *parent();
122122
const Group *parent() const;
123123

124-
/// Renders all the children in an attribute block along
125-
/// with the transform and state.
126-
void render( Renderer *renderer ) const override;
127-
/// As above, but can optionally omit the
128-
/// attribute block scoping.
129-
void render( Renderer *renderer, bool inAttributeBlock ) const;
130-
/// Renders all the held state, without any attributeBegin/End.
131-
/// Doesn't render the transform.
132-
void renderState( Renderer *renderer ) const;
133-
/// Renders all the held children, without any attributeBegin/End.
134-
void renderChildren( Renderer *renderer ) const;
135-
136124
/// Returns the union of the bounds of the children, transformed
137125
/// by transformMatrix().
138126
Imath::Box3f bound() const override;

include/IECoreScene/Light.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,6 @@ class IECORESCENE_API Light : public StateRenderable
6868
IECore::CompoundDataPtr parametersData();
6969
const IECore::CompoundDataPtr parametersData() const;
7070

71-
/// Sets this light in Renderer.
72-
void render( Renderer *renderer ) const override;
73-
7471
private:
7572

7673
static TypeDescription<Light> m_description;

0 commit comments

Comments
 (0)