dolfin team mailing list archive

[Johannes Ring <johannr@xxxxxxxxx>]

After this commit I get the following error when building DOLFIN:

[  20%] Building CXX object dolfin/CMakeFiles/dolfin.dir/log/Logger.cpp.o
/home/johannr/dev/fenics/src/dolfin/trunk/dolfin/log/Logger.cpp: In
member function ‘void dolfin::Logger::monitor_memory_usage()’:
/home/johannr/dev/fenics/src/dolfin/trunk/dolfin/log/Logger.cpp:362:7:
error: ‘_thread_monitor_memory_usage’ was not declared in this scope
/home/johannr/dev/fenics/src/dolfin/trunk/dolfin/log/Logger.cpp:369:3:
error: ‘_thread_monitor_memory_usage’ was not declared in this scope
make[2]: *** [dolfin/CMakeFiles/dolfin.dir/log/Logger.cpp.o] Error 1
make[1]: *** [dolfin/CMakeFiles/dolfin.dir/all] Error 2
make: *** [all] Error 2

The same error is reported on the buildbot.

Johannes

On Mon, Feb 11, 2013 at 7:33 PM,  <noreply@xxxxxxxxxxxxx> wrote:
> ------------------------------------------------------------
> revno: 7409
> committer: Garth N. Wells <gnw20@xxxxxxxxx>
> branch nick: trunk
> timestamp: Mon 2013-02-11 18:31:30 +0000
> message:
>   Some fixes for latest clang compiler
> modified:
>   dolfin/la/PETScKrylovSolver.cpp
>   dolfin/log/Logger.cpp
>   dolfin/log/Logger.h
>   dolfin/log/Progress.cpp
>   dolfin/log/Progress.h
>
>
> --
> lp:dolfin
> https://code.launchpad.net/~dolfin-core/dolfin/trunk
>
> Your team DOLFIN Core Team is subscribed to branch lp:dolfin.
> To unsubscribe from this branch go to https://code.launchpad.net/~dolfin-core/dolfin/trunk/+edit-subscription
>
> === modified file 'dolfin/la/PETScKrylovSolver.cpp'
> --- dolfin/la/PETScKrylovSolver.cpp     2012-12-04 11:29:59 +0000
> +++ dolfin/la/PETScKrylovSolver.cpp     2013-02-11 18:31:30 +0000
> @@ -135,7 +135,7 @@
>  //-----------------------------------------------------------------------------
>  PETScKrylovSolver::PETScKrylovSolver(std::string method,
>                                       PETScPreconditioner& preconditioner)
> -  : preconditioner(reference_to_no_delete_pointer(preconditioner)),
> + : preconditioner(reference_to_no_delete_pointer(preconditioner)),
>      preconditioner_set(false)
>  {
>    // Set parameter values
> @@ -540,7 +540,7 @@
>    PCType sub_pc_type;
>    #endif
>    PC sub_pc;
> -  KSP* sub_ksp;
> +  KSP* sub_ksp = NULL;
>    if (pc_type_str == PCASM || pc_type_str == PCBJACOBI)
>    {
>      if (pc_type_str == PCASM)
>
> === modified file 'dolfin/log/Logger.cpp'
> --- dolfin/log/Logger.cpp       2013-01-07 18:56:28 +0000
> +++ dolfin/log/Logger.cpp       2013-02-11 18:31:30 +0000
> @@ -95,8 +95,7 @@
>  //-----------------------------------------------------------------------------
>  Logger::Logger()
>    : active(true), log_level(INFO), indentation_level(0), logstream(&std::cout),
> -    num_processes(0), process_number(0),
> -    _thread_monitor_memory_usage(0), _maximum_memory_usage(-1)
> +    num_processes(0), process_number(0), _maximum_memory_usage(-1)
>  {
>    // Do nothing
>  }
>
> === modified file 'dolfin/log/Logger.h'
> --- dolfin/log/Logger.h 2013-01-07 18:34:28 +0000
> +++ dolfin/log/Logger.h 2013-02-11 18:31:30 +0000
> @@ -147,9 +147,6 @@
>      mutable std::size_t num_processes;
>      mutable std::size_t process_number;
>
> -    // Thread used for monitoring memory usage
> -    boost::thread* _thread_monitor_memory_usage;
> -
>      // Maximum memory usage so far
>      long int _maximum_memory_usage;
>
>
> === modified file 'dolfin/log/Progress.cpp'
> --- dolfin/log/Progress.cpp     2011-11-14 22:43:00 +0000
> +++ dolfin/log/Progress.cpp     2013-02-11 18:31:30 +0000
> @@ -33,8 +33,7 @@
>
>  //-----------------------------------------------------------------------------
>  Progress::Progress(std::string title, unsigned int n)
> -  : title(title), n(n), i(0), p_step(0.1), t_step(0.5), c_step(1),
> -    p(0), t(0), tc(0),
> +  : title(title), n(n), i(0), t_step(0.5), c_step(1), p(0), t(0), tc(0),
>      always(false), finished(false), displayed(false), counter(0)
>  {
>    if (n <= 0)
> @@ -53,8 +52,7 @@
>  }
>  //-----------------------------------------------------------------------------
>  Progress::Progress(std::string title)
> -  : title(title), n(0), i(0), p_step(0.1), t_step(0.5), c_step(1),
> -    p(0), t(0), tc(0),
> +  : title(title), n(0), i(0), t_step(0.5), c_step(1), p(0), t(0), tc(0),
>      always(false), finished(false), displayed(false), counter(0)
>  {
>    // LogManager::logger.progress(title, 0.0);
> @@ -118,7 +116,7 @@
>    const bool t_check = (t - this->t >= t_step - DOLFIN_EPS);
>
>    // Adjust counter step
> -  const bool fraction = 0.1;
> +  const double fraction = 0.1;
>    if (t - this->tc < fraction * t_step)
>      c_step *= 2;
>    else if (t - this->tc > t_step && c_step >= 2)
>
> === modified file 'dolfin/log/Progress.h'
> --- dolfin/log/Progress.h       2012-12-19 17:29:40 +0000
> +++ dolfin/log/Progress.h       2013-02-11 18:31:30 +0000
> @@ -103,9 +103,6 @@
>      // Current position
>      std::size_t i;
>
> -    // Minimum progress increment
> -    double p_step;
> -
>      // Minimum time increment
>      double t_step;
>
>
>