Dave Smith
Dave Smith
@brockallen, I was getting "The client application is not known or is not authorized" errors without this fix. Updated client_id to match ClientId on line 195 of Client.cs in Host.Configuration/Config/Clients.cs...
- Use only one dash in the `clang-format` call preceding the `style` switch (two dashes don't work) - Correct "VS Code" spelling - Link to exaplanation of how to change...
Closes #66.
Continued work on #43.
To eliminate a distinction without a difference between `KDIS::UTIL` and `KDIS::UTILS`.
Per @karljj1's suggestion here: https://github.com/karljj1/kdis/pull/64#issuecomment-2710154461
Backed up by the C++ Core Guidelines: * [SL.con.1](https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#slcon1-prefer-using-stl-array-or-vector-instead-of-a-c-array): Prefer using STL array or vector instead of a C array Based on early work on #49, the need for fixing...
Per C++ Core Guidelines [[ES.20](https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es20-always-initialize-an-object)], this has two primary benefits: 1. simplify refactoring work like that in #44 2. protect against used-before-set errors Also apply these closely related rules from...
Currently assigns to `m_SpecificAppearanceUnion.m_TankDitchConcertinaWire.m_ui32BreachLength`, should assign to `m_SpecificAppearanceUnion.m_TankDitchConcertinaWire.m_ui32BreachLoc`, instead.
@karljj1, @carlocorradini: is this #49 refactoring one you'd support? I want to show you my intended direction early to receive your feedback. In my early work on #44, I quickly...