View Issue Details

IDProjectCategoryView StatusLast Update
0000137XdebugFeature/Change requestpublic2006-01-24 22:32
ReportermccabeAssigned Toderick 
PrioritynormalSeveritytrivialReproducibilityalways
Status resolvedResolutionfixed 
Product Version 
Target VersionFixed in Version 
Summary0000137: feature_get for general commands
DescriptionIf you query xdebug with "feature_get -n breakpoint_set" you will get a reply:
<response command="feature_get" transaction_id="2"
feature_name="breakpoint_set" supported="1"></response>

Shouldn't the text data of the tag be set as well?

ie, the default case of DBGP_FUNC(feature_get) should be changed to:
        XDEBUG_STR_CASE_DEFAULT
                xdebug_xml_add_text(*retval, xdstrdup(lookup_cmd(CMD_OPTION('n')) ? "1" : "0"));
            xdebug_xml_add_attribute(*retval, "supported", lookup_cmd(CMD_OPTION('n')) ? "1" : "0");
        XDEBUG_STR_CASE_DEFAULT_END

?
        XDEBUG_STR_CASE_DEFAULT
            xdebug_xml_add_attribute(*retval, "supported", lookup_cmd(CMD_OPTION('n')) ? "1" : "0");
        XDEBUG_STR_CASE_DEFAULT_END
TagsNo tags attached.
Operating System
PHP Version4.3.10

Activities

derick

2006-01-24 22:32

administrator   ~0000358

Fixed in CVS - thanks for your report.

Issue History

Date Modified Username Field Change
2005-07-31 15:15 mccabe New Issue
2006-01-24 22:32 derick Status new => resolved
2006-01-24 22:32 derick Resolution open => fixed
2006-01-24 22:32 derick Assigned To => derick
2006-01-24 22:32 derick Note Added: 0000358