On 23.05.2015 15:53, Francisco Jerez wrote:
>> diff --git a/src/gallium/state_trackers/clover/core/resource.cpp 
>> b/src/gallium/state_trackers/clover/core/resource.cpp
>> index 8ed4c42..8e51b3c 100644
>> --- a/src/gallium/state_trackers/clover/core/resource.cpp
>> +++ b/src/gallium/state_trackers/clover/core/resource.cpp
>> @@ -118,6 +118,8 @@ root_resource::root_resource(clover::device &dev, 
>> memory_obj &obj,
>>                               command_queue &q, const std::string &data) :
>>     resource(dev, obj) {
>>     pipe_resource info {};
>> +   bool user_ptr_support = (bool)dev.pipe->get_param(dev.pipe,
> 
> Let's make this variable "const", and remove the cast to bool, it
> shouldn't be necessary.  With that fixed:
> Reviewed-by: Francisco Jerez <curroje...@riseup.net>
>

Thanks for the review. Pushed with minor changes.

Grigori

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to