SRL Archive - MSI Suggestions

For those of you who don't yet know, MSI has been following SRL 5 standards.

In SRL 5, all constants will be CapLocked, as so: SRL_CONSTANT, WAIT_TIME_UPTEXT, TREE_COLOR

So, for all the contants that we use, let's make sure they look this way :)

~RM

From: Rasta Magician
For those of you who don't yet know, MSI has been following SRL 5 standards.

In SRL 5, all constants will be CapLocked, as so: SRL_CONSTANT, WAIT_TIME_UPTEXT, TREE_COLOR

So, for all the contants that we use, let's make sure they look this way :)

~RM

Why capitalized? Just for them to stand out? Or…

Uhm, I'm not sure if we agree on the all caps thing. :p <3

From: Wizzup?
Uhm, I'm not sure if we agree on the all caps thing. :p <3

It makes sense if you want them to stand out from other variables.

From: Coh3n
It makes sense if you want them to stand out from other variables.

<sarcasm< It had crossed my mind… :duh: </sarcasm>

Anyway, what I said is that RM said SRL 5 is going to have constants that way, and I'm not sure the dev team agrees on that.

From: Wizzup?
<sarcasm< It had crossed my mind… :duh: </sarcasm>

Anyway, what I said is that RM said SRL 5 is going to have constants that way, and I'm not sure the dev team agrees on that.

Oh possibly my mistake, that was the impression I had from the thread.

Let's have a vote here at MSI then :p

Caps or no Caps?

~RM

From: Rasta Magician
Oh possibly my mistake, that was the impression I had from the thread.

Let's have a vote here at MSI then :p

Caps or no Caps?

~RM

Lol, I say Caps. Although we'll have to convert a million of them over, I think it makes more sense.

EDIT: After all, you said you wanted MSI to be understood by everyone. Having the constants stand out, will definitely help out.

Then it will differ from SRL 5 I guess…

Personally, I think it look's somewhat ugly. :redface:
I wouldn't do it in my personal scripts, but as far as trying to make it as easy as possible for people to follow, it's not a bad idea.

Either way, I don't mind.


Edit:
It would, however, make sense that one of SRL's largest projects would follow SRL standard's.

Earlier I thought it was a good idea but now I oppose it.
Lets keep them clean looking. :)

From: marpis
Earlier I thought it was a good idea but now I oppose it.
Lets keep them clean looking. :)

I think at first it may look ugly, but it's an obvious distinction between constants and variables. However, if the constant's names are obvious to their purpose, it shouldn't matter either way.

Since all the functions are already signified with a "MSI_" why not just put a "C_" in front of constants?

Just an idea.

From: NCDS
Since all the functions are already signified with a "MSI_" why not just put a "C_" in front of constants?

Just an idea.

Not a bad idea. I think that would make it look uglier, but that's just my personal preference. :o

I think either would work because it gives a good distinction.

I'm still for the capitalized. So much better.
At least for the include, idc if people do it in their scripts.

We'll have a shitloads of consts with SRL 5, might as well make a way to spot them all.