Mercurial > octave-nkf

changeset 11584:cda4aa780d58

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Another round of initialising members in the constructor initialisation list
author Pascal Dupuis <Pascal.Dupuis@uclouvain.be>
date Thu, 20 Jan 2011 17:07:26 -0500
parents c4c2cd67c440
children 1473d0cf86d2
files src/ChangeLog src/debug.h src/dynamic-ld.cc src/gl-render.cc src/gl-render.h src/gl2ps-renderer.h src/lex.h src/ls-hdf5.h src/oct-stream.h src/oct-strstrm.h src/ov-base-diag.h src/ov-base-mat.h src/ov-base-scalar.h src/ov-base-sparse.h src/ov-cell.h src/ov-class.h src/ov-dld-fcn.h src/ov-fcn-handle.h src/ov-lazy-idx.h src/ov-perm.h src/ov-range.h src/ov-struct.h src/procstream.h src/pt-assign.h src/symtab.h
diffstat 25 files changed, 88 insertions(+), 45 deletions(-) [+]
line wrap: on
line diff
--- a/src/ChangeLog	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ChangeLog	Thu Jan 20 17:07:26 2011 -0500
@@ -1,3 +1,13 @@
+2011-01-20  Pascal Dupuis <Pascal.Dupuis@worldonline.be>.
+
+	* debug.h, dynamic-ld.cc, gl-render.cc, gl-render.h,
+	gl2ps-renderer.h, lex.h, ls-hdf5.h, oct-stream.h, oct-strstrm.h,
+	ov-base-diag.h, ov-base-mat.h, ov-base-scalar.h, ov-base-sparse.h,
+	ov-cell.h, ov-class.h, ov-dld-fcn.h, ov-fcn-handle.h,
+	ov-lazy-idx.h, ov-perm.h, ov-range.h, ov-struct.h, procstream.h,
+	pt-assign.h, symtab.h, unwind-prot.h: Initialize
+	all data members in initialization list.
+
 2011-01-20  Kai Habel  <kai.habel@gmx.de>
 
 	* DLD-FUNCTIONS/__init_fltk__.cc (__fltk_uigetfile__): Append file
