Type check not working in conditional breakpoints

[sano98]

I wanted to use a conditional breakpoint that should trigger when a general function (like a superclass constructor or a function that checks all onscreen objects for mouseover) is dealing with a object of a certain type.

For my breakpoint expression, I used:

Std.is (this, ObjectType)

where all instances of this are childclasses of FlxSprite.
However, that expression never triggers. It even does not when I set ObjectType to FlxSprite.

The breakpoint does trigger for simple numerical conditions, like:

this.x < 25

[sano98]

Used versions:

hxcpp: 4.0.8
hxcpp-debug-server: 1.1.1
lime: 7.2.1
openfl: 8.8.0
hscript: 2.3.0
haxe: 4.0.0-rc.1

[nulld]

For now, iterpreter, which executes expressions for conditional breakpoints has access only to properties of object, where breakpoint was hit, and know nothing about Std.is, for example.

Maybe, some kind of custom initialization of interpreter will be usefull, how do you think?

For e.g. api can looks like:

DebugServer.interp_expose("Std", Std);

[Gama11]

Shouldn't we just expose top-level types such as Std, Math, etc by default?

[nulld]

Hm.. Sounds more user friendly than my solution - I like it :) But how many of top-level types average project use? It can significantly slowdown everything, if someone sets conditional breakpoint in game loop for e.g. ( For now, iterpreter initializes each time conditional breakpoint met )

[Gama11]

For now, iterpreter initializes each time conditional breakpoint met

Why?

[sano98]

Just from a user perspective: When I run the debugger to identify a nasty problem, I generally don't care very much about slowdowns.

Once the problem is found, I can remove the breakpoint and everything should be back to full speed, right?

[nulld]

Why?

Because, all varialbles with thier values, available in the line of breakpoint, need to be exposed to iterpreter.

[nulld]

I just thought, that we can hold all top-level types in static member of interpreter, and expose it once on debug session start - it must work pretty fast. Will try.

Can I rename an issue?

[nulld]

please test it with last commit, and reopen if it doesn't work for you for some reason

[Gama11]

Seems to basically work. :) But I guess right now stuff you want to use still needs to be exposed manually?

[Gama11]

I guess cpp.vm.Debugger.getClasses() must return an empty list in my case?

Anyway, I'll rename staticVariables to globals for now (since the classes you're adding to it aren't static vars) and make it read-only, and release a 1.2.0.

[nulld]

Found, that top-level types, declared inside hxcpp library, not presented in result of cpp.vm.Debugger.getClasses() call, so have add it manually (Math, String).

By the way, I have exposed other types too in order to use them in expressions, for e.g. flixel.FlxSprite.

[Gama11]

Seems to work great now. 👍

[sano98]

When trying to use the condition with a classtype of my project, it does not trigger.
Am I missing something?
Does this not work, as my own classes are not exposed?
Is this a resolution thing - would I have to write the absolute file path of the Apple class?
Or is it generally not possible to do what I'm trying? Thx!

[nulld]

would I have to write the absolute file path of the Apple class?

It should work, try path with package name. пт, 28 июн. 2019 г. в 10:36, sano98 <notifications@github.com>:

…

[image: grafik] <https://user-images.githubusercontent.com/8257477/60325276-388e5f00-9987-11e9-944a-907937e9ee94.png> When trying to use the condition with a classtype of my project, it does not trigger. Am I missing something? Does this not work, as my own classes are not exposed? Is this a resolution thing - would I have to write the absolute file path of the Apple class? Or is it generally not possible to do what I'm trying? Thx! — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#17?email_source=notifications&email_token=AAVQQSCV5JJLOH6XXHAF2PLP4W5PRA5CNFSM4GU5D332YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYZKL3Y#issuecomment-506635759>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAVQQSE4QRSJEDYCXKIMLUTP4W5PRANCNFSM4GU5D33Q> .