Cycles: Cleanup, don't use strict C prototypes
authorSergey Sharybin <sergey.vfx@gmail.com>
Fri, 9 Nov 2018 11:01:38 +0000 (12:01 +0100)
committerSergey Sharybin <sergey.vfx@gmail.com>
Fri, 9 Nov 2018 11:04:41 +0000 (12:04 +0100)
Those are more like a legacy of language, which is not
needed in C++.

29 files changed:
intern/cycles/blender/blender_logging.cpp
intern/cycles/device/device.h
intern/cycles/device/device_cpu.cpp
intern/cycles/device/device_cuda.cpp
intern/cycles/device/device_intern.h
intern/cycles/device/device_opencl.cpp
intern/cycles/device/device_task.h
intern/cycles/util/util_avxf.h
intern/cycles/util/util_guarded_allocator.cpp
intern/cycles/util/util_guarded_allocator.h
intern/cycles/util/util_logging.cpp
intern/cycles/util/util_logging.h
intern/cycles/util/util_progress.h
intern/cycles/util/util_ssef.h
intern/cycles/util/util_thread.cpp
intern/cycles/util/util_thread.h
intern/cycles/util/util_types_float3.h
intern/cycles/util/util_types_float3_impl.h
intern/cycles/util/util_types_float4.h
intern/cycles/util/util_types_float4_impl.h
intern/cycles/util/util_types_float8.h
intern/cycles/util/util_types_float8_impl.h
intern/cycles/util/util_types_int3.h
intern/cycles/util/util_types_int3_impl.h
intern/cycles/util/util_types_int4.h
intern/cycles/util/util_types_int4_impl.h
intern/cycles/util/util_vector.h
intern/cycles/util/util_view.cpp
intern/cycles/util/util_view.h

index d0f82e37662ad9b251ed640f0c40aa9a95b273f3..3fca4efd0975ac951034e7fe4d3b1bb6c8d79587 100644 (file)
@@ -22,7 +22,7 @@ void CCL_init_logging(const char *argv0)
        ccl::util_logging_init(argv0);
 }
 
-void CCL_start_debug_logging(void)
+void CCL_start_debug_logging()
 {
        ccl::util_logging_start();
 }
index f49915bc40d9530aefa56bb01f873a08fe84d95a..f3fb338e63872a262e645f7f72736f06fb4c56eb 100644 (file)
@@ -181,7 +181,7 @@ public:
        /* Convert the requested features structure to a build options,
         * which could then be passed to compilers.
         */
