comparison libinterp/corefcn/graphics.cc @ 17861:870f3e12e163

maint: Use phrase "FIXME:" for problem areas in code. * libinterp/corefcn/betainc.cc, libinterp/corefcn/bitfcns.cc, libinterp/corefcn/display.cc, libinterp/corefcn/gammainc.cc, libinterp/corefcn/gl2ps-renderer.cc, libinterp/corefcn/graphics.cc, libinterp/corefcn/load-save.cc, libinterp/corefcn/ls-hdf5.cc, libinterp/corefcn/ls-mat5.cc, libinterp/corefcn/ls-oct-binary.cc, libinterp/corefcn/luinc.cc, libinterp/corefcn/mex.cc, libinterp/corefcn/regexp.cc, libinterp/corefcn/sighandlers.cc, libinterp/corefcn/sparse-xpow.cc, libinterp/corefcn/urlwrite.cc, libinterp/corefcn/variables.cc, libinterp/dldfcn/__init_fltk__.cc, libinterp/dldfcn/__magick_read__.cc, libinterp/dldfcn/chol.cc, libinterp/octave-value/ov-bool-sparse.cc, libinterp/octave-value/ov-fcn-inline.cc, libinterp/octave-value/ov-flt-re-mat.cc, libinterp/octave-value/ov-re-mat.cc, libinterp/octave-value/ov-str-mat.cc, libinterp/parse-tree/lex.ll, liboctave/numeric/SparseCmplxQR.cc, liboctave/numeric/SparseQR.cc, liboctave/numeric/sparse-dmsolve.cc, liboctave/util/lo-utils.cc, liboctave/util/sparse-util.cc, liboctave/util/url-transfer.cc: maint: Use phrase "FIXME:" for problem areas in code.
author Rik <rik@octave.org>
date Wed, 06 Nov 2013 11:42:35 -0800
parents 7533f4482006
children 6a71e5030df5 2e7cad6f180c 5e99fd6ea758
comparison
equal deleted inserted replaced
17860:93edd282a2bc 17861:870f3e12e163
3128 mark_modified (); 3128 mark_modified ();
3129 } 3129 }
3130 } 3130 }
3131 } 3131 }
3132 3132
3133 // FIXME This should update monitorpositions and pointerlocation, but 3133 // FIXME: This should update monitorpositions and pointerlocation, but
3134 // as these properties are yet used, and so it doesn't matter that they 3134 // as these properties are yet used, and so it doesn't matter that they
3135 // aren't set yet. 3135 // aren't set yet.
3136 void 3136 void
3137 root_figure::properties::update_units (void) 3137 root_figure::properties::update_units (void)
3138 { 3138 {
6204 if (min_val == 0 && max_val == 0) 6204 if (min_val == 0 && max_val == 0)
6205 { 6205 {
6206 min_val = -1; 6206 min_val = -1;
6207 max_val = 1; 6207 max_val = 1;
6208 } 6208 }
6209 // FIXME -- maybe this test should also be relative? 6209 // FIXME: maybe this test should also be relative?
6210 else if (std::abs (min_val - max_val) 6210 else if (std::abs (min_val - max_val)
6211 < sqrt (std::numeric_limits<double>::epsilon ())) 6211 < sqrt (std::numeric_limits<double>::epsilon ()))
6212 { 6212 {
6213 min_val -= 0.1 * std::abs (min_val); 6213 min_val -= 0.1 * std::abs (min_val);
6214 max_val += 0.1 * std::abs (max_val); 6214 max_val += 0.1 * std::abs (max_val);
6244 6244
6245 double lo = (lims.get ().matrix_value ()) (0); 6245 double lo = (lims.get ().matrix_value ()) (0);
6246 double hi = (lims.get ().matrix_value ()) (1); 6246 double hi = (lims.get ().matrix_value ()) (1);
6247 bool is_negative = lo < 0 && hi < 0; 6247 bool is_negative = lo < 0 && hi < 0;
6248 double tmp; 6248 double tmp;
6249 // FIXME should this be checked for somewhere else? (i.e. set{x,y,z}lim) 6249 // FIXME: should this be checked for somewhere else? (i.e. set{x,y,z}lim)
6250 if (hi < lo) 6250 if (hi < lo)
6251 { 6251 {
6252 tmp = hi; 6252 tmp = hi;
6253 hi = lo; 6253 hi = lo;
6254 lo = tmp; 6254 lo = tmp;
6271 6271
6272 double tick_sep = calc_tick_sep (lo , hi); 6272 double tick_sep = calc_tick_sep (lo , hi);
6273 6273
6274 if (is_logscale && ! (xisinf (hi) || xisinf (lo))) 6274 if (is_logscale && ! (xisinf (hi) || xisinf (lo)))
6275 { 6275 {
6276 // FIXME - what if (hi-lo) < tick_sep? 6276 // FIXME: what if (hi-lo) < tick_sep?
6277 // ex: loglog ([1 1.1]) 6277 // ex: loglog ([1 1.1])
6278 tick_sep = std::max (tick_sep, 1.); 6278 tick_sep = std::max (tick_sep, 1.);
6279 tick_sep = std::ceil (tick_sep); 6279 tick_sep = std::ceil (tick_sep);
6280 } 6280 }
6281 6281