godot-next icon indicating copy to clipboard operation
godot-next copied to clipboard

Add clang-format support

Open willnationsdev opened this issue 5 years ago • 7 comments

We should add clang-format to the codebase so that any added C++ code will have consistent styling. We should probably use the exact same clang-format as Godot's source code for consistency.

willnationsdev avatar Apr 16 '19 17:04 willnationsdev

I don't understand why this issue is open, this repo does not have any C++ code in it and I don't think there are plans to add any.

aaronfranke avatar Aug 22 '20 22:08 aaronfranke

why not? Porting these codes to GDNative and/or Modules, is a good approach!!

nonunknown avatar Aug 22 '20 22:08 nonunknown

Would you like to volunteer to do this? I don't think anyone has announced they're doing the work to port these.

I just don't see why we need an issue open to track something that can't be meaningfully acted upon right now since there are not currently any C++ files in this repo, and we have no idea if/when this will happen.

aaronfranke avatar Aug 22 '20 22:08 aaronfranke

well I would like to, I've already Implemented a state machine using GDNative, But I must say, I'm no expert, but I would like to. Also of course if @willnationsdev or you give me support (discord questions) I'm totally in...

nonunknown avatar Aug 22 '20 22:08 nonunknown

@aaronfranke is right, for the most part. Before adding clang-format support is even really useful, we need to have C++ code integrated. And there were plans to add C++ code stuff to the codebase based on xDGameStudios' work for a graph implementation intended for my proposed Godot Journey project. However, before merging into the codebase, I wanted to ensure that there would be a consistent methodology for people to build and use C++ scripts in general which would require Godot having an official build script that was reusable and shared with the community (ideally something that would be part of the godotengine/godot-proposals#119 changes which have yet to be done).

I was going to start working on that again (and did initially), but got sidetracked by work stuff and haven't had a chance to go back since.

willnationsdev avatar Aug 22 '20 23:08 willnationsdev

well, godot 4 is approaching, we're having a lot of improvements into graphics and GDScript itself, In my opinion you should contact Vnen, since he is getting into GDNative, And you have a lot of experience and know about users needs.

nonunknown avatar Aug 22 '20 23:08 nonunknown

vnen is busy with other things, it's best not to bother him about godot-next. Giving him feedback about your experience with GDNative will only help if you have experience with GDNative, so someone else attempting a port should come first :)

aaronfranke avatar Aug 23 '20 00:08 aaronfranke