-       string get_build_options(void) const
+       string get_build_options() const
        {
                string build_options = "";
                if(experimental) {
@@ -240,8 +240,8 @@ std::ostream& operator <<(std::ostream &os,
 /* Device */
 
 struct DeviceDrawParams {
-       function<void(void)> bind_display_space_shader_cb;
-       function<void(void)> unbind_display_space_shader_cb;
+       function<void()> bind_display_space_shader_cb;
+       function<void()> unbind_display_space_shader_cb;
 };
 
 class Device {
index bb5e9980457cd9173bf680ce64de427b0a3df014..9b37f4773bb9e8017141283dee15d84de5a0e0d6 100644 (file)
@@ -1065,7 +1065,7 @@ void device_cpu_info(vector<DeviceInfo>& devices)
        devices.insert(devices.begin(), info);
 }
 
-string device_cpu_capabilities(void)
+string device_cpu_capabilities()
 {
        string capabilities = "";
        capabilities += system_cpu_support_sse2() ? "SSE2 " : "";
index a52e77f707b3f8fadbc662faa31422d2d3fd9248..d338a565a02a7479c9b521dec4c861a916288c39 100644 (file)
@@ -73,12 +73,12 @@ const char *cuewErrorString(CUresult result)
        return error.c_str();
 }
 
-const char *cuewCompilerPath(void)
+const char *cuewCompilerPath()
 {
        return CYCLES_CUDA_NVCC_EXECUTABLE;
 }
 
-int cuewCompilerVersion(void)
+int cuewCompilerVersion()
 {
        return (CUDA_VERSION / 100) + (CUDA_VERSION % 100 / 10);
 }
@@ -2355,7 +2355,7 @@ int2 CUDASplitKernel::split_kernel_global_size(device_memory& kg, device_memory&
        return global_size;
 }
 
-bool device_cuda_init(void)
+bool device_cuda_init()
 {
 #ifdef WITH_CUDA_DYNLOAD
        static bool initialized = false;
@@ -2497,7 +2497,7 @@ void device_cuda_info(vector<DeviceInfo>& devices)
                devices.insert(devices.end(), display_devices.begin(), display_devices.end());
 }
 
-string device_cuda_capabilities(void)
+string device_cuda_capabilities()
 {
        CUresult result = device_cuda_safe_init();
        if(result != CUDA_SUCCESS) {
index d5a96c2b82b20a2d876f858b3a12df1660203b49..e6495c2bff39cb3b3ce48baaf99d958300da4c9a 100644 (file)
@@ -22,9 +22,9 @@ CCL_NAMESPACE_BEGIN
 class Device;
 
 Device *device_cpu_create(DeviceInfo& info, Stats &stats, bool background);
-bool device_opencl_init(void);
+bool device_opencl_init();
 Device *device_opencl_create(DeviceInfo& info, Stats &stats, bool background);
-bool device_cuda_init(void);
+bool device_cuda_init();
 Device *device_cuda_create(DeviceInfo& info, Stats &stats, bool background);
 Device *device_network_create(DeviceInfo& info, Stats &stats, const char *address);
 Device *device_multi_create(DeviceInfo& info, Stats &stats, bool background);
@@ -34,9 +34,9 @@ void device_opencl_info(vector<DeviceInfo>& devices);
 void device_cuda_info(vector<DeviceInfo>& devices);
 void device_network_info(vector<DeviceInfo>& devices);
 
-string device_cpu_capabilities(void);
-string device_opencl_capabilities(void);
-string device_cuda_capabilities(void);
+string device_cpu_capabilities();
+string device_opencl_capabilities();
+string device_cuda_capabilities();
 
 CCL_NAMESPACE_END
 
index 036f8a2e6e7e904599dff94089aabfe9827dbccf..71410f80d5711b9ac9fc797f55443c87d215296c 100644 (file)
@@ -44,7 +44,7 @@ Device *device_opencl_create(DeviceInfo& info, Stats &stats, bool background)
        }
 }
 
-bool device_opencl_init(void)
+bool device_opencl_init()
 {
        static bool initialized = false;
        static bool result = false;
@@ -146,7 +146,7 @@ void device_opencl_info(vector<DeviceInfo>& devices)
        }
 }
 
-string device_opencl_capabilities(void)
+string device_opencl_capabilities()
 {
        if(OpenCLInfo::device_type() == 0) {
                return "All OpenCL devices are forced to be OFF";
index db27470a03fe04469ca2c56ddf1c81ff43c4181e..861014373b38eed125c95b863986ff1ebbb6c706 100644 (file)
@@ -64,7 +64,7 @@ public:
        function<void(long, int)> update_progress_sample;
        function<void(RenderTile&)> update_tile_sample;
        function<void(RenderTile&)> release_tile;
-       function<bool(void)> get_cancel;
+       function<bool()> get_cancel;
        function<void(RenderTile*, Device*)> map_neighbor_tiles;
        function<void(RenderTile*, Device*)> unmap_neighbor_tiles;
 
index 0c15ba5cbbdac62b116b30eae58f7ac3cadf01e7..f00c722f25b524b59509b6130ccddcc2d9dd79c3 100644 (file)
@@ -40,8 +40,8 @@ struct avxf
        __forceinline avxf(const __m256 a) : m256(a) {}
        __forceinline avxf(const __m256i a) : m256(_mm256_castsi256_ps (a)) {}
 
-       __forceinline operator const __m256&(void) const { return m256; }
-       __forceinline operator       __m256&(void)       { return m256; }
+       __forceinline operator const __m256&() const { return m256; }
+       __forceinline operator       __m256&()       { return m256; }
 
        __forceinline avxf          (float a) : m256(_mm256_set1_ps(a)) {}
 
index 54fa6a80df589c7233277207e9f6ce19343516dc..ae1d217c54f3b60c6357c65a4a57fc7ddbeeb5c7 100644 (file)
@@ -35,12 +35,12 @@ void util_guarded_mem_free(size_t n)
 
 /* Public API. */
 
-size_t util_guarded_get_mem_used(void)
+size_t util_guarded_get_mem_used()
 {
        return global_stats.mem_used;
 }
 
-size_t util_guarded_get_mem_peak(void)
+size_t util_guarded_get_mem_peak()
 {
        return global_stats.mem_peak;
 }
index 87c1526dee4f5e3685350964f51e772e85148a87..c02aed93aa2520b23811af581c0f3fd21dcb71d7 100644 (file)
@@ -158,8 +158,8 @@ public:
 };
 
 /* Get memory usage and peak from the guarded STL allocator. */
-size_t util_guarded_get_mem_used(void);
-size_t util_guarded_get_mem_peak(void);
+size_t util_guarded_get_mem_used();
+size_t util_guarded_get_mem_peak();
 
 /* Call given function and keep track if it runs out of memory.
  *
index f38683bf7de2fe73409dd4724b6ca213afb5d76d..b0922db32fbf3d0475834c5089747934b1bcff88 100644 (file)
@@ -45,7 +45,7 @@ void util_logging_init(const char *argv0)
 #endif
 }
 
-void util_logging_start(void)
+void util_logging_start()
 {
 #ifdef WITH_CYCLES_LOGGING
        using CYCLES_GFLAGS_NAMESPACE::SetCommandLineOption;
index 4a5c3cd15bfa9d0c9606790dec880ab2c39352c6..01a9b62ad931ea37304e4500b50ea3ad42dfc165 100644 (file)
@@ -52,7 +52,7 @@ struct int2;
 struct float3;
 
 void util_logging_init(const char *argv0);
-void util_logging_start(void);
+void util_logging_start();
 void util_logging_verbosity_set(int verbosity);
 
 std::ostream& operator <<(std::ostream &os,
index 453af7de36b781cc3ae0d351dad6d8b072f8b2a9..4ed9ebd60ffda756313f4ad3cfbdd79b3cedf8b9 100644 (file)
@@ -114,7 +114,7 @@ public:
                return cancel_message;
        }
 
-       void set_cancel_callback(function<void(void)> function)
+       void set_cancel_callback(function<void()> function)
        {
                cancel_cb = function;
        }
@@ -323,7 +323,7 @@ public:
                }
        }
 
-       void set_update_callback(function<void(void)> function)
+       void set_update_callback(function<void()> function)
        {
                update_cb = function;
        }
@@ -331,8 +331,8 @@ public:
 protected:
        thread_mutex progress_mutex;
        thread_mutex update_mutex;
-       function<void(void)> update_cb;
-       function<void(void)> cancel_cb;
+       function<void()> update_cb;
+       function<void()> cancel_cb;
 
        /* pixel_samples counts how many samples have been rendered over all pixel, not just per pixel.
         * This makes the progress estimate more accurate when tiles with different sizes are used.
index b5623860e33becf263594505787eb33c0c996dd2..66670c9a7790b98ddb6905b708d5b54b243ae2c2 100644 (file)
@@ -44,8 +44,8 @@ struct ssef
        __forceinline ssef& operator=(const ssef& other) { m128 = other.m128; return *this; }
 
        __forceinline ssef(const __m128 a) : m128(a) {}
-       __forceinline operator const __m128&(void) const { return m128; }
-       __forceinline operator       __m128&(void)       { return m128; }
+       __forceinline operator const __m128&() const { return m128; }
+       __forceinline operator       __m128&()       { return m128; }
 
        __forceinline ssef          (float a) : m128(_mm_set1_ps(a)) {}
        __forceinline ssef          (float a, float b, float c, float d) : m128(_mm_setr_ps(a, b, c, d)) {}
@@ -517,12 +517,12 @@ ccl_device_inline float len3(const ssef& a)
 /* faster version for SSSE3 */
 typedef ssei shuffle_swap_t;
 
-ccl_device_inline shuffle_swap_t shuffle_swap_identity(void)
+ccl_device_inline shuffle_swap_t shuffle_swap_identity()
 {
        return _mm_set_epi8(15, 14, 13, 12, 11, 10, 9, 8, 7, 6, 5, 4, 3, 2, 1, 0);
 }
 
-ccl_device_inline shuffle_swap_t shuffle_swap_swap(void)
+ccl_device_inline shuffle_swap_t shuffle_swap_swap()
 {
        return _mm_set_epi8(7, 6, 5, 4, 3, 2, 1, 0, 15, 14, 13, 12, 11, 10, 9, 8);
 }
@@ -537,12 +537,12 @@ ccl_device_inline const ssef shuffle_swap(const ssef& a, const shuffle_swap_t& s
 /* somewhat slower version for SSE2 */
 typedef int shuffle_swap_t;
 
-ccl_device_inline shuffle_swap_t shuffle_swap_identity(void)
+ccl_device_inline shuffle_swap_t shuffle_swap_identity()
 {
        return 0;
 }
 
-ccl_device_inline shuffle_swap_t shuffle_swap_swap(void)
+ccl_device_inline shuffle_swap_t shuffle_swap_swap()
 {
        return 1;
 }
index 16a8591a8a95c91d7dc5c6183e0915ea2ff698cc..37d8bdbd4b090f0026a0aed6ac329d6887fb201e 100644 (file)
@@ -21,7 +21,7 @@
 
 CCL_NAMESPACE_BEGIN
 
-thread::thread(function<void(void)> run_cb, int group)
+thread::thread(function<void()> run_cb, int group)
   : run_cb_(run_cb),
     joined_(false),
        group_(group)
index f7d2d452d8914a6c37616faf40db7b4e008844f7..6250bb95dcfdc3f5ea5320d06f5c314a2f1eef22 100644 (file)
@@ -46,14 +46,14 @@ typedef std::condition_variable thread_condition_variable;
 
 class thread {
 public:
-       thread(function<void(void)> run_cb, int group = -1);
+       thread(function<void()> run_cb, int group = -1);
        ~thread();
 
        static void *run(void *arg);
        bool join();
 
 protected:
-       function<void(void)> run_cb_;
+       function<void()> run_cb_;
        std::thread thread_;
        bool joined_;
        int group_;
index 28146ad04f75bf3b492ca7a4e5982a25caf96459..ed2300e7996db4088f57e020169d83f167021d6c 100644 (file)
@@ -35,8 +35,8 @@ struct ccl_try_align(16) float3 {
        __forceinline float3(const float3& a);
        __forceinline explicit float3(const __m128& a);
 
-       __forceinline operator const __m128&(void) const;
-       __forceinline operator __m128&(void);
+       __forceinline operator const __m128&() const;
+       __forceinline operator __m128&();
 
        __forceinline float3& operator =(const float3& a);
 #else  /* __KERNEL_SSE__ */
index 45f61767d3f4d3c812ce4671da49672f17a1a7a9..2e840a5c399b055a16169edb9c43584adeff1ae6 100644 (file)
@@ -43,12 +43,12 @@ __forceinline float3::float3(const __m128& a)
 {
 }
 
-__forceinline float3::operator const __m128&(void) const
+__forceinline float3::operator const __m128&() const
 {
        return m128;
 }
 
-__forceinline float3::operator __m128&(void)
+__forceinline float3::operator __m128&()
 {
        return m128;
 }
index 154391f6881ac31b7aa25c6359d167cef76dd958..5c10d483c2e12ea2af5c19294afd85b8e959618a 100644 (file)
@@ -36,8 +36,8 @@ struct ccl_try_align(16) float4 {
        __forceinline float4();
        __forceinline explicit float4(const __m128& a);
 
-       __forceinline operator const __m128&(void) const;
-       __forceinline operator __m128&(void);
+       __forceinline operator const __m128&() const;
+       __forceinline operator __m128&();
 
        __forceinline float4& operator =(const float4& a);
 
index a49fac65b10093a8523285400f4d6f7a1c7529f6..a83148031f1365e6ef18fb499321d3f1ac5469e7 100644 (file)
@@ -38,12 +38,12 @@ __forceinline float4::float4(const __m128& a)
 {
 }
 
-__forceinline float4::operator const __m128&(void) const
+__forceinline float4::operator const __m128&() const
 {
        return m128;
 }
 
-__forceinline float4::operator __m128&(void)
+__forceinline float4::operator __m128&()
 {
        return m128;
 }
index a91fddb20386364335f78d9e5c173b15f6c59d5d..08720b8ff485afa0bf842520f994840e7a18a898 100644 (file)
@@ -48,8 +48,8 @@ struct ccl_try_align(32) float8 {
        __forceinline float8(const float8& a);
        __forceinline explicit float8(const __m256& a);
 
-       __forceinline operator const __m256&(void) const;
-       __forceinline operator __m256&(void);
+       __forceinline operator const __m256&() const;
+       __forceinline operator __m256&();
 
        __forceinline float8& operator =(const float8& a);
 
index ebf8260bc7c0606562bb679c8726f5cd7060cd28..84fe233c334e4feb6598420d60a41eb1e81b2d31 100644 (file)
@@ -55,12 +55,12 @@ __forceinline float8::float8(const __m256& f)
 {
 }
 
-__forceinline float8::operator const __m256&(void) const
+__forceinline float8::operator const __m256&() const
 {
        return m256;
 }
 
-__forceinline float8::operator __m256&(void)
+__forceinline float8::operator __m256&()
 {
        return m256;
 }
index 9d43b201c02b09d41e628b3c0acc4c0d7fb1bd33..f68074b982bca2b26078acfc65debb8bbd87d8e5 100644 (file)
@@ -35,8 +35,8 @@ struct ccl_try_align(16) int3 {
        __forceinline int3(const int3& a);
        __forceinline explicit int3(const __m128i& a);
 
-       __forceinline operator const __m128i&(void) const;
-       __forceinline operator __m128i&(void);
+       __forceinline operator const __m128i&() const;
+       __forceinline operator __m128i&();
 
        __forceinline int3& operator =(const int3& a);
 #else  /* __KERNEL_SSE__ */
index ada50c4812c5e15ca67b3983d03b7df2212dc735..1b195ca753f2ffc43b5788945c8a35a7c2acb4fa 100644 (file)
@@ -43,12 +43,12 @@ __forceinline int3::int3(const int3& a)
 {
 }
 
-__forceinline int3::operator const __m128i&(void) const
+__forceinline int3::operator const __m128i&() const
 {
        return m128;
 }
 
-__forceinline int3::operator __m128i&(void)
+__forceinline int3::operator __m128i&()
 {
        return m128;
 }
index 4ef162f1ac66093fe6af9e64fc0b991ea15ddf78..52e6fed8c145d6ada54a676ca45cf93a4bea81eb 100644 (file)
@@ -39,8 +39,8 @@ struct ccl_try_align(16) int4 {
        __forceinline int4(const int4& a);
        __forceinline explicit int4(const __m128i& a);
 
-       __forceinline operator const __m128i&(void) const;
-       __forceinline operator __m128i&(void);
+       __forceinline operator const __m128i&() const;
+       __forceinline operator __m128i&();
 
        __forceinline int4& operator=(const int4& a);
 #else  /* __KERNEL_SSE__ */
index a62561709deb9a9ef7b03457333b38b95a01df34..c058f86c40019181ee139204dd3125685b686ad6 100644 (file)
@@ -43,12 +43,12 @@ __forceinline int4::int4(const __m128i& a)
 {
 }
 
-__forceinline int4::operator const __m128i&(void) const
+__forceinline int4::operator const __m128i&() const
 {
        return m128;
 }
 
-__forceinline int4::operator __m128i&(void)
+__forceinline int4::operator __m128i&()
 {
        return m128;
 }
index fb9c5784adf6088d98aff72345c5284bff5f65ac..18fa231d6e71923004773abbd4a6ba6a4ca741e0 100644 (file)
@@ -43,7 +43,7 @@ public:
        using BaseClass::vector;
 
        /* Try as hard as possible to use zero memory. */
-       void free_memory(void)
+       void free_memory()
        {
                BaseClass::resize(0);
                BaseClass::shrink_to_fit();
index 3836cc86ee0ee5a2111d2f5e4658a5d658f19a7d..9dffd7a80bdd9990a0cf3f211b6ff4e90685ef89 100644 (file)
@@ -215,7 +215,7 @@ static void view_motion(int x, int y)
        V.mouseY = y;
 }
 
-static void view_idle(void)
+static void view_idle()
 {
        if(V.redraw) {
                V.redraw = false;
index a757b56a5d22c3b1894bc02f34a9fea0812f9fa8..ae50b098b3903749182f50c29815b253c9589201 100644 (file)
 
 CCL_NAMESPACE_BEGIN
 
-typedef void (*ViewInitFunc)(void);
-typedef void (*ViewExitFunc)(void);
+typedef void (*ViewInitFunc)();
+typedef void (*ViewExitFunc)();
 typedef void (*ViewResizeFunc)(int width, int height);
-typedef void (*ViewDisplayFunc)(void);
+typedef void (*ViewDisplayFunc)();
 typedef void (*ViewKeyboardFunc)(unsigned char key);
 typedef void (*ViewMotionFunc)(int x, int y, int button);