Qt: Replace remaining 0 with nullptr #15114

Empact · 2019-01-06T10:34:01Z

This corrects all violations of -Wzero-as-null-pointer-constant identified in the Qt codebase.

These changes are extracted from #15112 as suggested by @MarcoFalke to ease review. This is in service of enabling -Wzero-as-null-pointer-constant, which should eliminate this as a concern going forward.

Note there are 2 non-Qt changes: src/test/allocator_tests.cpp and src/wallet/db.cpp.

Empact · 2019-01-06T10:40:11Z

src/wallet/db.cpp

https://docs.oracle.com/cd/E17076_05/html/api_reference/CXX/envversion.html

hebasto · 2019-01-06T10:49:55Z

Concept ACK.

Empact · 2019-01-06T11:12:58Z

src/qt/addresstablemodel.cpp

http://doc.qt.io/qt-5/qt.html#ItemFlag-enum

Empact · 2019-01-06T11:17:31Z

src/qt/utilitydialog.h

http://doc.qt.io/qt-5/qt.html#WindowType-enum

luke-jr · 2019-01-06T11:34:02Z

Rationale? Changing code style stuff for no reason isn't useful...

practicalswift · 2019-01-06T15:56:26Z

Concept ACK

practicalswift · 2019-01-06T16:00:21Z

@luke-jr The change @Empact is suggesting is not a code style change.

Reasons to use nullptr include:

nullptr has a well-specified (very restrictive) type, and thus works in more scenarios where type deduction might do the wrong thing on NULL or 0.
Improves readability.
Minimises surprises: nullptr cannot be confused with an int.

Enabling -Wzero-as-null-pointer-constant will help developers follow our developer guide which is a good thing. This in turn means that human reviewers will not have to point out the benefits of nullptr ever again :-)

practicalswift · 2019-01-06T16:05:01Z

utACK 60a2a0b24275757dee767deafadd86d24889bdf8

hebasto · 2019-01-06T17:22:35Z

utACK 60a2a0b24275757dee767deafadd86d24889bdf8.
nit: could clang-format-diff.py be applied?

promag · 2019-01-06T17:50:04Z

Concept ACK, agree it improves code.

I don't think this should be merged without something to prevent more zero-as-null-pointer.

practicalswift · 2019-01-06T17:56:35Z

@promag That is taken care of in #15112, right? :-)

laanwj · 2019-01-07T14:33:19Z

I don't think this should be merged without something to prevent more zero-as-null-pointer.

I agree, as discussed many times, we don't want PR after PR fixing 'straggler' occurrences. It's not that important that it warrants some kind of crusade.

But if #15112 enables failing the build on this I'm okay with that and they should probably be merged together.

DrahtBot · 2019-01-09T22:49:35Z

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

#15157 (rpc: Bumpfee units change, satoshis to BTC by benthecarman)
#15153 (gui: Add Open Wallet menu by promag)
#15101 (gui: Add WalletController by promag)
#15040 (qt: Add workaround for QProgressDialog bug on macOS by hebasto)
#13880 (gui: Try shutdown also on failure by MarcoFalke)
#12096 ([rpc] [wallet] Allow specifying the output index when using bumpfee by kallewoof)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

…ant) Qt-only changes.

Also used type-appropriate enum values such as Qt::NoItemFlags in some cases. All cases identified via -Wzero-as-null-pointer-constant

Empact · 2019-01-13T17:11:10Z

Rebased for #13216

practicalswift · 2019-01-14T08:37:23Z

utACK 3a0e76f

laanwj · 2019-01-14T14:21:25Z

utACK 3a0e76f

@MarcoFalke

3a0e76f Replace remaining 0 with nullptr in Qt code (Ben Woosley) 9096276 Don't use zero as null pointer constant (-Wzero-as-null-pointer-constant) (practicalswift) Pull request description: This corrects all violations of `-Wzero-as-null-pointer-constant` identified in the Qt codebase. These changes are extracted from bitcoin#15112 as suggested by @MarcoFalke to ease review. This is in service of enabling `-Wzero-as-null-pointer-constant`, which should eliminate this as a concern going forward. Note there are 2 non-Qt changes: `src/test/allocator_tests.cpp` and `src/wallet/db.cpp`. Tree-SHA512: 206bd668802147ba42bc413c2d7d259cb59aca9ec1da74a6bf2ca3932e60ae492faacbc61bcee0fd6b4b49a4d59d075b7e5404f0526b36c47718f9b0587e7768

@MarcoFalke

Summary: 3a0e76fc12b91b2846d756981e15f09b767a9c37 Replace remaining 0 with nullptr in Qt code (Ben Woosley) 9096276e0b2d5b7e19af9a5f3c144ef108ee55e0 Don't use zero as null pointer constant (-Wzero-as-null-pointer-constant) (practicalswift) --- Pull request description: This corrects all violations of `-Wzero-as-null-pointer-constant` identified in the Qt codebase. These changes are extracted from #15112 as suggested by @MarcoFalke to ease review. This is in service of enabling `-Wzero-as-null-pointer-constant`, which should eliminate this as a concern going forward. Note there are 2 non-Qt changes: `src/test/allocator_tests.cpp` and `src/wallet/db.cpp`. --- This is a backport from Core [[bitcoin/bitcoin#15114 | PR15114]] (bitcoin/bitcoin#15114) Test Plan: ninja check ninja check-functional Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D5341

