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

Added .clang-format file and formatted everything #286

Closed COM8 closed 2 years ago

COM8 commented 2 years ago

This PR adds a .clang-format file based on the configuration specified in the Makefile. I also run it over all files in the repo once.

COM8 commented 2 years ago

Those are things I could have thought about :D Fixed it.

axsaucedo commented 2 years ago

Nice one! Thanks for updating it so fast - just having a look. I can see that the format command now reflects the file command, but I can see that also the examples folder also has some udpates - did you run the command through the Makefile? Or did you run it through a different command? Just thinking it makes sense that we also format the examples but may be worth making the command in the makefile consistent with the changes

COM8 commented 2 years ago

I've used this script: https://github.com/eklitzke/clang-format-all/blob/master/clang-format-all But you are right it makes sense to reflect the changes in the Makefile. I fixed that.

axsaucedo commented 2 years ago

Perfect - looks great. Nice one @COM8 , merging - thanks for the contribution