Skip to content

Conversation

@ringabout
Copy link
Member

Ref #15307

@timotheecour
Copy link
Member

timotheecour commented Apr 24, 2021

I'm assuming this is possible thanks to csources being updated (refs #16282) to 1.0.11 ?

can these be removed too?

lib/system.nim:889:13:  template `<//>`*(t: untyped): untyped = owned(t)
lib/system.nim:912:13:  template `<//>`*(t: untyped): untyped = t

if for some reason they can't, then how about hiding it behind -d:nimLegacyOwnedOperator

@ringabout
Copy link
Member Author

ringabout commented Apr 24, 2021

I'm assuming this is possible thanks to csources being updated (refs #16282) to 1.0.11 ?

Yeah

if for some reason they can't, then how about hiding it behind -d:nimLegacyOwnedOperator

Ok, let me try to remove it. It should migrate easily even if it's breaking changes. And I guess nobody used it. It looks scary :)

@ringabout
Copy link
Member Author

only fusion/btreetables.nim failed, so I made an associated PR
nim-lang/fusion#90

Araq pushed a commit to nim-lang/fusion that referenced this pull request Apr 25, 2021
* add std/scripting

* thanks for juancarlospaco's suggestion

* Update src/fusion/scripting.nim

Co-authored-by: ee7 <[email protected]>

* Update changelog.md

Co-authored-by: ee7 <[email protected]>

* remove

Co-authored-by: ee7 <[email protected]>
@ringabout ringabout closed this Apr 26, 2021
@ringabout ringabout reopened this Apr 26, 2021
@ringabout ringabout requested a review from Araq April 26, 2021 03:41
@Araq Araq merged commit 68e522e into nim-lang:devel Apr 26, 2021
@timotheecour timotheecour mentioned this pull request Apr 29, 2021
2 tasks
PMunch pushed a commit to PMunch/Nim that referenced this pull request Mar 28, 2022
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.

3 participants