AGS 4: Script API: add "split options" for String.Split method about ags HOT 6 CLOSED

I agree, it's a good convenience for several cases.

from ags.

It misses the actual flags, but I believe the gist of the changes would be something like this

ericoporto@de5d29c

Not sure if there is a pretty way to get the trim in a single function.

from ags.

Uhm, I wonder if a string like " , , , ," is passed with both flags, what the result should be, would it be null or does an empty array exists in AGS script?

from ags.

The trim implementation from above commit looks fine, similar to what I've imagined.
(I'd only use camelcase function names similar to all names around, according to our code style)

Uhm, I wonder if a string like " , , , ," is passed with both flags, what the result should be, would it be null or does an empty array exists in AGS script?

AGS does not currently allow creating a zero-length dynamic array in script (with the new operator). But if this is a matter of convenience, we may investigate if it's possible to support such array, and return one. In fact, maybe it's already technically possible.

from ags.

I put a PR in #2461, please test it!

from ags.

Resolved by #2461

from ags.