Skip to content

Commit

Permalink
Update HdRenderPassSharedPtr to use std instead of boost.
Browse files Browse the repository at this point in the history
(Internal change: 2051755)
  • Loading branch information
c64kernal authored and pixar-oss committed Mar 30, 2020
1 parent 31b7433 commit 677978c
Show file tree
Hide file tree
Showing 11 changed files with 17 additions and 23 deletions.
2 changes: 1 addition & 1 deletion pxr/imaging/hd/engine.h
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ class HdRenderIndex;
class HdRenderDelegate;
class HdResourceRegistry;

typedef boost::shared_ptr<class HdRenderPass> HdRenderPassSharedPtr;
using HdRenderPassSharedPtr = std::shared_ptr<class HdRenderPass>;
using HdRenderPassStateSharedPtr = std::shared_ptr<class HdRenderPassState>;

/// \class HdEngine
Expand Down
5 changes: 1 addition & 4 deletions pxr/imaging/hd/renderDelegate.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,10 @@
#include "pxr/base/vt/dictionary.h"
#include "pxr/base/tf/token.h"

#include <boost/shared_ptr.hpp>
#include <memory>

PXR_NAMESPACE_OPEN_SCOPE


class SdfPath;
class HdRprim;
class HdSprim;
Expand All @@ -47,8 +45,7 @@ class HdRenderPass;
class HdInstancer;
class HdDriver;

typedef boost::shared_ptr<class HdRenderPass> HdRenderPassSharedPtr;

using HdRenderPassSharedPtr = std::shared_ptr<class HdRenderPass>;
using HdRenderPassStateSharedPtr = std::shared_ptr<class HdRenderPassState>;
using HdResourceRegistrySharedPtr = std::shared_ptr<class HdResourceRegistry>;
using HdDriverVector = std::vector<HdDriver*>;
Expand Down
5 changes: 1 addition & 4 deletions pxr/imaging/hd/renderPass.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,6 @@
#include "pxr/imaging/hd/rprimCollection.h"
#include "pxr/imaging/hd/task.h"

#include <boost/shared_ptr.hpp>

#include <memory>

PXR_NAMESPACE_OPEN_SCOPE
Expand All @@ -41,10 +39,9 @@ class HdRenderIndex;
class HdSceneDelegate;

using HdDirtyListSharedPtr = std::shared_ptr<class HdDirtyList>;
using HdRenderPassSharedPtr = std::shared_ptr<class HdRenderPass>;
using HdRenderPassStateSharedPtr = std::shared_ptr<class HdRenderPassState>;

typedef boost::shared_ptr<class HdRenderPass> HdRenderPassSharedPtr;

/// \class HdRenderPass
///
/// An abstract class representing a single render iteration over a set of prims
Expand Down
6 changes: 3 additions & 3 deletions pxr/imaging/hdSt/imageShaderRenderPass.h
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,8 @@ PXR_NAMESPACE_OPEN_SCOPE
using HdStResourceRegistrySharedPtr =
std::shared_ptr<class HdStResourceRegistry>;

typedef boost::shared_ptr<class HdSt_ImageShaderRenderPass>
HdSt_ImageShaderRenderPassSharedPtr;
using HdSt_ImageShaderRenderPassSharedPtr =
std::shared_ptr<class HdSt_ImageShaderRenderPass>;
typedef boost::shared_ptr<class HdSt_DrawBatch>
HdSt_DrawBatchSharedPtr;

