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, Jan 29, 3:12 PM
Unknown Object (File)
Dec 28 2024, 8:15 AM
Unknown Object (File)
Dec 12 2024, 2:22 AM
Unknown Object (File)
Dec 9 2024, 1:06 PM
Unknown Object (File)
Dec 8 2024, 8:21 PM
Unknown Object (File)
Nov 29 2024, 11:46 PM
Unknown Object (File)
Nov 26 2024, 5:12 PM
Unknown Object (File)
Nov 18 2024, 5:27 AM
View All 64 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 26285
Build 24767: 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 61667

View Options

sys/kern/kern_ctf.c

Loading...