is that a GC hole

Aqua•1/6/23, 9:58 PM

so basically
if i am understanding this
storing it as a span is the correct way to do it
instead of storing a raw pointer?
i think i have confused myself

AAqua so basically if i am understanding this storing it as a span is the correct way ...

PerkseyOP•1/6/23, 9:58 PM

well storing it as a ref

PPerksey is that a GC hole

TechPizza•1/6/23, 9:58 PM

nope, pointers to locals with GC refs are now correctly tracked

PerkseyOP•1/6/23, 9:58 PM

so that could be ref fields, or if you're on anything but .net 7 span is the way to do that

Aqua•1/6/23, 9:58 PM

ah
gotcha

TechPizza•1/6/23, 9:59 PM

so you can do this:

object local = new();
object* ptr = &local;
PassToNative(ptr);

object local = new();
object* ptr = &local;
PassToNative(ptr);

TechPizza•1/6/23, 9:59 PM

it's a bit cursed but super useful

PerkseyOP•1/6/23, 9:59 PM

TechPizza•1/6/23, 10:03 PM

way better than GC handles. i'm already using it in my V8 bindings lol

Aqua•1/6/23, 10:03 PM

so i changed the struct from the pointer to the span and it doesn't seem to be complaining so i'm happy with this, thanks!

TTechPizza way better than GC handles. i'm already using it in my V8 bindings lol

TechPizza•1/6/23, 10:04 PM

although non-pinning GCHandle is still useful for long-term stuff in native

Aqua•1/6/23, 10:06 PM

this also helped me understand my original question of why you need to deallocate the memory with silkmarshal instead of just using fixed on it so that's good too

Aqua•1/6/23, 10:06 PM

how i have had 0 issues with what i've been doing i've got no idea, i wish i had had issues with it so i could have worked out the garbage i was doing earlier

AAqua how i have had 0 issues with what i've been doing i've got no idea, i wish i *ha...

TechPizza•1/6/23, 10:07 PM

if you created large arrays they probably got sent to LOH and rarely moved

Aqua•1/6/23, 10:07 PM

ah
yeah
since it is texture data

Aqua•1/6/23, 10:08 PM

wait

Aqua•1/6/23, 10:08 PM

this is a problem

Aqua•1/6/23, 10:09 PM

okay gc handles it is i guess
hopefully

PerkseyOP•1/6/23, 10:10 PM

what's wrong with just storing a

byte[]

byte[]

again?

Aqua•1/6/23, 10:10 PM

it needs to be an

unmanaged

unmanaged

type

Aqua•1/6/23, 10:10 PM

hm though

PerkseyOP•1/6/23, 10:10 PM

TechPizza•1/6/23, 10:10 PM

why use an array when you can just make one thicc TextureData<T>

Aqua•1/6/23, 10:10 PM

i might be able to remove that constraint now
i'll test

Aqua•1/6/23, 10:11 PM

i considered that but i'd like it to be friendly for the user
i went the directx route of taking an array of subresourcedata

Aqua•1/6/23, 10:12 PM

yeah i can remove that constraint lol
because my internal createtexture function just takes a pointer to the texturedata struct anyway
it used to be that it took an array which is what necessitated that type constraint

Aqua•1/6/23, 10:16 PM

i think i'll still go the gchandle route though since i'd like textures to be able to use raw pointers if desired

TechPizza•1/6/23, 10:16 PM

a GCHandle can't pin a ref though :)

TechPizza•1/6/23, 10:16 PM

so you have to use a pointer + a construct that can free a GCHandle if array was provided

PerkseyOP•1/6/23, 10:17 PM

i need to figure out the implications of making the vast majority of structs in Silk.NET a ref struct tbf

Aqua•1/6/23, 10:17 PM

ok i officially hate garbage collected languages
why is this so much pain
i just want to store a pointer to some data lmao

TechPizza•1/6/23, 10:17 PM

then just use a pointer when

Aqua•1/6/23, 10:19 PM

b-but that's what i was doing

basically

struct TextureData {
  void* DataPointer:
}

public Texture CreateTexture(blah stuff, TextureData* datas);

struct TextureData {
  void* DataPointer:
}

public Texture CreateTexture(blah stuff, TextureData* datas);

is the gist of what i'm doing
while i'd like the user to be able to use

byte[]

byte[]

and whatnot, i'd like to internally store it all as raw pointers
which i seem to be making an absolute mess of

Aqua•1/6/23, 10:20 PM

but i'm guessing that's not something you can easily do without resorting to spans and ref structs
... that i take an array of so i can't do that

Aqua•1/6/23, 10:21 PM

i'm guessing the simple answer is that i can't use pointers for this
which is really heckndarn annoying

TechPizza•1/6/23, 10:22 PM

meh, i wouldn't bother making immutable texture creation methods nice-to-use

Aqua•1/6/23, 10:22 PM

lol, fair enough

Aqua•1/6/23, 10:29 PM

yeah my solution here is to make texturedata managed and if the user wants to use the pointer overloads then it will just take a

void*

void*

so they have to manage it themselves
seems like a decent tradeoff because it still allows the pointer overloads but also allows the nice managed method that most people will use

Aqua•1/6/23, 10:57 PM

i temporarily solved this problem by removing the pointer overloads entirely because i don't even use them and i'll figure it out another day when i actually care about pointer overloads lol

TTechPizza meh, i wouldn't bother making immutable texture creation methods nice-to-use

Redhacker2•1/7/23, 1:30 AM

That's why I make transparently mutable textures gabe

Redhacker2•1/7/23, 1:30 AM

And then you shit on me for it

TechPizza•1/7/23, 1:31 AM

resizable textures are pain

Redhacker2•1/7/23, 2:00 AM

Horrible to use APIs are pain

TTechPizza resizable textures are pain

Redhacker2•1/7/23, 2:01 AM

We're at an impasse

flyke•1/7/23, 2:20 AM

If use custom class for textures to store all pointers, id, name, gabage collector will not delete instances of such class and pointers will be not affected. Or I didnt understend the issue?
Some day I will lean how to load textured mesh with C#

Now only theory

flyke•1/7/23, 3:49 PM

Maybe anyone knows, how to use antialiasing with Silky.Nvg?
The heart image - it is the PNG texture which is 1000px in size, but rendered on screen in small size 100px and has visible jagged edges.

TechPizza•1/7/23, 4:43 PM

antialiasing won't be effective for images

flyke•1/7/23, 5:09 PM

I just want it look normal, smooth, the tecnical term is not important

Had same problem in UE and it solved by changes in texture settings.

TechPizza•1/7/23, 5:10 PM

hmm, linear instead of point clamp filtering maybe

is that a GC hole

Similar Threads