Expand All @@ -53,7 +53,7 @@ class Hgi;
/// The benefit of using RenderPassShader is that it participates in codegen.
/// This means your full-screen shader can use buffers created by other tasks.
///
class HdSt_ImageShaderRenderPass : public HdRenderPass {
class HdSt_ImageShaderRenderPass final : public HdRenderPass {
public:
HDST_API
HdSt_ImageShaderRenderPass(HdRenderIndex *index,
Expand Down
2 changes: 1 addition & 1 deletion pxr/imaging/hdx/oitResolveTask.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,7 @@ HdxOitResolveTask::Prepare(HdTaskContext* ctx,
return;
}

_renderPass = boost::make_shared<HdSt_ImageShaderRenderPass>(
_renderPass = std::make_shared<HdSt_ImageShaderRenderPass>(
renderIndex, collection);

// We do not use renderDelegate->CreateRenderPassState because
Expand Down
3 changes: 1 addition & 2 deletions pxr/imaging/hdx/oitResolveTask.h
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,7 @@ class HdSceneDelegate;

using HdStRenderPassStateSharedPtr = std::shared_ptr<class HdStRenderPassState>;

typedef boost::shared_ptr<class HdRenderPass>
HdRenderPassSharedPtr;
using HdRenderPassSharedPtr = std::shared_ptr<class HdRenderPass>;
typedef boost::shared_ptr<class HdStRenderPassShader>
HdStRenderPassShaderSharedPtr;

Expand Down
2 changes: 1 addition & 1 deletion pxr/imaging/hdx/renderTask.h
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ PXR_NAMESPACE_OPEN_SCOPE
class HdSceneDelegate;

using HdRenderPassStateSharedPtr = std::shared_ptr<class HdRenderPassState>;
typedef boost::shared_ptr<class HdRenderPass> HdRenderPassSharedPtr;
using HdRenderPassSharedPtr = std::shared_ptr<class HdRenderPass>;
typedef boost::shared_ptr<class HdxRenderSetupTask> HdxRenderSetupTaskSharedPtr;

/// \class HdxRenderTask
Expand Down
2 changes: 1 addition & 1 deletion pxr/imaging/hdx/shadowTask.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ HdxShadowTask::Sync(HdSceneDelegate* delegate,
if (_passes[shadowId]) {
_passes[shadowId]->SetRprimCollection(col);
} else {
_passes[shadowId] = boost::make_shared<HdSt_RenderPass>
_passes[shadowId] = std::make_shared<HdSt_RenderPass>
(&renderIndex, col);
}
}
Expand Down
4 changes: 2 additions & 2 deletions pxr/imaging/hdx/shadowTask.h
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,9 @@ class HdStRenderPassState;

typedef boost::shared_ptr<class HdStRenderPassShader> HdStRenderPassShaderSharedPtr;
typedef boost::shared_ptr<class HdStShaderCode> HdStShaderCodeSharedPtr;
typedef boost::shared_ptr<class HdRenderPass> HdRenderPassSharedPtr;
typedef std::vector<HdRenderPassSharedPtr> HdRenderPassSharedPtrVector;

using HdRenderPassSharedPtr = std::shared_ptr<class HdRenderPass>;
using HdRenderPassSharedPtrVector = std::vector<HdRenderPassSharedPtr>;
using HdRenderPassStateSharedPtr = std::shared_ptr<class HdRenderPassState>;
using HdRenderPassStateSharedPtrVector =
std::vector<HdRenderPassStateSharedPtr>;
Expand Down
4 changes: 3 additions & 1 deletion pxr/imaging/hdx/simpleLightTask.h
Original file line number Diff line number Diff line change
Expand Up @@ -40,12 +40,14 @@

#include <boost/shared_ptr.hpp>

#include <memory>

PXR_NAMESPACE_OPEN_SCOPE

class HdRenderIndex;
class HdSceneDelegate;

typedef boost::shared_ptr<class HdRenderPass> HdRenderPassSharedPtr;
using HdRenderPassSharedPtr = std::shared_ptr<class HdRenderPass>;
typedef boost::shared_ptr<class HdStSimpleLightingShader>
HdStSimpleLightingShaderSharedPtr;
typedef boost::shared_ptr<class HdxShadowMatrixComputation>
Expand Down
5 changes: 2 additions & 3 deletions pxr/usdImaging/usdImaging/unitTestHelper.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,13 +39,12 @@
#include "pxr/imaging/hd/unitTestNullRenderDelegate.h"
#include "pxr/imaging/hd/unitTestNullRenderPass.h"

#include <memory>
#include <string>
#include <boost/shared_ptr.hpp>

PXR_NAMESPACE_OPEN_SCOPE


typedef boost::shared_ptr<HdRenderPass> HdRenderPassSharedPtr;
using HdRenderPassSharedPtr = std::shared_ptr<HdRenderPass>;

/// A simple test task that just causes sync processing
class UsdImaging_TestTask final : public HdTask
Expand Down

0 comments on commit 677978c

Please sign in to comment.