unrpyc icon indicating copy to clipboard operation
unrpyc copied to clipboard

Custom screen language statements changed their implementation

Open CensoredUsername opened this issue 1 year ago • 6 comments

Now they appear as a SLCustomUse node.

CensoredUsername avatar Feb 14 '24 20:02 CensoredUsername

Wow.They're four years in and AFAIK never appeared anywhere. At least as a error src. Are they never used or slipped simply somehow through the decompiler?

If i get this from the Ren'Py implementation right, its similar to SLUse and could be used as template. Argumentinfo and ID is gone, but the args are in another way present and we gain a positional property?

madeddy avatar Feb 14 '24 22:02 madeddy

There's not much reason to use it really. You only need them if you're doing something special in a screen that's still reasonable enough to do in a screen, yet not simple enough to just be a displayable.

No reason to work on that until there's a test case.

CensoredUsername avatar Feb 15 '24 01:02 CensoredUsername

Applicable to both 7.5 and 8.0.

CensoredUsername avatar Feb 18 '24 01:02 CensoredUsername

I add this small cosmetic/clean code issue here, because its related to print_use method in SL2:

Do we need the second arginfo call there? IMHO there should be just the "args" var from the first call passed in. Or do i make another mistake?

madeddy avatar Feb 18 '24 11:02 madeddy

Nah you're correct. Seems like the first call is just unnecessary. Minor perf fix.

CensoredUsername avatar Feb 18 '24 13:02 CensoredUsername

I add this small cosmetic/clean code issue here, because its related to print_use method in SL2:

Do we need the second arginfo call there? IMHO there should be just the "args" var from the first call passed in. Or do i make another mistake?

Fixed in 4a1ac8f

CensoredUsername avatar Feb 20 '24 01:02 CensoredUsername