Skip to content

Fix concat#356

Merged
shargon merged 9 commits intoneo-project:masterfrom
shargon:fix-concat
Sep 19, 2020
Merged

Fix concat#356
shargon merged 9 commits intoneo-project:masterfrom
shargon:fix-concat

Conversation

@shargon
Copy link
Copy Markdown
Member

@shargon shargon commented Sep 15, 2020

Close #355

Tommo-L
Tommo-L previously approved these changes Sep 16, 2020
lightszero
lightszero previously approved these changes Sep 16, 2020
Copy link
Copy Markdown
Member

@lightszero lightszero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had fix this a little.
Parse 2string,3string,4string overload.
Add a throw for other overload.
Add parse for String::concat(string[] args).

lightszero
lightszero previously approved these changes Sep 16, 2020
Tommo-L
Tommo-L previously approved these changes Sep 16, 2020
@erikzhang
Copy link
Copy Markdown
Member

erikzhang commented Sep 16, 2020

Tested? @neo-project/ngd-shanghai

@shargon shargon dismissed stale reviews from Tommo-L and lightszero via 602159c September 16, 2020 13:31
@shargon
Copy link
Copy Markdown
Member Author

shargon commented Sep 18, 2020

ready to merge?

@shargon shargon merged commit 511d990 into neo-project:master Sep 19, 2020
@shargon shargon deleted the fix-concat branch September 19, 2020 07:21
Jim8y pushed a commit that referenced this pull request Aug 3, 2025
* Fix concat
* Add comment
* dotnet format
* fix concat parse
* fix format
* format
* add enter
* Remove enter
Co-authored-by: lights <lightsever@hotmail.com>
Jim8y pushed a commit that referenced this pull request Aug 18, 2025
* Fix concat

* Add comment

* dotnet format

* fix concat parse

* fix format

* format

* add enter

* Remove enter

Co-authored-by: lights <lightsever@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Concat string failure

4 participants