Changeset 6c2ae0 for ThirdParty/CodePatterns

Timestamp:

Apr 4, 2018, 4:58:46 PM (7 years ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Adding_MD_integration_tests, Adding_StructOpt_integration_tests, AutomationFragmentation_failures, Candidate_v1.6.1, ChemicalSpaceEvaluator, Enhanced_StructuralOptimization, Enhanced_StructuralOptimization_continued, Exclude_Hydrogens_annealWithBondGraph, Fix_Verbose_Codepatterns, ForceAnnealing_with_BondGraph, ForceAnnealing_with_BondGraph_contraction-expansion, Gui_displays_atomic_force_velocity, JobMarket_RobustOnKillsSegFaults, JobMarket_StableWorkerPool, PythonUI_with_named_parameters, StoppableMakroAction, TremoloParser_IncreasedPrecision

Children:

6491d3

Parents:

d86d97

git-author:

Frederik Heber <frederik.heber@…> (04/04/18 11:59:06)

git-committer:

Frederik Heber <frederik.heber@…> (04/04/18 16:58:46)

Message:

FIX: Verbose assumed Info::Verbosity is non-negative.

• while generally true, access to the current "tabbing level" in debugging output is not semaphore protected.
• this might run into a long loop over the whole of negative number space.

File:

• ThirdParty/CodePatterns/src/Helpers/Verbose.cpp (modified) (1 diff)

ThirdParty/CodePatterns/src/Helpers/Verbose.cpp

@@ -31 +31 @@
 ostream& Verbose::print (ostream &ost) const
 {
-  for (int i=Verbosity+Info::verbosity;i--;)
+  for (int i=Verbosity+Info::verbosity;i>0;i--)
     ost.put('\t');
   //Log() << Verbose(0) << "Verbose(.) called." << endl;