--- a/src/debug.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/debug.h	Thu Jan 20 17:07:26 2011 -0500
@@ -39,7 +39,7 @@
 {
 private:
 
-  bp_table (void) { }
+  bp_table (void) : bp_set () { }
 
   ~bp_table (void) { }
 
--- a/src/dynamic-ld.cc	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/dynamic-ld.cc	Thu Jan 20 17:07:26 2011 -0500
@@ -64,7 +64,7 @@
 
 private:
 
-  octave_shlib_list (void) { }
+  octave_shlib_list (void) : lib_list () { }
 
   ~octave_shlib_list (void) { }
 
@@ -203,7 +203,7 @@
 
 private:
 
-  octave_mex_file_list (void) { }
+  octave_mex_file_list (void) : file_list () { }
 
   ~octave_mex_file_list (void) { }
 
--- a/src/gl-render.cc	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/gl-render.cc	Thu Jan 20 17:07:26 2011 -0500
@@ -63,7 +63,10 @@
   class texture_rep
   {
   public:
-    texture_rep (void) : valid (false), count (1) { }
+    texture_rep (void) 
+      : id (), w (), h (), tw (), th (), tx (), ty (),
+        valid (false), count (1) 
+    { }
 
     texture_rep (GLuint id_arg, int w_arg, int h_arg, int tw_arg, int th_arg)
         : id (id_arg), w (w_arg), h (h_arg), tw (tw_arg), th (th_arg),
@@ -239,7 +242,7 @@
 
 public:
 
-  opengl_tesselator (void) : glu_tess (0) { init (); }
+  opengl_tesselator (void) : glu_tess (0), fill() { init (); }
 
   virtual ~opengl_tesselator (void)
     { if (glu_tess) gluDeleteTess (glu_tess); }
@@ -343,7 +346,9 @@
     // reference counter
     int count;
 
-    vertex_data_rep (void) : count (1) { }
+    vertex_data_rep (void) 
+      : coords (), color (), normal (), alpha (),
+        ambient (), diffuse (), specular (), specular_exp (),count (1) { }
 
     vertex_data_rep (const Matrix& c, const Matrix& col, const Matrix& n,
                      double a, float as, float ds, float ss, float se)
--- a/src/gl-render.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/gl-render.h	Thu Jan 20 17:07:26 2011 -0500
@@ -48,7 +48,14 @@
 opengl_renderer
 {
 public:
-  opengl_renderer (void) { }
+  opengl_renderer (void)
+    : toolkit (), xform (), xmin (), xmax (), ymin (), ymax (),
+    zmin (), zmax (), xZ1 (), xZ2 (), marker_id (), filled_marker_id (),
+    camera_pos (), camera_dir () 
+#if HAVE_FREETYPE
+    , text_renderer ()
+#endif 
+  { }
 
   virtual ~opengl_renderer (void) { }
 
@@ -111,7 +118,14 @@
                             GLenum type, const GLvoid *data);
 
 private:
-  opengl_renderer (const opengl_renderer&) { }
+  opengl_renderer (const opengl_renderer&) 
+    : toolkit (), xform (), xmin (), xmax (), ymin (), ymax (),
+    zmin (), zmax (), xZ1 (), xZ2 (), marker_id (), filled_marker_id (),
+    camera_pos (), camera_dir () 
+#if HAVE_FREETYPE
+    , text_renderer ()
+#endif
+    { }
 
   opengl_renderer& operator = (const opengl_renderer&)
     { return *this; }
--- a/src/gl2ps-renderer.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/gl2ps-renderer.h	Thu Jan 20 17:07:26 2011 -0500
@@ -32,7 +32,8 @@
 {
 public:
   glps_renderer (const int _fid, const std::string& _term) 
-    : opengl_renderer () , fid (_fid), term (_term) { }
+    : opengl_renderer () , fid (_fid), term (_term), 
+    fontsize (), fontname () { }
 
   ~glps_renderer (void) { }
 
--- a/src/lex.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/lex.h	Thu Jan 20 17:07:26 2011 -0500
@@ -57,16 +57,19 @@
 public:
 
   lexical_feedback (void)
+
     : bracketflag (0), braceflag (0), looping (0),
       convert_spaces_to_comma (true), at_beginning_of_statement (true),
       defining_func (0), looking_at_function_handle (false),
       looking_at_return_list (false), looking_at_parameter_list (false),
       looking_at_decl_list (false), looking_at_initializer_expression (false),
       looking_at_matrix_or_assign_lhs (false), looking_at_object_index (),
-      looking_for_object_index (false), looking_at_indirect_ref (false),
-      parsed_function_name (), parsing_class_method (false),
-      maybe_classdef_get_set_method (false), parsing_classdef (false),
-      quote_is_transpose (false), pending_local_variables ()
+      looking_for_object_index (false), do_comma_insert (false), 
+      looking_at_indirect_ref (false), parsed_function_name (), 
+      parsing_class_method (false), maybe_classdef_get_set_method (false), 
+      parsing_classdef (false), quote_is_transpose (false), 
+      pending_local_variables ()
+
     {
       init ();
     }
--- a/src/ls-hdf5.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ls-hdf5.h	Thu Jan 20 17:07:26 2011 -0500
@@ -43,11 +43,12 @@
   // keep track of current item index in the file
   int current_item;
 
-  hdf5_fstreambase () { file_id = -1; }
+  hdf5_fstreambase () : file_id (-1), current_item () { }
 
   ~hdf5_fstreambase () { close (); }
 
   hdf5_fstreambase (const char *name, int mode, int /* prot */ = 0)
+    : file_id (-1), current_item (-1)
     {
       if (mode & std::ios::in)
         file_id = H5Fopen (name, H5F_ACC_RDONLY, H5P_DEFAULT);
--- a/src/oct-stream.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/oct-stream.h	Thu Jan 20 17:07:26 2011 -0500
@@ -333,7 +333,8 @@
   octave_base_stream (std::ios::openmode arg_md = std::ios::in|std::ios::out,
                       oct_mach_info::float_format ff
                         = oct_mach_info::native_float_format ())
-    : count (0), md (arg_md), flt_fmt (ff), fail (false), open_state (true)
+    : count (0), md (arg_md), flt_fmt (ff), fail (false), open_state (true),
+      errmsg ()
   { }
 
   virtual ~octave_base_stream (void) { }
--- a/src/oct-strstrm.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/oct-strstrm.h	Thu Jan 20 17:07:26 2011 -0500
@@ -135,7 +135,7 @@
   octave_ostrstream (std::ios::openmode arg_md = std::ios::out,
                      oct_mach_info::float_format ff
                        = oct_mach_info::native_float_format ())
-    : octave_base_strstream (arg_md, ff) { }
+    : octave_base_strstream (arg_md, ff), os () { }
 
   static octave_stream
   create (std::ios::openmode arg_md = std::ios::out,
--- a/src/ov-base-diag.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-base-diag.h	Thu Jan 20 17:07:26 2011 -0500
@@ -47,14 +47,14 @@
 public:
 
   octave_base_diag (void)
-    : octave_base_value () { }
+    : octave_base_value (), matrix (), dense_cache () { }
 
   octave_base_diag (const DMT& m)
-    : octave_base_value (), matrix (m)
+    : octave_base_value (), matrix (m), dense_cache ()
   { }
 
   octave_base_diag (const octave_base_diag& m)
-    : octave_base_value (), matrix (m.matrix) { }
+    : octave_base_value (), matrix (m.matrix), dense_cache () { }
 
   ~octave_base_diag (void) { }
 
--- a/src/ov-base-mat.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-base-mat.h	Thu Jan 20 17:07:26 2011 -0500
@@ -49,7 +49,7 @@
 public:
 
   octave_base_matrix (void)
-    : octave_base_value (), typ (), idx_cache () { }
+    : octave_base_value (), matrix (), typ (), idx_cache () { }
 
   octave_base_matrix (const MT& m, const MatrixType& t = MatrixType ())
     : octave_base_value (), matrix (m), 
--- a/src/ov-base-scalar.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-base-scalar.h	Thu Jan 20 17:07:26 2011 -0500
@@ -46,7 +46,7 @@
 public:
 
   octave_base_scalar (void)
-    : octave_base_value () { }
+    : octave_base_value (), scalar () { }
 
   octave_base_scalar (const ST& s)
     : octave_base_value (), scalar (s) { }
--- a/src/ov-base-sparse.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-base-sparse.h	Thu Jan 20 17:07:26 2011 -0500
@@ -49,7 +49,9 @@
 {
  public:
  
-  octave_base_sparse (void) : octave_base_value (), typ (MatrixType ()) { }
+  octave_base_sparse (void)
+    : octave_base_value (), matrix (), typ (MatrixType ()) 
+  { }
 
   octave_base_sparse (const T& a) : octave_base_value (), matrix (a),
                                     typ (MatrixType ())
--- a/src/ov-cell.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-cell.h	Thu Jan 20 17:07:26 2011 -0500
@@ -51,16 +51,16 @@
 public:
 
   octave_cell (void)
-    : octave_base_matrix<Cell> () { }
+    : octave_base_matrix<Cell> (), cellstr_cache () { }
 
   octave_cell (const Cell& c)
-    : octave_base_matrix<Cell> (c) { }
+    : octave_base_matrix<Cell> (c), cellstr_cache () { }
 
   octave_cell (const Array<std::string>& str)
     : octave_base_matrix<Cell> (Cell (str)), cellstr_cache (new Array<std::string> (str)) { }
 
   octave_cell (const octave_cell& c)
-    : octave_base_matrix<Cell> (c) { }
+    : octave_base_matrix<Cell> (c), cellstr_cache () { }
 
   ~octave_cell (void) { }
 
--- a/src/ov-class.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-class.h	Thu Jan 20 17:07:26 2011 -0500
@@ -50,10 +50,14 @@
 public:
 
   octave_class (void)
-    : octave_base_value (), obsolete_copies (0)  { }
+    : octave_base_value (), map (), c_name (), 
+      parent_list (), obsolete_copies (0)  
+    { }
 
   octave_class (const octave_map& m, const std::string& id)
-    : octave_base_value (), map (m), c_name (id), obsolete_copies (0) { }
+    : octave_base_value (), map (m), c_name (id), 
+      parent_list (), obsolete_copies (0) 
+    { }
 
   octave_class (const octave_class& s)
     : octave_base_value (s), map (s.map), c_name (s.c_name),
--- a/src/ov-dld-fcn.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-dld-fcn.h	Thu Jan 20 17:07:26 2011 -0500
@@ -44,7 +44,9 @@
 {
 public:
 
-  octave_dld_function (void) { }
+  octave_dld_function (void) 
+    : sh_lib (), t_checked (), system_fcn_file () 
+  { }
 
   octave_dld_function (octave_builtin::fcn ff, const octave_shlib& shl,
                        const std::string& nm = std::string (),
--- a/src/ov-fcn-handle.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-fcn-handle.h	Thu Jan 20 17:07:26 2011 -0500
@@ -50,16 +50,16 @@
   static const std::string anonymous;
 
   octave_fcn_handle (void)
-    : fcn (), nm (), has_overloads (false) { }
+    : fcn (), nm (), has_overloads (false), overloads () { }
 
   octave_fcn_handle (const std::string& n)
-    : fcn (), nm (n), has_overloads (false) { }
+    : fcn (), nm (n), has_overloads (false), overloads () { }
 
   octave_fcn_handle (const octave_value& f,  const std::string& n = anonymous);
 
   octave_fcn_handle (const octave_fcn_handle& fh)
     : octave_base_value (fh), fcn (fh.fcn), nm (fh.nm),
-    has_overloads (fh.has_overloads)
+    has_overloads (fh.has_overloads), overloads ()
    {
      for (int i = 0; i < btyp_num_types; i++)
        builtin_overloads[i] = fh.builtin_overloads[i];
--- a/src/ov-lazy-idx.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-lazy-idx.h	Thu Jan 20 17:07:26 2011 -0500
@@ -35,10 +35,10 @@
 public:
 
   octave_lazy_index (void)
-    : octave_base_value () { }
+    : octave_base_value (), index (), value () { }
 
   octave_lazy_index (const idx_vector& idx)
-    : octave_base_value (), index (idx) { } 
+    : octave_base_value (), index (idx), value () { } 
 
   octave_lazy_index (const octave_lazy_index& i)
     : octave_base_value (), index (i.index), value (i.value) { } 
--- a/src/ov-perm.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-perm.h	Thu Jan 20 17:07:26 2011 -0500
@@ -35,9 +35,9 @@
 octave_perm_matrix : public octave_base_value
 {
 public:
-  octave_perm_matrix (void) : matrix () { }
+  octave_perm_matrix (void) : matrix (), dense_cache () { }
 
-  octave_perm_matrix (const PermMatrix& p) : matrix (p) { }
+  octave_perm_matrix (const PermMatrix& p) : matrix (p), dense_cache () { }
 
   octave_base_value *clone (void) const { return new octave_perm_matrix (*this); }
   octave_base_value *empty_clone (void) const { return new octave_perm_matrix (); }
--- a/src/ov-range.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-range.h	Thu Jan 20 17:07:26 2011 -0500
@@ -54,7 +54,7 @@
 public:
 
   octave_range (void)
-    : octave_base_value (), idx_cache () { }
+    : octave_base_value (), range (), idx_cache () { }
 
   octave_range (double base, double limit, double inc)
     : octave_base_value (), range (base, limit, inc), idx_cache ()
--- a/src/ov-struct.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/ov-struct.h	Thu Jan 20 17:07:26 2011 -0500
@@ -49,7 +49,7 @@
 public:
 
   octave_struct (void)
-    : octave_base_value () { }
+    : octave_base_value (), map () { }
 
   octave_struct (const octave_map& m)
     : octave_base_value (), map (m) { }
@@ -173,7 +173,7 @@
 public:
 
   octave_scalar_struct (void)
-    : octave_base_value () { }
+    : octave_base_value (), map () { }
 
   octave_scalar_struct (const octave_scalar_map& m)
     : octave_base_value (), map (m) { }
--- a/src/procstream.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/procstream.h	Thu Jan 20 17:07:26 2011 -0500
@@ -36,7 +36,7 @@
 {
 public:
 
-  procstreambase (void) { pb_init (); }
+  procstreambase (void) : pb () { pb_init (); }
 
   procstreambase (const std::string& name, int mode);
 
--- a/src/pt-assign.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/pt-assign.h	Thu Jan 20 17:07:26 2011 -0500
@@ -46,8 +46,8 @@
 
   tree_simple_assignment (bool plhs = false, int l = -1, int c = -1,
                           octave_value::assign_op t = octave_value::op_asn_eq)
-    : tree_expression (l, c), lhs (0), rhs (0), preserve (plhs), etype (t),
-      first_execution (true) { }
+    : tree_expression (l, c), lhs (0), rhs (0), preserve (plhs), ans_ass (),
+      etype (t), first_execution (true) { }
 
   tree_simple_assignment (tree_expression *le, tree_expression *re,
                           bool plhs = false, int l = -1, int c = -1,
--- a/src/symtab.h	Thu Jan 20 20:31:30 2011 +0100
+++ b/src/symtab.h	Thu Jan 20 17:07:26 2011 -0500
@@ -529,9 +529,9 @@
 
       fcn_info_rep (const std::string& nm)
         : name (nm), subfunctions (), private_functions (),
-          class_constructors (), class_methods (), cmdline_function (),
-          autoload_function (), function_on_path (), built_in_function (),
-          count (1) { }
+          class_constructors (), class_methods (), dispatch_map (),
+          cmdline_function (), autoload_function (), function_on_path (), 
+          built_in_function (), count (1) { }
 
       octave_value load_private_function (const std::string& dir_name);
 
@@ -1947,7 +1947,7 @@
   static context_id xcurrent_context;
 
   symbol_table (void)
-    : table_name (), table (), curr_fcn (0) { }
+    : table_name (), table (), curr_fcn (0), persistent_table () { }
 
   ~symbol_table (void) { }