MantisBT - Xdebug
View Issue Details
0001353XdebugFeature/Change requestpublic2016-10-12 05:592016-12-13 21:42
sidharth_k 
 
normalfeaturealways
acknowledgedopen 
2.4.1 
 
7.0.10-7.0.14
0001353: Make static function xdebug_dbgp_parse_option() extern
Dontbug (see https://github.com/sidkshatriya/dontbug [^] ) is a reversible debugger for PHP

Dontbug depends on Xdebug ( see Credits at https://github.com/sidkshatriya/dontbug#credits [^] )

Currently a very small patch needs to be applied to Xdebug to get it to work with Dontbug (see https://github.com/sidkshatriya/dontbug/blob/14ddeda6f527b399c6653e7e41291ccbd28a8c4b/ext/dontbug/xdebug_2_4.make_extern.dontbug.patch [^] )

Basically the patch makes the static function xdebug_dbgp_parse_option() extern.

The rationale for this patch is described here:

https://github.com/sidkshatriya/dontbug/wiki/How-the-Dontbug-Debugger-works#getting-the-value-of-variables-and-inspecting-the-call-stack-1 [^]

I was wondering if we could incorporate the patch into the Xdebug master & 2.4.x branches so that end users will not need to build a patched Xdebug locally in order to use Dontbug. (The extraneous C comments in the patch should be removed, of course, while incorporating it into Xdebug sources)

I would be very grateful if you considered this patch for inclusion. I think this will be a great help in reducing entry barriers to reverse debugging on PHP. Thank you for all the help Xdebug has already given Dontbug.
No tags attached.
Issue History
2016-10-12 05:59sidharth_kNew Issue
2016-10-19 15:40sidharth_kNote Added: 0003730
2016-10-19 15:44sidharth_kNote Edited: 0003730bug_revision_view_page.php?bugnote_id=3730#r305
2016-10-28 19:42sidharth_kNote Added: 0003731
2016-10-28 19:42sidharth_kNote Edited: 0003731bug_revision_view_page.php?bugnote_id=3731#r307
2016-12-13 21:42derickStatusnew => acknowledged
2016-12-13 21:42derickNote Added: 0004057

Notes
(0003730)
sidharth_k   
2016-10-19 15:40   
(edited on: 2016-10-19 15:44)
@derrick Hope you didn't miss this ticket. Would greatly appreciate if you could have a quick look at this! :-)

(0003731)
sidharth_k   
2016-10-28 19:42   
Filed a corresponding pull request https://github.com/xdebug/xdebug/pull/288 [^]

(0004057)
derick   
2016-12-13 21:42   
I might want to implement this, but it's not high on my list of things to do.