Skip to content

Commit

Permalink
We should output & instead of & in an HTML context
Browse files Browse the repository at this point in the history
  • Loading branch information
derickr committed Sep 27, 2015
1 parent b87a5bd commit 86346cb
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion tests/xdebug_var_dump.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ xdebug.file_link_format=
<i>private</i> 'timestamp' <font color='#888a85'>=&gt;</font> <small>int</small> <font color='#4e9a06'>1092515106</font>
<i>private</i> 'user_defined' <font color='#888a85'>=&gt;</font> <small>boolean</small> <font color='#75507b'>true</font>
<i>private</i> 'self' <font color='#888a85'>=&gt;</font>
<i>&</i><b>object</b>(<i>TimeStuff</i>)[<i>1</i>]
<i>&amp;</i><b>object</b>(<i>TimeStuff</i>)[<i>1</i>]
<i>protected</i> 'tm' <font color='#888a85'>=&gt;</font>
<b>array</b> <i>(size=11)</i>
'seconds' <font color='#888a85'>=&gt;</font> <small>int</small> <font color='#4e9a06'>6</font>
Expand Down
2 changes: 1 addition & 1 deletion tests/xdebug_var_dump_file.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ xdebug.file_link_format=
<i>private</i> 'timestamp' <font color='#888a85'>=&gt;</font> <small>int</small> <font color='#4e9a06'>1092515106</font>
<i>private</i> 'user_defined' <font color='#888a85'>=&gt;</font> <small>boolean</small> <font color='#75507b'>true</font>
<i>private</i> 'self' <font color='#888a85'>=&gt;</font>
<i>&</i><b>object</b>(<i>TimeStuff</i>)[<i>1</i>]
<i>&amp;</i><b>object</b>(<i>TimeStuff</i>)[<i>1</i>]
<i>protected</i> 'tm' <font color='#888a85'>=&gt;</font>
<b>array</b> <i>(size=11)</i>
'seconds' <font color='#888a85'>=&gt;</font> <small>int</small> <font color='#4e9a06'>6</font>
Expand Down
2 changes: 1 addition & 1 deletion tests/xdebug_var_dump_file_link.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ xdebug.file_link_format=xdebug://%f@%l
<i>private</i> 'timestamp' <font color='#888a85'>=&gt;</font> <small>int</small> <font color='#4e9a06'>1092515106</font>
<i>private</i> 'user_defined' <font color='#888a85'>=&gt;</font> <small>boolean</small> <font color='#75507b'>true</font>
<i>private</i> 'self' <font color='#888a85'>=&gt;</font>
<i>&</i><b>object</b>(<i>TimeStuff</i>)[<i>1</i>]
<i>&amp;</i><b>object</b>(<i>TimeStuff</i>)[<i>1</i>]
<i>protected</i> 'tm' <font color='#888a85'>=&gt;</font>
<b>array</b> <i>(size=11)</i>
'seconds' <font color='#888a85'>=&gt;</font> <small>int</small> <font color='#4e9a06'>6</font>
Expand Down
4 changes: 2 additions & 2 deletions xdebug_var.c
Original file line number Diff line number Diff line change
Expand Up @@ -2372,7 +2372,7 @@ void xdebug_var_export_fancy(zval **struc, xdebug_str *str, int level, int debug
xdebug_str_add(str, xdebug_sprintf("%*s...\n", (level * 4) - 2, ""), 1);
}
} else {
xdebug_str_addl(str, "<i>&</i><b>array</b>\n", 21, 0);
xdebug_str_addl(str, "<i>&amp;</i><b>array</b>\n", 21, 0);
}
break;

Expand Down Expand Up @@ -2402,7 +2402,7 @@ void xdebug_var_export_fancy(zval **struc, xdebug_str *str, int level, int debug
xdebug_str_add(str, xdebug_sprintf("%*s...\n", (level * 4) - 2, ""), 1);
}
} else {
xdebug_str_add(str, xdebug_sprintf("<i>&</i><b>object</b>(<i>%s</i>)", STR_NAME_VAL(Z_OBJCE_P(*struc)->name)), 1);
xdebug_str_add(str, xdebug_sprintf("<i>&amp;</i><b>object</b>(<i>%s</i>)", STR_NAME_VAL(Z_OBJCE_P(*struc)->name)), 1);
xdebug_str_add(str, xdebug_sprintf("[<i>%d</i>]\n", Z_OBJ_HANDLE_P(*struc)), 1);
}
if (is_temp) {
Expand Down

0 comments on commit 86346cb

Please sign in to comment.