Skip to content

Commit 41678fe

Browse files
committed
Fixed checkMasks in DescriptorMatcher with train descs in UMats
1 parent 5bc6b6f commit 41678fe

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

modules/features2d/src/matchers.cpp

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -622,15 +622,20 @@ void DescriptorMatcher::checkMasks( InputArrayOfArrays _masks, int queryDescript
622622
if( isMaskSupported() && !masks.empty() )
623623
{
624624
// Check masks
625-
size_t imageCount = std::max(trainDescCollection.size(), utrainDescCollection.size() );
625+
const size_t imageCount = std::max(trainDescCollection.size(), utrainDescCollection.size() );
626626
CV_Assert( masks.size() == imageCount );
627627
for( size_t i = 0; i < imageCount; i++ )
628628
{
629-
if( !masks[i].empty() && (!trainDescCollection[i].empty() || !utrainDescCollection[i].empty() ) )
629+
if (masks[i].empty())
630+
continue;
631+
const bool hasTrainDesc = !trainDescCollection.empty() && !trainDescCollection[i].empty();
632+
const bool hasUTrainDesc = !utrainDescCollection.empty() && !utrainDescCollection[i].empty();
633+
if (hasTrainDesc || hasUTrainDesc)
630634
{
631-
int rows = trainDescCollection[i].empty() ? utrainDescCollection[i].rows : trainDescCollection[i].rows;
632-
CV_Assert( masks[i].rows == queryDescriptorsCount &&
633-
masks[i].cols == rows && masks[i].type() == CV_8UC1);
635+
const int rows = hasTrainDesc ? trainDescCollection[i].rows : utrainDescCollection[i].rows;
636+
CV_Assert(masks[i].type() == CV_8UC1
637+
&& masks[i].rows == queryDescriptorsCount
638+
&& masks[i].cols == rows);
634639
}
635640
}
636641
}

0 commit comments

Comments
 (0)