Add Type Convert by lightszero · Pull Request #196 · neo-project/neo-devpack-dotnet

lightszero · 2020-03-05T04:43:01Z

fixed #195

Tommo-L · 2020-03-05T05:40:49Z

tests/Neo.Compiler.MSIL.UnitTests/TestClasses/Contract_TypeConvert.cs

+
+
+        [OpCode(SmartContract.Framework.OpCode.CONVERT, "0x21")]
+        public extern static BigInteger ConvertToInteger(byte[] from);


Could we still use the AsByteArray as the function name?

Maybe we should move these to SmartContract.Framework

if we do not care about old contract code,we can direct change AsByteArray.

Just ignore the old contract, as the previous version could not invoke contract successful. 😂

ok，let us add more convert code in framework and compiler

pushNumber pushBoolean pushDataArray 過去合約中所有的數值是統一的Push 小於等於16的數字和 byte[0] 和 true false 是Integer >16的數字和 byte[n>0] 是 ByteArray 現在在合約中使用數值將直接得到正確的類型 new byte[0] 是ByteArray true 是Boolean 17 是Integer

因爲現在會返回integer類型了，不再是DataArray 2.修改了TypeConvert 應該現在 new byte[0] 直接是ByteArray類型了

…ect/neo-devpack-dotnet into Branch_lights_utconvert

lightszero · 2020-03-05T10:23:17Z

I had do some fix

now
new byte[0] =>ByteArray
new byte[3]{1,2,3}=>ByteArray
true =>Boolean
1=>Integer
2=>Integer
17 =>Integer

const value will got right type without convert.

2.Use Helper.AsBigInteger() to convert to BigInteger 3.fix unittest

lightszero · 2020-03-05T11:04:03Z

Helper.AsByteArray() fixed.

tests/Neo.Compiler.MSIL.UnitTests/UnitTest_TypeConvert.cs

lightszero · 2020-03-09T13:49:09Z

@shargon has a format problem,fix it.
@Tommo-L what do you think?should we merge this?

Tommo-L · 2020-03-10T02:14:38Z

I'll review it today.

src/Neo.Compiler.MSIL/MSIL/Conv_Common.cs

Tommo-L · 2020-03-10T03:39:39Z

src/Neo.SmartContract.Framework/Helper.cs

        /// </summary>
-        [Script]
+        [OpCode(OpCode.CONVERT, StackItemType_ByteArray)]
        public extern static byte[] AsByteArray(this BigInteger source);


Maybe we need to update the comments later.

Tommo-L · 2020-03-10T04:04:57Z

tests/Neo.Compiler.MSIL.UnitTests/TestClasses/Contract_TypeConvert.cs

+            var int3 = bts3.AsBigInteger();
+
+            var arrobj = new object[8];
+            arrobj[0] = int0;


I think we should add a test case for _ConvertPushBoolean.

That's hard,in IL sometimes bool is int too.

erikzhang · 2020-03-11T09:06:34Z

src/Neo.SmartContract.Framework/Helper.cs

        /// </summary>
-        [Script]
+        [OpCode(OpCode.CONVERT, StackItemType_ByteArray)]
        public extern static byte[] AsByteArray(this byte source);


If we use CONVERT, we should rename the methods from As*** to To***. Because "As" implies the meaning of not performing the actual type conversion.

#196 because Insert a Opcode Convert but newarr still get last opcode for a PushNumber 2.add some unittest for it.

* this unittest for #195 * Clean code * 區分push pushNumber pushBoolean pushDataArray 過去合約中所有的數值是統一的Push 小於等於16的數字和 byte[0] 和 true false 是Integer >16的數字和 byte[n>0] 是 ByteArray 現在在合約中使用數值將直接得到正確的類型 new byte[0] 是ByteArray true 是Boolean 17 是Integer * 1.修改了unittest null 因爲現在會返回integer類型了，不再是DataArray 2.修改了TypeConvert 應該現在 new byte[0] 直接是ByteArray類型了 * 1.Use Helper.AsByteArray() to convert to ByteArray 2.Use Helper.AsBigInteger() to convert to BigInteger 3.fix unittest * fix one small fault. * add value equal test * Clean code * Clean code * fix format Co-authored-by: Shargon <shargon@gmail.com>

#196 because Insert a Opcode Convert but newarr still get last opcode for a PushNumber 2.add some unittest for it.

* this unittest for #195 * Clean code * 區分push pushNumber pushBoolean pushDataArray 過去合約中所有的數值是統一的Push 小於等於16的數字和 byte[0] 和 true false 是Integer >16的數字和 byte[n>0] 是 ByteArray 現在在合約中使用數值將直接得到正確的類型 new byte[0] 是ByteArray true 是Boolean 17 是Integer * 1.修改了unittest null 因爲現在會返回integer類型了，不再是DataArray 2.修改了TypeConvert 應該現在 new byte[0] 直接是ByteArray類型了 * 1.Use Helper.AsByteArray() to convert to ByteArray 2.Use Helper.AsBigInteger() to convert to BigInteger 3.fix unittest * fix one small fault. * add value equal test * Clean code * Clean code * fix format Co-authored-by: Shargon <shargon@gmail.com>

