natives icon indicating copy to clipboard operation
natives copied to clipboard

fix(doc/blip): correct link and param

Open PsychoShock opened this issue 1 year ago • 4 comments

Reference for the link should be this instead of the current link commit.

This reverts citizenfx/natives#1060 commit

PsychoShock avatar Apr 25 '24 17:04 PsychoShock

Blip isn't a char* though, its an int.

AvarianKnight avatar Apr 25 '24 17:04 AvarianKnight

Yep thanks, no idea what I was thinking with char.

PsychoShock avatar Apr 25 '24 17:04 PsychoShock

why not keep it as Blip since all blip natives use that type?

freedy69 avatar Apr 26 '24 22:04 freedy69

As freddy said, changing the type here is un-needed.

AvarianKnight avatar Aug 02 '24 19:08 AvarianKnight

This change would break compatibility with the C# runtime by changing the C# codegen from RemoveBlip(ref blipIndex) -> RemoveBlip(blipIndex)

AvarianKnight avatar Aug 09 '24 04:08 AvarianKnight