Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [Dltk-dev] why do breakpoints have the full system path as the location?

Hi Johan,

It looks like we can use workspace-local path for the workspace resources. Full path is required only for the external modules.

I have committed the change to create new breakpoints with workspace-local path. Also I have added some checks so that double clicking old breakpoint in the editor will find the previous one and delete it and do not create the new one.

As far as I can test it - everything works correctly.

Regards,
Alex

Johan Compagner wrote:
the dltk (or at least javascript) breakpoints are not really distributable because there is one property that has the complete system path in it

I got a workspace from a co worker and i notice that i had breakpoints but toggle it didnt work (i got another one) when i investigated this i did see that there is a location property of the marker that is a full system path
for example if i export breakpoints (eclipse feature) then i get this:

<breakpoint enabled="true" persistant="true" registered="true">
<resource path="/main/globals.js" type="1"/>
<marker charStart="338" lineNumber="15" type="org.eclipse.dltk.debug.scriptLineBreakpointMarker"> <attrib name="message" value="Javascript: globals.js: [line: 15], 0 hits"/> <attrib name="org.eclipse.debug.core.id <http://org.eclipse.debug.core.id>" value="org.eclipse.dltk.debug.javascriptModel"/> <attrib name="location" value="C:/servoy4/workspace/infogroen/main/globals.js"/>
<attrib name="charStart" value="338"/>
<attrib name="charEnd" value="350"/>
<attrib name="org.eclipse.dltk.debug.id <http://org.eclipse.dltk.debug.id>" value="p5"/>
<attrib name="org.eclipse.dltk.debug.hit_count" value="0"/>
<attrib name="org.eclipse.debug.core.enabled" value="true"/>
</marker>


that location attribute seems wrong to me
i would say location should just be the resource path
Is that path also used for sending to the debugger engine? I guess we should make it a full path then instead of keeping the full path like we do now

johan

------------------------------------------------------------------------

_______________________________________________
dltk-dev mailing list
dltk-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/dltk-dev


Back to the top