Raylib-cs: [Bug] Fix names in structs!
Before submitting a new issue, please verify and check:
- The issue is specific to Raylib-cs and not raylib
- I checked there is no similar issue already reported
- My code has no errors or misuse of Raylib-cs
Issue description
Hello, i saw variable names that not agree with the C# rules.
Example:
Yours:
public partial struct Transform
{
/// <summary>
/// Translation
/// </summary>
public Vector3 translation;
/// <summary>
/// Rotation
/// </summary>
public Quaternion rotation;
/// <summary>
/// Scale
/// </summary>
public Vector3 scale;
}
How it should look like:
public partial struct Transform
{
/// <summary>
/// Translation
/// </summary>
public Vector3 Translation;
/// <summary>
/// Rotation
/// </summary>
public Quaternion Rotation;
/// <summary>
/// Scale
/// </summary>
public Vector3 Scale;
}
Environment
*Raylib-cs
About this issue
- Original URL
- State: closed
- Created a year ago
- Reactions: 1
- Comments: 22 (20 by maintainers)
And i would suggest to rename the
Raylib_csnamespace toRaylibIf you agree with that i could do that for the full class
Yes Please! I always hated how raylib-cs didn’t use C# naming conventions.
I have added a dev branch to the examples to test the changes and check for any issues. Looks good so far!
Nothing specific at the moment. Thinking about other design changes I may make but have not decided yet.
@MrScautHD Done, dev branch created. Can you fix the conflicts then I can merge it in.
@MrScautHD It is a breaking change so I agree with @9ParsonsB that it should go into a major release. Considering creating a separate branch to track and test breaking changes like this.