search

KomputeProject / kompute

General purpose GPU compute framework built on Vulkan to support 1000s of cross vendor graphics cards (AMD, Qualcomm, NVIDIA & friends). Blazing fast, mobile-enabled, asynchronous and optimized for advanced GPU data processing usecases. Backed by the Linux Foundation.
http://kompute.cc/
Apache License 2.0
1.94k stars 146 forks source link

Fixed compiler warnings #289

Closed COM8 closed 2 years ago

COM8 commented 2 years ago

In preparation for the build system rewrite where -Wall -Wextra -Wpedantic -Werror will be set by default, this PR fixes all warnings reported.

I removed

if (this->mQueueIndex < 0) {
    throw std::runtime_error("Kompute Sequence queue index not provided");
}

without any replacement since mQueueIndex will be always set in the constructor anyway.

Fixes: https://github.com/KomputeProject/kompute/issues/278

axsaucedo commented 2 years ago

Nice - this is really good thank you @COM8 , we can now close #278