Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

Remove System.Slices #1295

Closed
ahsonkhan opened this issue Mar 10, 2017 · 7 comments
Closed

Remove System.Slices #1295

ahsonkhan opened this issue Mar 10, 2017 · 7 comments

Comments

@ahsonkhan
Copy link
Member

@KrzysztofCwalina, can you provide more details/context?

@KrzysztofCwalina
Copy link
Member

KrzysztofCwalina commented Mar 10, 2017

I think OwnedMemory, Memory, ReadableBuffer/ReadOnlyBytes should ship in a package called something like System.Buffers.<something>, maybe System.Buffers.Primitives.

At that point, why do we still need System.Slices? What other APIs are there that won't be either in System.Memory or in System.Buffers.?

@ahsonkhan
Copy link
Member Author

cc @davidfowl

@jkotas
Copy link
Member

jkotas commented Mar 10, 2017

System.Buffers., maybe System.Buffers.Primitives.

I would vote for System.Buffers.

@KrzysztofCwalina
Copy link
Member

The problem with System.Buffers is that it already exists in corefx. We probably don't have enough bake in time to put these types there.

@jkotas
Copy link
Member

jkotas commented Mar 10, 2017

I see ... so should be just some temporary name that does not really matter then?

@aL3891
Copy link
Contributor

aL3891 commented Mar 10, 2017

I like System.Memory[.]Primitives because it feels like "memory" is a more generic concept than "buffer".

@shiftylogic
Copy link
Contributor

Slices is gone with #1319. There is a couple pieces of span extension cleanup being done as part of a different work item.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants