search

karljj1 / kdis

DIS(IEEE 1278.1) Fully Implemented in C++. A complete open source implementation of DIS (Distributed Interactive Simulation) in C++. 1278.1 and 1278.1a are fully implemented including enumerations. Includes several utility classes to help with developing your DIS app.
BSD 2-Clause "Simplified" License
26 stars 10 forks source link

chore: add .clang-format #8

Closed carlocorradini closed 12 months ago

carlocorradini commented 1 year ago

@karljj1 I've started a refactoring. Can you take a look at https://github.com/carlocorradini/kdis/tree/new 🥳

karljj1 commented 1 year ago

Yeah sure. I'll take a look next week. Thanks for spending some time on this.

carlocorradini commented 1 year ago

@karljj1 Can I give you "edit" permissions? Soooo we can collaborate without touching"master"

karljj1 commented 1 year ago

Yeah that's fine. I don't mind you working on a branch on the main repo if you like. Whatever you prefer.

carlocorradini commented 1 year ago

Done 🥳

karljj1 commented 1 year ago

Hey, Looks good so far. What is the reasons for using hpp instead of h for the file extensions? You have my permission to fix the formatting ;) I long ago stopped liking the spaces in brackets I was using if ( something ) or void someMethod( float value ). So if you want to run some formatting tool to fix that I would be happy ;) I did use a tool to do it a long time ago but can't remember the name now.

carlocorradini commented 1 year ago

Working on it :)

For why .hpp is preferable see https://stackoverflow.com/a/152671/6676781

We don't provide C compatible headers so... keep them different