Resolve "Consider using noquote by default" - [merged] #89

Closed rainman110 closed 11 months ago

In GitLab by @mariusalexander on Aug 23, 2022, 07:27

Merges 25-consider-using-noqoute-by-default -> master

This MR enables gtDebug().noquote() by default.

Define QS_LOG_USE_QUOTE to reenable the qouting behaviour.

Define QS_LOG_USE_NOSPACE to disable the auto inserted spaces.

Closes #25

In GitLab by @mariusalexander on Aug 23, 2022, 07:42

added 1 commit

1deaf0ab - initial draft

Compare with previous version

In GitLab by @mariusalexander on Aug 23, 2022, 11:04

added 1 commit

403281cb - enable noqoute by default

Compare with previous version

In GitLab by @mariusalexander on Aug 23, 2022, 12:49

marked this merge request as ready

In GitLab by @mariusalexander on Sep 20, 2022, 10:54

added 2 commits

ccd8a1ee - renamed defines
d3b68653 - updated changelog

Compare with previous version

In GitLab by @mariusalexander on Sep 20, 2022, 10:56

Commented on src/QsLog.h line 74

#ifndef GT_LOG_USE_QUOTE

In GitLab by @mariusalexander on Sep 20, 2022, 10:57

resolved all threads

In GitLab by @mariusalexander on Sep 20, 2022, 10:57

Commented on src/QsLog.h line 74

changed this line in version 4 of the diff

In GitLab by @mariusalexander on Sep 20, 2022, 10:57

added 1 commit

9a70b056 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

In GitLab by @mariusalexander on Sep 20, 2022, 11:08

added 1 commit

a3049923 - initial draft

Compare with previous version

In GitLab by @mariusalexander on Sep 20, 2022, 11:09

approved this merge request

In GitLab by @mariusalexander on Sep 20, 2022, 11:09

unapproved this merge request

In GitLab by @mariusalexander on Sep 20, 2022, 11:09

requested review from @real-ct-ac and @jensschmeink

In GitLab by @jensschmeink on Sep 20, 2022, 11:10

approved this merge request

In GitLab by @jensschmeink on Sep 20, 2022, 11:10

I like it. :)

In GitLab by @mariusalexander on Sep 26, 2022, 08:35

mentioned in commit cb1217ccf108535db71f8fc0ba31771fce0bac3b

dlr-gtlab / gt-logging

Resolve "Consider using noquote by default" - [merged] #89