Project

General

Profile

Feature #23673

Add debug, info, warning, critical messages throughout the code

Added by Iker de Icaza Astiz about 1 year ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Start date:
12/02/2019
Due date:
% Done:

100%

Estimated time:
Co-Assignees:
Duration:

Description

Look at the following link as a reference.

https://doc.qt.io/qt-5/debug.html#warning-and-debugging-messages


Related issues

Blocks ArtDaq RunControl GUI - Support #23729: Add debug binary productClosed12/09/2019

Blocks ArtDaq RunControl GUI - Milestone #22444: v0_01_00: Plug and PlayClosed12/09/2019

History

#3 Updated by Iker de Icaza Astiz 12 months ago

#4 Updated by Iker de Icaza Astiz 12 months ago

#5 Updated by Iker de Icaza Astiz 12 months ago

#6 Updated by Iker de Icaza Astiz 12 months ago

#7 Updated by Iker de Icaza Astiz 12 months ago

  • Assignee set to Iker de Icaza Astiz

#8 Updated by Iker de Icaza Astiz 10 months ago

Branch feature/logging_levels enables support to output different levels of messages to the terminal. Namely, DEBUG, INFO, WARNING, CRITICAL and FATAL.

An adequate colourful format has been put in place in revision 60e6b5d.

Revision bfe2451 shows how to write each level.

By default debug messages won't be shown, to have debug messaging you need to comment line

add_definitions(-DQT_NO_DEBUG_OUTPUT) # comment to have debug messages printed

on CMakeLists.txt

#9 Updated by Iker de Icaza Astiz 10 months ago

  • % Done changed from 0 to 60

Updating progress percentage.

Now we need to put a bunch of messages everywhere.

In particular, every single valid click on an active button or selection should output a debug message.

#10 Updated by Iker de Icaza Astiz 10 months ago

  • % Done changed from 60 to 90
  • Status changed from New to Feedback

Some more changes went in. Enough for now.

#11 Updated by Esteban Cristaldo 10 months ago

  • % Done changed from 90 to 100
  • Status changed from Feedback to Resolved

#12 Updated by Esteban Cristaldo 10 months ago

  • Status changed from Resolved to Closed

Agreed, messages are enough for now. Closing

Also available in: Atom PDF