Update assert(3) DESCRIPTION section regards its usage.
AbandonedPublic
Actions

Authored by araujo on May 28 2018, 5:40 AM.

Details

Reviewers

brooks
imp
emaste
rgrimes
jhb

Group Reviewers

manpages

Summary

As it was discussed, I do believe we need to explicit add an explanation
about the usage of assert(3) on runtime data.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 16877
Build 16754: arc lint + arc unit

Event Timeline

araujo created this revision.May 28 2018, 5:40 AM

Herald added a reviewer: manpages. · View Herald TranscriptMay 28 2018, 5:40 AM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B16877: Diff 43063.May 28 2018, 5:40 AM

araujo added reviewers: brooks, imp, emaste, rgrimes, jhb.May 28 2018, 5:41 AM

This is not quite the right way to phrase it. Discussion is ongoing on mailing lists now, let's let that continue before revisiting this.

This revision now requires changes to proceed.May 28 2018, 6:33 PM

@imp add a better example at: https://svnweb.freebsd.org/changeset/base/334431

Dang, sorry I stepped on this review with my commit. I missed it somehow. Glad you liked mine better, but I would have coordinated had I noticed this before now.

In D15596#330255, @imp wrote:

Dang, sorry I stepped on this review with my commit. I missed it somehow. Glad you liked mine better, but I would have coordinated had I noticed this before now.

@imp Not a problem at all! The main issue was solved. Thank you again.

Revision Contents
Changeset List

Path

Size

share/

man/

man3/

assert.3

7 lines

Diff 43063

View Options

Update assert(3) DESCRIPTION section regards its usage.AbandonedPublicActions