Skip to content

Commit 7b5d110

Browse files
committed
gnuplot -e "print GPVAL_TERMINALS" outputs to STDERR.
Therefore use gnuplot_errors to get the terminals. (Currently only logged, not used anywhere...)
1 parent e6601c4 commit 7b5d110

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/wxMaxima.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -3685,7 +3685,7 @@ void wxMaxima::VariableActionGnuplotCommand(const wxString &value) {
36853685
if (wxExecute(gnuplot_terminal_command, gnuplot_output, gnuplot_errors, wxEXEC_SYNC | wxEXEC_HIDE_CONSOLE | wxEXEC_MAKE_GROUP_LEADER, NULL) < 0)
36863686
wxLogMessage(_("Cannot start gnuplot"));
36873687
else {
3688-
wxString gnuplot_terminals = wxJoin(gnuplot_output, ' ');
3688+
wxString gnuplot_terminals = wxJoin(gnuplot_errors, ' ', 0); // REMARK: In the documentation of wxJoin, NULL is suggested as 3rd argument ("If the escape character is non-NULL, ..."), but NULL causes an warning. (warning: passing NULL to non-pointer argument 3). Therefore I use 0 here.
36893689
wxLogMessage(wxS("Gnuplot terminals: ") + gnuplot_terminals);
36903690
/* FIXME: What should happen with the result? Returned by the function? Stored anywhere? Currently the result is nowhere used... */
36913691
}

0 commit comments

Comments
 (0)