Skip to content

Commit 6d6d2de

Browse files
Merge pull request #1111 from vsg-dev/revert-1110-ref_ptr-nullptr
Revert "Add std::nullptr_t constructor overload for ref_ptr"
2 parents 3555f16 + 91c39b1 commit 6d6d2de

File tree

3 files changed

+4
-7
lines changed

3 files changed

+4
-7
lines changed

include/vsg/core/ref_ptr.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,6 @@ namespace vsg
6060
if (_ptr) _ptr->ref();
6161
}
6262

63-
ref_ptr(std::nullptr_t) noexcept :
64-
_ptr(nullptr) {}
65-
6663
~ref_ptr()
6764
{
6865
if (_ptr) _ptr->unref();

src/vsg/state/DescriptorImage.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ uint32_t DescriptorImage::getNumDescriptors() const
164164

165165
VSG_DECLSPEC ref_ptr<DescriptorImage> vsg::createSamplerDescriptor(ref_ptr<Sampler> sampler, uint32_t dstBinding, uint32_t dstArrayElement)
166166
{
167-
ref_ptr<ImageInfo> imageImageInfo = ImageInfo::create(sampler, nullptr, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL);
167+
ref_ptr<ImageInfo> imageImageInfo = ImageInfo::create(sampler, ref_ptr<ImageView>(), VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL);
168168
return DescriptorImage::create(imageImageInfo, dstBinding, dstArrayElement, VK_DESCRIPTOR_TYPE_SAMPLER);
169169
}
170170

@@ -176,6 +176,6 @@ VSG_DECLSPEC ref_ptr<DescriptorImage> vsg::createCombinedImageSamplerDescriptor(
176176

177177
VSG_DECLSPEC ref_ptr<DescriptorImage> vsg::createSampedImageDescriptor(ref_ptr<Data> image, uint32_t dstBinding, uint32_t dstArrayElement)
178178
{
179-
ref_ptr<ImageInfo> imageImageInfo = ImageInfo::create(nullptr, image, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL);
179+
ref_ptr<ImageInfo> imageImageInfo = ImageInfo::create(ref_ptr<Sampler>(), image, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL);
180180
return DescriptorImage::create(imageImageInfo, dstBinding, dstArrayElement, VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE);
181181
}

src/vsg/state/ViewDependentState.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,7 @@ void ViewDependentState::init(ResourceRequirements& requirements)
299299
depthImageView->subresourceRange.baseArrayLayer = 0;
300300
depthImageView->subresourceRange.layerCount = maxShadowMaps;
301301

302-
auto depthImageInfo = ImageInfo::create(nullptr, depthImageView, VK_IMAGE_LAYOUT_DEPTH_STENCIL_READ_ONLY_OPTIMAL);
302+
auto depthImageInfo = ImageInfo::create(vsg::ref_ptr<Sampler>(), depthImageView, VK_IMAGE_LAYOUT_DEPTH_STENCIL_READ_ONLY_OPTIMAL);
303303

304304
shadowMapImages = DescriptorImage::create(ImageInfoList{depthImageInfo}, 2, 0, VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE);
305305
}
@@ -323,7 +323,7 @@ void ViewDependentState::init(ResourceRequirements& requirements)
323323
depthImageView->subresourceRange.baseArrayLayer = 0;
324324
depthImageView->subresourceRange.layerCount = 1;
325325

326-
auto depthImageInfo = ImageInfo::create(nullptr, depthImageView, VK_IMAGE_LAYOUT_DEPTH_STENCIL_READ_ONLY_OPTIMAL);
326+
auto depthImageInfo = ImageInfo::create(vsg::ref_ptr<Sampler>(), depthImageView, VK_IMAGE_LAYOUT_DEPTH_STENCIL_READ_ONLY_OPTIMAL);
327327

328328
shadowMapImages = DescriptorImage::create(ImageInfoList{depthImageInfo}, 2, 0 , VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE);
329329
}

0 commit comments

Comments
 (0)