polterguy / lizzie

A script language for .Net and the CLR
https://ainiro.io/
MIT License
190 stars 27 forks source link

Coding standard question? #39

Closed polterguy closed 5 years ago

polterguy commented 5 years ago

As I accepted @robert-mccausland I realized I'm probably one of few using Lizzie's current coding standard, seeing the merge request had braces on new lines - Example.

if (foo)
{
   /* do stuff ... */
}
else
{
    /* do other stuff ... /*
}

Instead of what I used as I started out the project, having braces on the same line - Example.

if (foo) {

    /* do stuff ... */

} else {

    /* do other stuff ... */
}

If most other prefer Robert's (more commonly used) standard, I'd be willing to switch though, realizing it's a more "popular" coding standard. However, I'll switch the entire project if so, since I like the idea of that the same coding standard is used throughout the entirety of its code ...

Thoughts ...?

robert-mccausland commented 5 years ago

One of the issues I had was when I was using Visual Studio's auto-formatter it wanted to format all of the braces a be on new lines, so I did the formatting manually. However after a google I realise there is a way of changing it to format braces in the same line. But as most other projects use the the other convention it could be annoying switching between the standards.

Having said that I don't mind either way of doing things. I am just so used to using the new lines for all my braces in C# so I automatically did it.

polterguy commented 5 years ago

Thx for your input, I'll keep it the way it is then for now, unless everybody else have different opinions :)

polterguy commented 5 years ago

I'll close this then ...