if it's anything, these are the overloads I provide for creating a buffer in pie <https://github.com

if it's anything, these are the overloads I provide for creating a buffer in pie
https://github.com/piegfx/Pie/blob/main/src/Pie/GraphicsDevice.cs#L83-L122
they are not perfect and could do with a little refining, but they encompass pretty much everything one may want to do with them
the unsafe overload is there for those who want it, and there are safe options as well

PPerksey tbh i'm content with having a wrapped overload and an unwrapped overload i.e. if...

Maxine•8/4/23, 9:07 PM

yeah fuck it why not

PPerksey tbh i'm content with having a wrapped overload and an unwrapped overload i.e. if...

Kai•8/4/23, 9:07 PM

this I would prefer personally

KKai now we want to pass different versions of this to different args, so you take th...

Lyris the Kitori•8/4/23, 9:07 PM

yeah if your methods have a lot of thing* and you do array overloading you start getting to 3**11 overload territory

PPerksey tbh i'm content with having a wrapped overload and an unwrapped overload i.e. if...

AquaOP•8/4/23, 9:07 PM

yeah, that could work quite well

Perksey•8/4/23, 9:07 PM

most of our users don't want that though, so we will encounter the education problem still.

Perksey•8/4/23, 9:08 PM

but again, my intention is to have a "Silk DSL 101" page that just tells you everything you need to know in using these wrappers

Maxine•8/4/23, 9:08 PM

silk documentation already isn't any good, it can't get worse

Kai•8/4/23, 9:08 PM

It's the same problem Span introduces though. These are advanced APIs, you spent hours digging through docs. Read the 2 lines the intellisense will give you once you hover over the Ptr type

Perksey•8/4/23, 9:08 PM

again we literally can't ship 3.0 until we have docs

Perksey•8/4/23, 9:08 PM

the Working Group has decided that

AquaOP•8/4/23, 9:08 PM

if you need docs written you know who to ask heh

Maxine•8/4/23, 9:08 PM

what a daunting name

Maxine•8/4/23, 9:09 PM

the Working Group

AquaOP•8/4/23, 9:09 PM

i've been meaning to write new tutorials I've just been juggling around 6 projects at once

MMaxine the Working Group

Perksey•8/4/23, 9:09 PM

it used to be the ARB - API Review Board

AAqua yeah, that could work quite well

Lyris the Kitori•8/4/23, 9:09 PM

i feel like this line from zig zen shows some of my reasonings for being against that

 * Only one obvious way to do things.

 * Only one obvious way to do things.

Lyris the Kitori•8/4/23, 9:09 PM

we want people to use these safer types because they would be less error prone

Lyris the Kitori•8/4/23, 9:09 PM

and more readable

AAqua if it's anything, these are the overloads I provide for creating a buffer in pie...

Kai•8/4/23, 9:09 PM

yeah this makes sense. Overloading void* and IntPtr is a bit tricky (and in modern code you'd prefer nint but whatever)

Lyris the Kitori•8/4/23, 9:09 PM

we dont want newcomers to just fallback to the unsafe API when they would probably be happier with the safer one, if they learned how to use it

LLyris the Kitori i feel like this line from zig zen shows some of my reasonings for being against...

Kai•8/4/23, 9:10 PM

Yes, but this is out the window given all we're doing is providing one way for each way C# provides us

KKai yeah this makes sense. Overloading void* and IntPtr is a bit tricky (and in mode...

AquaOP•8/4/23, 9:10 PM

yeah i have been moving towards nint, I wrote this code before I was aware that IntPtr == nintIntPtr == nintwhich is why it is like it is, i should change it

Kai•8/4/23, 9:10 PM

like there's multiple obvious ways to represent a data structure that does this in C#, and so we provide one obvious way to go from each possible obvious data structure design to one obvious call pattern in Silk

Kai•8/4/23, 9:10 PM

it's not on us to reduce the bullshit C# has layed upon itself for years.

AAqua yeah i have been moving towards nint, I wrote this code before I was aware that ...

Kai•8/4/23, 9:11 PM

I think (might be wrong) void* implicitly converts to nint so

LLyris the Kitori we want people to use these safer types because they would be less error prone

Perksey•8/4/23, 9:11 PM

yeah this is my reservation in providing both. like yeah i'm content in doing so, but also

please use the better types
you can still use the better types in exactly the same way as if you weren't

AquaOP•8/4/23, 9:11 PM

hmm you might be right

Maxine•8/4/23, 9:11 PM

think of it this way, if we were the BCL, we'd have two overloads, like they have overloads for byte[] and Span<byte>

Kai•8/4/23, 9:11 PM

not really

Kai•8/4/23, 9:11 PM

byte[] and Span<byte> are for legacy reasons

Maxine•8/4/23, 9:12 PM

i suppose that's true

Perksey•8/4/23, 9:12 PM

then again it is my intention to have things like the ReSharper/Rider plugin help out a lot in saying "you're using unsafe, wanna use the better, more concise way?"

Kai•8/4/23, 9:12 PM

BCL has mostly decided that Span<byte> is a strict improvement to byte[] and done that everywhere.
I might be wrong / outdated here but I don't think new APIs are introduced with byte[]byte[] where SpanSpan/ROSpanROSpan make more sense

Kai•8/4/23, 9:12 PM

(sometimes maybe for equality with old legacy APIs)

Perksey•8/4/23, 9:14 PM

yes 100%

Perksey•8/4/23, 9:14 PM

AND GUESS WHAT

PPerksey then again it is my intention to have things like the ReSharper/Rider plugin hel...

Lyris the Kitori•8/4/23, 9:14 PM

in an ideal world, rider/resharper would autocomplete parameters for all the types that are implicitly convertible to

Ptr<T>

Ptr<T>

, but i dont think its smart enough to filter by that

Perksey•8/4/23, 9:14 PM

ARRAY IMPLICITLY CASTS

Perksey•8/4/23, 9:14 PM

JUST LIKE HERE

Perksey•8/4/23, 9:14 PM

HUZZAH

Maxine•8/4/23, 9:14 PM

okay, i reckon if you understand unsafe then it is an easier bridge from byte* to Ptr<byte> than it is for someone who's a beginner and uses byte[]

LLyris the Kitori in an ideal world, rider/resharper would autocomplete parameters for all the typ...

Perksey•8/4/23, 9:14 PM

you can do that in a ReSharper plugin though

Perksey•8/4/23, 9:14 PM

i've seen plugins do that

Perksey•8/4/23, 9:14 PM

i.e. we could put

thing.AsPtr2D

thing.AsPtr2D

at the top of the list

MMaxine okay, i reckon if you understand unsafe then it is an easier bridge from byte* t...

Maxine•8/4/23, 9:15 PM

so providing unsafe overloads seems excessive

Lyris the Kitori•8/4/23, 9:15 PM

thats kindof why i prefer the name

PtrLike<T>

PtrLike<T>

or maybe

PtrRepresentable<T>

PtrRepresentable<T>

to be even more verbose

Perksey•8/4/23, 9:16 PM

yeah the name originally was PointerLike

Perksey•8/4/23, 9:16 PM

inspired by numpy actually

Lyris the Kitori•8/4/23, 9:16 PM

PtrRepresentable<T>

PtrRepresentable<T>

would indicate that whatever you pass in can be represented by a pointer, but its kindof a mouthful to type?

Maxine•8/4/23, 9:16 PM

that would kill people to type

if it's anything, these are the overloads I provide for creating a buffer in pie <https://github.com

Similar Threads