comparison libinterp/octave-value/ov-fcn-handle.cc @ 18384:bd9d34f28b0f stable

Use std::ostream::fail instead of unsafe implicit bool conversion (bug #41335) * ls-mat4.cc (save_mat_binary_data): Use std::ostream::fail to return stream status instead of unsafe implicit bool conversion. * ov-cell.cc (octave_cell::save_ascii) Likewise. * ov-class.cc (octave_class::save_ascii, octave_class::save_binary) Likewise. * ov-fcn-handle.cc (octave_fcn_handle::save_ascii, octave_fcn_handle::save_binary) Likewise. * ov-struct.cc (octave_struct::save_ascii, octave_struct::save_binary, octave_scalar_struct::save_ascii, octave_scalar_struct::save_binary) Likewise.
author Mike Miller <mtmiller@ieee.org>
date Sat, 25 Jan 2014 01:02:12 -0500
parents 8c33abdd2f9a
children 1e148ecdeedd ea0d4dea1a17
comparison
equal deleted inserted replaced
18383:abc78c10acb6 18384:bd9d34f28b0f
378 378
379 for (std::list<symbol_table::symbol_record>::const_iterator 379 for (std::list<symbol_table::symbol_record>::const_iterator
380 p = vars.begin (); p != vars.end (); p++) 380 p = vars.begin (); p != vars.end (); p++)
381 { 381 {
382 if (! save_ascii_data (os, p->varval (0), p->name (), false, 0)) 382 if (! save_ascii_data (os, p->varval (0), p->name (), false, 0))
383 return os; 383 return ! os.fail ();
384 } 384 }
385 } 385 }
386 } 386 }
387 else 387 else
388 { 388 {
554 for (std::list<symbol_table::symbol_record>::const_iterator 554 for (std::list<symbol_table::symbol_record>::const_iterator
555 p = vars.begin (); p != vars.end (); p++) 555 p = vars.begin (); p != vars.end (); p++)
556 { 556 {
557 if (! save_binary_data (os, p->varval (0), p->name (), 557 if (! save_binary_data (os, p->varval (0), p->name (),
558 "", 0, save_as_floats)) 558 "", 0, save_as_floats))
559 return os; 559 return ! os.fail ();
560 } 560 }
561 } 561 }
562 } 562 }
563 else 563 else
564 { 564 {