kyua: Don't pass NULL to unique_ptr constructors
ClosedPublic
Actions

Authored by jhb on Wed, Apr 30, 4:57 PM.

Details

Reviewers

olce
emaste

Commits

rG26d4ac7a2725: kyua: Don't pass NULL to unique_ptr constructors

Summary

This is ambiguous when NULL is defined to an integral constant.
Instead, use the more iodiomatic pattern of {} to construct a default
pointer.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 63805
Build 60689: arc lint + arc unit

Event Timeline

jhb created this revision.Wed, Apr 30, 4:57 PM

Herald added a subscriber: imp. · View Herald TranscriptWed, Apr 30, 4:57 PM

jhb requested review of this revision.Wed, Apr 30, 4:57 PM

jhb added a parent revision: D50089: Don't define NULL to nullptr in C++11 mode.

jhb added a child revision: D50091: kyua: Don't pass NULL to std::unique_ptr<>::reset().

Harbormaster completed remote builds in B63805: Diff 154556.Wed, Apr 30, 4:57 PM

This fixes the build with GCC.

Also included in https://github.com/freebsd/kyua/pull/265

olce accepted this revision.Thu, May 1, 9:13 AM

This revision is now accepted and ready to land.Thu, May 1, 9:13 AM

Closed by commit rG26d4ac7a2725: kyua: Don't pass NULL to unique_ptr constructors (authored by jhb). · Explain WhySat, May 3, 3:59 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG26d4ac7a2725: kyua: Don't pass NULL to unique_ptr constructors.

Revision Contents
Changeset List

Path

Size

contrib/

kyua/

utils/

process/

child.cpp

4 lines

Diff 154556

View Options

contrib/kyua/utils/process/child.cpp

kyua: Don't pass NULL to unique_ptr constructorsClosedPublicActions