Apply several comments

Signed-off-by: Dmitry Sidorov <[email protected]>

Author: MrSidims

sycl/include/CL/sycl/properties/buffer_properties.hpp

@@ -43,11 +43,11 @@ class context_bound
 class mem_channel : public detail::PropertyWithData<
                         detail::PropWithDataKind::BufferMemChannel> {
 public:
-  mem_channel(uint32_t Channel) : m_Channel(Channel) {}
-  uint32_t get_channel() const { return m_Channel; }
+  mem_channel(uint32_t Channel) : MChannel(Channel) {}
+  uint32_t get_channel() const { return MChannel; }
 
 private:
-  uint32_t m_Channel;
+  uint32_t MChannel;
 };
 
 } // namespace buffer

sycl/plugins/opencl/pi_opencl.cpp

@@ -572,19 +572,18 @@ pi_result piMemBufferCreate(pi_context context, pi_mem_flags flags, size_t size,
                                                properties + propSize);
       // Go through buffer properties. If there is one, that shall be propagated
       // to an OpenCL runtime - check if this property is being supported.
-      for (const auto &prop = supported.begin(); prop != supported.end();
-           ++(*prop)) {
+      for (auto prop = supported.begin(); prop != supported.end(); ++prop) {
         if (SupportedExtensions.find(context) == SupportedExtensions.end())
           ret_err = getSupportedExtensionsWithinContext(context);
         // Check if PI_MEM_PROPERTIES_CHANNEL property is supported. If it's
         // not - just ignore it, as it's an optimization hint.
         if (*prop == PI_MEM_PROPERTIES_CHANNEL) {
           if (SupportedExtensions[context].find(
-                  "cl_intel_mem_channel_property") !=
+                  "cl_intel_mem_channel_property") ==
               SupportedExtensions[context].end())
-            supported.erase(prop);
+            prop = supported.erase(prop);
         } else
-          assert("Unsupported property found");
+          assert(!"Unsupported property found");
       }
       if (!supported.empty()) {
         *ret_mem =