Page MenuHomeFreeBSD
Differential D21531

Use zlib's uncompress function for simpler code.
ClosedPublic
Actions

Authored by ota_j.email.ne.jp on Sep 5 2019, 4:37 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 15, 10:44 AM
Unknown Object (File)
Sun, Jul 5, 7:32 PM
Unknown Object (File)
Tue, Jun 30, 12:21 PM
Unknown Object (File)
Sun, Jun 28, 1:55 PM
Unknown Object (File)
Sat, Jun 27, 1:21 PM
Unknown Object (File)
Thu, Jun 25, 4:13 AM
Unknown Object (File)
Thu, Jun 25, 2:48 AM
Unknown Object (File)
Mon, Jun 22, 10:09 PM
View All Files
Subscribers
imp

Details

Reviewers
markj
delphij
Commits
rG852336780877: kern_ctf: Use zlib's uncompress function for simpler code.
rGcb17f4a6bdaf: kern_ctf: Use zlib's uncompress function for simpler code.
Summary

uncompress() does the same job.

Test Plan

DDB_CTF compile option is on by default.
But I'm not sure how to run this section of code.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 26547
Build 24935: arc lint + arc unit

Event Timeline

ota_j.email.ne.jp created this revision.Sep 5 2019, 4:37 AM
Herald added a subscriber: imp. · View Herald TranscriptSep 5 2019, 4:37 AM
Harbormaster completed remote builds in B26285: Diff 61667.Sep 5 2019, 4:37 AM
markj accepted this revision.Sep 8 2019, 4:14 PM
markj added inline comments.
sys/kern/kern_ctf.c
247

FreeBSD style discourages CamelCase. The variable should be called something like destlen instead.

This revision is now accepted and ready to land.Sep 8 2019, 4:14 PM
ota_j.email.ne.jp updated this revision to Diff 62252.Sep 18 2019, 4:16 AM

destLen -> destlen.

This revision now requires review to proceed.Sep 18 2019, 4:16 AM
Harbormaster completed remote builds in B26547: Diff 62252.Sep 18 2019, 4:16 AM
delphij accepted this revision.Sep 18 2019, 4:38 AM

LGTM.

This revision is now accepted and ready to land.Sep 18 2019, 4:38 AM
markj accepted this revision.Sep 18 2019, 2:35 PM
Closed by commit rGcb17f4a6bdaf: kern_ctf: Use zlib's uncompress function for simpler code. (authored by ota_j.email.ne.jp, committed by delphij). · Explain WhySep 26 2021, 6:34 AM
This revision was automatically updated to reflect the committed changes.
delphij added a commit: rGcb17f4a6bdaf: kern_ctf: Use zlib's uncompress function for simpler code..
delphij added a commit: rG852336780877: kern_ctf: Use zlib's uncompress function for simpler code..Oct 10 2021, 5:10 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
24 lines

Diff 62252

View Options

sys/kern/kern_ctf.c

Loading...