These are chat archives for derekparker/delve

26th
Jun 2015
Nan Xiao
@NanXiao
Jun 26 2015 00:21
@derekparker This morning, it works OK. Very weird
@derekparker BTW, I find another issue, the si command seems can't take effect immediately. I must execute twice or more, the debugger can go into the function.
delve_si.JPG
Derek Parker
@derekparker
Jun 26 2015 12:32
@NanXiao yeah the si command steps a single instruction, it does not immediately step into a function
Could be useful to change it though, or have another command that intentionally steps into a function
Alessandro Arzilli
@aarzilli
Jun 26 2015 13:48
the thing that happened to @NanXiao also happened to me once but it went away before I could debug it
Derek Parker
@derekparker
Jun 26 2015 14:48
@aarzilli that's odd.. I haven't ran into it yet. I wonder if there's a race somewhere.
Alessandro Arzilli
@aarzilli
Jun 26 2015 14:59
My guess is that the server can't start the process or fails to attach to it for some reason and dies and the client doesn't know
Derek Parker
@derekparker
Jun 26 2015 15:00
@aarzilli could be.
Alessandro Arzilli
@aarzilli
Jun 26 2015 15:00
for example if you run "dlv ./somethingthatdoesntexist" and then execute "continue"
it hangs forever
not the same thing but it could be something similar
Derek Parker
@derekparker
Jun 26 2015 15:01
@aarzilli pushed a few things up that cause some conflicts with your branch on thread.Step, you should be able to solve them by replacing the body with your code
@aarzilli definitely a bug in any case, mind filing a bug report?
Alessandro Arzilli
@aarzilli
Jun 26 2015 15:02
yes, I saw that I will rebase, tell me what you think about the rest of the pull request
Derek Parker
@derekparker
Jun 26 2015 15:02
I gave some feedback on the PR, just some minor things, but looks great, thanks a lot
Alessandro Arzilli
@aarzilli
Jun 26 2015 15:02
thanks
Derek Parker
@derekparker
Jun 26 2015 15:09
Also, if you see a conflict on thread.run in thread.Step ignore it and re pull master and rebase again
I pulled it out to make rebasing easier and because it really wasn't necessary
\cc @aarzilli