Minor code changes to PR
This commit is contained in:
@@ -827,10 +827,8 @@ bool TextureTool::ImportTextureDirectXTex(ImageType type, const StringView& path
|
|||||||
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
||||||
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
||||||
{
|
{
|
||||||
static const DirectX::XMVECTORU32 s_selecty = { { { DirectX::XM_SELECT_0, DirectX::XM_SELECT_1, DirectX::XM_SELECT_0, DirectX::XM_SELECT_0 } } };
|
const DirectX::XMVECTORU32 s_selecty = { { { DirectX::XM_SELECT_0, DirectX::XM_SELECT_1, DirectX::XM_SELECT_0, DirectX::XM_SELECT_0 } } };
|
||||||
|
|
||||||
UNREFERENCED_PARAMETER(y);
|
UNREFERENCED_PARAMETER(y);
|
||||||
|
|
||||||
for (size_t j = 0; j < w; ++j)
|
for (size_t j = 0; j < w; ++j)
|
||||||
{
|
{
|
||||||
const DirectX::XMVECTOR value = inPixels[j];
|
const DirectX::XMVECTOR value = inPixels[j];
|
||||||
@@ -853,10 +851,8 @@ bool TextureTool::ImportTextureDirectXTex(ImageType type, const StringView& path
|
|||||||
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
||||||
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
||||||
{
|
{
|
||||||
static const DirectX::XMVECTORU32 s_selectx = { { { DirectX::XM_SELECT_1, DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_0 } } };
|
const DirectX::XMVECTORU32 s_selectx = { { { DirectX::XM_SELECT_1, DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_0 } } };
|
||||||
|
|
||||||
UNREFERENCED_PARAMETER(y);
|
UNREFERENCED_PARAMETER(y);
|
||||||
|
|
||||||
for (size_t j = 0; j < w; ++j)
|
for (size_t j = 0; j < w; ++j)
|
||||||
{
|
{
|
||||||
const DirectX::XMVECTOR value = inPixels[j];
|
const DirectX::XMVECTOR value = inPixels[j];
|
||||||
@@ -879,10 +875,8 @@ bool TextureTool::ImportTextureDirectXTex(ImageType type, const StringView& path
|
|||||||
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
||||||
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
||||||
{
|
{
|
||||||
static const DirectX::XMVECTORU32 s_selectz = { { { DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_1, DirectX::XM_SELECT_0 } } };
|
const DirectX::XMVECTORU32 s_selectz = { { { DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_1, DirectX::XM_SELECT_0 } } };
|
||||||
|
|
||||||
UNREFERENCED_PARAMETER(y);
|
UNREFERENCED_PARAMETER(y);
|
||||||
|
|
||||||
for (size_t j = 0; j < w; ++j)
|
for (size_t j = 0; j < w; ++j)
|
||||||
{
|
{
|
||||||
const DirectX::XMVECTOR value = inPixels[j];
|
const DirectX::XMVECTOR value = inPixels[j];
|
||||||
@@ -905,10 +899,8 @@ bool TextureTool::ImportTextureDirectXTex(ImageType type, const StringView& path
|
|||||||
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
||||||
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
||||||
{
|
{
|
||||||
static const DirectX::XMVECTORU32 s_selectw = { { { DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_1 } } };
|
const DirectX::XMVECTORU32 s_selectw = { { { DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_1 } } };
|
||||||
|
|
||||||
UNREFERENCED_PARAMETER(y);
|
UNREFERENCED_PARAMETER(y);
|
||||||
|
|
||||||
for (size_t j = 0; j < w; ++j)
|
for (size_t j = 0; j < w; ++j)
|
||||||
{
|
{
|
||||||
const DirectX::XMVECTOR value = inPixels[j];
|
const DirectX::XMVECTOR value = inPixels[j];
|
||||||
@@ -932,10 +924,8 @@ bool TextureTool::ImportTextureDirectXTex(ImageType type, const StringView& path
|
|||||||
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
result = TransformImage(currentImage->GetImages(), currentImage->GetImageCount(), currentImage->GetMetadata(),
|
||||||
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
[&](DirectX::XMVECTOR* outPixels, const DirectX::XMVECTOR* inPixels, size_t w, size_t y)
|
||||||
{
|
{
|
||||||
static const DirectX::XMVECTORU32 s_selectz = { { { DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_1, DirectX::XM_SELECT_0 } } };
|
const DirectX::XMVECTORU32 s_selectz = { { { DirectX::XM_SELECT_0, DirectX::XM_SELECT_0, DirectX::XM_SELECT_1, DirectX::XM_SELECT_0 } } };
|
||||||
|
|
||||||
UNREFERENCED_PARAMETER(y);
|
UNREFERENCED_PARAMETER(y);
|
||||||
|
|
||||||
for (size_t j = 0; j < w; ++j)
|
for (size_t j = 0; j < w; ++j)
|
||||||
{
|
{
|
||||||
const DirectX::XMVECTOR value = inPixels[j];
|
const DirectX::XMVECTOR value = inPixels[j];
|
||||||
|
|||||||
@@ -541,10 +541,10 @@ bool TextureTool::ImportTextureStb(ImageType type, const StringView& path, Textu
|
|||||||
// TODO: impl this
|
// TODO: impl this
|
||||||
LOG(Warning, "Option 'Flip X' is not supported");
|
LOG(Warning, "Option 'Flip X' is not supported");
|
||||||
}
|
}
|
||||||
if (options.InvertGreenChannel)
|
if (options.InvertRedChannel || options.InvertGreenChannel || options.InvertBlueChannel || options.InvertAlphaChannel)
|
||||||
{
|
{
|
||||||
// TODO: impl this
|
// TODO: impl this
|
||||||
LOG(Warning, "Option 'Invert Green Channel' is not supported");
|
LOG(Warning, "Option to invert channels is not supported");
|
||||||
}
|
}
|
||||||
if (options.ReconstructZChannel)
|
if (options.ReconstructZChannel)
|
||||||
{
|
{
|
||||||
|
|||||||
Reference in New Issue
Block a user