@MarcoFalke

Summary: 3a0e76fc12b91b2846d756981e15f09b767a9c37 Replace remaining 0 with nullptr in Qt code (Ben Woosley) 9096276e0b2d5b7e19af9a5f3c144ef108ee55e0 Don't use zero as null pointer constant (-Wzero-as-null-pointer-constant) (practicalswift) --- Pull request description: This corrects all violations of `-Wzero-as-null-pointer-constant` identified in the Qt codebase. These changes are extracted from #15112 as suggested by @MarcoFalke to ease review. This is in service of enabling `-Wzero-as-null-pointer-constant`, which should eliminate this as a concern going forward. Note there are 2 non-Qt changes: `src/test/allocator_tests.cpp` and `src/wallet/db.cpp`. --- This is a backport from Core [[bitcoin/bitcoin#15114 | PR15114]] (bitcoin/bitcoin#15114) Test Plan: ninja check ninja check-functional Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D5341

@MarcoFalke

3a0e76f Replace remaining 0 with nullptr in Qt code (Ben Woosley) 9096276 Don't use zero as null pointer constant (-Wzero-as-null-pointer-constant) (practicalswift) Pull request description: This corrects all violations of `-Wzero-as-null-pointer-constant` identified in the Qt codebase. These changes are extracted from bitcoin#15112 as suggested by @MarcoFalke to ease review. This is in service of enabling `-Wzero-as-null-pointer-constant`, which should eliminate this as a concern going forward. Note there are 2 non-Qt changes: `src/test/allocator_tests.cpp` and `src/wallet/db.cpp`. Tree-SHA512: 206bd668802147ba42bc413c2d7d259cb59aca9ec1da74a6bf2ca3932e60ae492faacbc61bcee0fd6b4b49a4d59d075b7e5404f0526b36c47718f9b0587e7768

@MarcoFalke

3a0e76f Replace remaining 0 with nullptr in Qt code (Ben Woosley) 9096276 Don't use zero as null pointer constant (-Wzero-as-null-pointer-constant) (practicalswift) Pull request description: This corrects all violations of `-Wzero-as-null-pointer-constant` identified in the Qt codebase. These changes are extracted from bitcoin#15112 as suggested by @MarcoFalke to ease review. This is in service of enabling `-Wzero-as-null-pointer-constant`, which should eliminate this as a concern going forward. Note there are 2 non-Qt changes: `src/test/allocator_tests.cpp` and `src/wallet/db.cpp`. Tree-SHA512: 206bd668802147ba42bc413c2d7d259cb59aca9ec1da74a6bf2ca3932e60ae492faacbc61bcee0fd6b4b49a4d59d075b7e5404f0526b36c47718f9b0587e7768

Empact force-pushed the qt-zero-as-null-pointer-constant branch from 1667746 to 60a2a0b Compare January 6, 2019 10:35

maflcko added GUI Refactoring labels Jan 6, 2019

Empact commented Jan 6, 2019

View reviewed changes

practicalswift mentioned this pull request Jan 7, 2019

build: Optionally enable -Wzero-as-null-pointer-constant #15112

Closed

DrahtBot added the Needs rebase label Jan 12, 2019

practicalswift and others added 2 commits January 13, 2019 03:25

Don't use zero as null pointer constant (-Wzero-as-null-pointer-const…

9096276

…ant) Qt-only changes.

Replace remaining 0 with nullptr in Qt code

3a0e76f

Also used type-appropriate enum values such as Qt::NoItemFlags in some cases. All cases identified via -Wzero-as-null-pointer-constant

Empact force-pushed the qt-zero-as-null-pointer-constant branch from 60a2a0b to 3a0e76f Compare January 13, 2019 17:10

DrahtBot removed the Needs rebase label Jan 13, 2019

laanwj merged commit 3a0e76f into bitcoin:master Jan 14, 2019

ryanofsky mentioned this pull request Jan 23, 2019

Multiprocess bitcoin #10102

Draft

bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021

Qt: Replace remaining 0 with nullptr #15114

Qt: Replace remaining 0 with nullptr #15114

Uh oh!

Conversation

Empact commented Jan 6, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Empact Jan 6, 2019

Choose a reason for hiding this comment

Uh oh!

hebasto commented Jan 6, 2019

Uh oh!

Empact Jan 6, 2019

Choose a reason for hiding this comment

Uh oh!

Empact Jan 6, 2019

Choose a reason for hiding this comment

Uh oh!

luke-jr commented Jan 6, 2019

Uh oh!

practicalswift commented Jan 6, 2019

Uh oh!

practicalswift commented Jan 6, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

practicalswift commented Jan 6, 2019

Uh oh!

hebasto commented Jan 6, 2019

Uh oh!

promag commented Jan 6, 2019

Uh oh!

practicalswift commented Jan 6, 2019

Uh oh!

laanwj commented Jan 7, 2019

Uh oh!

DrahtBot commented Jan 9, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Conflicts

Uh oh!

Empact commented Jan 13, 2019

Uh oh!

practicalswift commented Jan 14, 2019

Uh oh!

laanwj commented Jan 14, 2019

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

Empact commented Jan 6, 2019 •

edited

Loading

practicalswift commented Jan 6, 2019 •

edited

Loading

DrahtBot commented Jan 9, 2019 •

edited

Loading