#196 because Insert a Opcode Convert but newarr still get last opcode for a PushNumber 2.add some unittest for it.

this unittest for

d4a231a

#195

lightszero requested review from Tommo-L and shargon March 5, 2020 04:43

Tommo-L reviewed Mar 5, 2020

View reviewed changes

shargon and others added 4 commits March 5, 2020 06:53

Clean code

2cc7c26

1.修改了unittest null

4d9b845

因爲現在會返回integer類型了，不再是DataArray 2.修改了TypeConvert 應該現在 new byte[0] 直接是ByteArray類型了

Merge branch 'Branch_lights_utconvert' of https://github.com/neo-proj…

4dbd044

…ect/neo-devpack-dotnet into Branch_lights_utconvert

lightszero changed the title ~~add a convert unittest.~~ Add Type Convert Mar 5, 2020

1.Use Helper.AsByteArray() to convert to ByteArray

8bac65b

2.Use Helper.AsBigInteger() to convert to BigInteger 3.fix unittest

fix one small fault.

5b63eaf

lightszero requested a review from Tommo-L March 5, 2020 13:41

Merge branch 'master' into Branch_lights_utconvert

d96499d

shargon reviewed Mar 5, 2020

View reviewed changes

tests/Neo.Compiler.MSIL.UnitTests/UnitTest_TypeConvert.cs Show resolved Hide resolved

add value equal test

d7f3037

lightszero requested a review from shargon March 9, 2020 02:34

shargon added 2 commits March 9, 2020 11:28

Clean code

b26a6ed

Clean code

fecb35e

shargon previously approved these changes Mar 9, 2020

View reviewed changes

fix format

494826f

lightszero dismissed shargon’s stale review via 494826f March 9, 2020 13:45

lightszero requested a review from shargon March 9, 2020 13:48

Tommo-L mentioned this pull request Mar 10, 2020

Ship Neo 3 Preview 2 neo-project/neo#1467

Closed

Tommo-L reviewed Mar 10, 2020

View reviewed changes

shargon approved these changes Mar 10, 2020

View reviewed changes

lightszero merged commit 8acf90f into master Mar 11, 2020

lightszero deleted the Branch_lights_utconvert branch March 11, 2020 03:41

erikzhang reviewed Mar 11, 2020

View reviewed changes

shargon mentioned this pull request Mar 11, 2020

Rename commands #200

Merged

superboyiii mentioned this pull request Mar 12, 2020

BigInteger 0 isn't able to be convert to Byte[]{ 0 } #201

Closed

lightszero added a commit that referenced this pull request Mar 12, 2020

1.fix a bug about convert func

275bae1

#196 because Insert a Opcode Convert but newarr still get last opcode for a PushNumber 2.add some unittest for it.

lightszero mentioned this pull request Mar 12, 2020

fix a bug about convert func #203

Merged

lightszero added a commit that referenced this pull request Mar 13, 2020

1.fix a bug about convert func (#203)

b51c2b1

#196 because Insert a Opcode Convert but newarr still get last opcode for a PushNumber 2.add some unittest for it.

Jim8y pushed a commit that referenced this pull request Aug 3, 2025

1.fix a bug about convert func (#203)

000c574

#196 because Insert a Opcode Convert but newarr still get last opcode for a PushNumber 2.add some unittest for it.

Jim8y pushed a commit that referenced this pull request Aug 18, 2025

1.fix a bug about convert func (#203)

a399a1f

#196 because Insert a Opcode Convert but newarr still get last opcode for a PushNumber 2.add some unittest for it.



		[OpCode(SmartContract.Framework.OpCode.CONVERT, "0x21")]
		public extern static BigInteger ConvertToInteger(byte[] from);

Conversation

lightszero commented Mar 5, 2020

Uh oh!

Tommo-L Mar 5, 2020

Choose a reason for hiding this comment

Uh oh!

shargon Mar 5, 2020

Choose a reason for hiding this comment

Uh oh!

lightszero Mar 5, 2020

Choose a reason for hiding this comment

Uh oh!

Tommo-L Mar 5, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

lightszero Mar 5, 2020

Choose a reason for hiding this comment

Uh oh!

lightszero commented Mar 5, 2020

Uh oh!

lightszero commented Mar 5, 2020

Uh oh!

Uh oh!

lightszero commented Mar 9, 2020

Uh oh!

Tommo-L commented Mar 10, 2020

Uh oh!

Uh oh!

Tommo-L Mar 10, 2020

Choose a reason for hiding this comment

Uh oh!

Tommo-L Mar 10, 2020

Choose a reason for hiding this comment

Uh oh!

lightszero Mar 11, 2020

Choose a reason for hiding this comment

Uh oh!

erikzhang Mar 11, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Tommo-L Mar 5, 2020 •

edited

Loading

erikzhang Mar 11, 2020 •

edited

Loading