Skip to main content
GameDev.net gamedev.net
🔒 Locked

Support for octals?

Started by dkrusu Dec 10, 2012 at 7:00 AM 5 replies 2.2k views
Original Post
dkrusu
dkrusu
It seems that angelscript doesn't support octals, any plans for this?

Krusu
WitchLord
WitchLord
I don't have any plans to implement this myself. However, if you would like to do it and send me a patch I can certainly include it in the SDK.

Regards,
Andreas
AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - <a href="http://www.angelcode.com/tower" rel
dkrusu
dkrusu
This was actually quite easy to implement. Any integer that is prefixed with a 0 is considered a octal just like in C. I added a simple test case, but I wasn't sure exactly where to stick it so I put it in test_bits.

Krusu
dkrusu
dkrusu
Forgot to check if the number is actually a valid octal (0-7), I'll upload the patch as soon as I fix this.
dkrusu
dkrusu
Okay, it now validates the octal. It does add another loop through the numbers, maybe you can think of a better way to validate it?
WitchLord
WitchLord
Thanks for the patch. I'll look into this as soon as I can and have it merged with the SDK.

Regards,
Andreas
AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - <a href="http://www.angelcode.com/tower" rel
WitchLord
WitchLord

I received another patch from ketmar that added support for binary numbers in addition to octal. I've checked in that in revision 1571.

The nice thing about that patch is the consistency with the prefixes, example:


int binary = 0b11111111;
int octal = 0o7777;
int decimal = 0d999999;
int hexadecimal = 0xFFFFFFFF;
AngelCode.com - game development and more - Reference DB - game developer references
AngelScript - free scripting library - BMFont - free bitmap font generator - <a href="http://www.angelcode.com/tower" rel

Topic Locked

This topic has been locked by a moderator. New replies are not allowed.

Sign in to reply to this topic.