Skip to content

Conversation

@lemz1
Copy link

@lemz1 lemz1 commented Feb 8, 2026

This pr fixes inline abstract functions that assign to this directly.

script.hxc:

import flixel.FlxBasic;
import flixel.util.FlxColor;

class TestScriptedClass extends FlxBasic {
    public function new() {
        super();

        var color = FlxColor.RED;
        trace(color);
        FlxColor.setRGB(color, 255, 255, 255, 255);
        trace(color);
    }
}

before:

hscriptClass(mods/introMod/script.hxc):9: -65536
hscriptClass(mods/introMod/script.hxc):11: -65536

after:

hscriptClass(mods/introMod/script.hxc):9: -65536
hscriptClass(mods/introMod/script.hxc):11: -1

@lemz1 lemz1 marked this pull request as draft February 8, 2026 02:30
@lemz1 lemz1 force-pushed the fix/inline-abstract-functions branch from 87eb833 to d809fac Compare February 8, 2026 04:50
@lemz1 lemz1 marked this pull request as ready for review February 8, 2026 04:50
this also includes a rework of abstractClassStatics
they are now moved into the new PolymodImpl class
@lemz1 lemz1 force-pushed the fix/inline-abstract-functions branch from d878bc6 to e142f43 Compare February 8, 2026 04:54
continue;
if (k != MethInline) continue;
if (abstractPath.startsWith('cpp')) continue;
if (abstractPath.startsWith('flixel.graphics.atlas.HashOrArray')) continue; // has to be ragebait
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HashOrArray seems to not work with the type parameter being Dynamic
if someone knows a fix or workaround please let me know

@AbnormalPoof AbnormalPoof self-requested a review February 11, 2026 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant