Artemis build without FORCEINT32 now correctly uses BigInteger in SystemBitManager

thelinuxlich / artemis_CSharp

Artemis Entity System Framework ported to C#

Other

342 stars 78 forks source link

Artemis build without FORCEINT32 now correctly uses BigInteger in SystemBitManager #95

Closed Maximusya closed 8 years ago

Maximusya commented 8 years ago

Fixes #93

Maximusya commented 8 years ago

Please review and merge if it's ok

KellanHiggins commented 8 years ago

I am sure these are equal, but I specifically used the static method BigInteger.One to give the system a 1. Probably works the same though.

Maximusya commented 8 years ago

I placed new BigInteger(1) because in Unity5 pull request @iskar909 used it like this instead of BigInteger.One: I assumed there was no such definition in Unity5 project that referenced custom BigInteger.dll library. @iskar909, correct me if I am wrong

coryleeio commented 8 years ago

Sounds correct