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
F102860271: D21531.diff
Mon, Nov 18, 2:49 AM
Unknown Object (File)
Fri, Nov 1, 12:40 PM
Unknown Object (File)
Tue, Oct 22, 6:51 PM
Unknown Object (File)
Oct 4 2024, 3:27 PM
Unknown Object (File)
Oct 4 2024, 8:50 AM
Unknown Object (File)
Oct 2 2024, 12:04 AM
Unknown Object (File)
Sep 26 2024, 5:49 PM
Unknown Object (File)
Sep 23 2024, 8:55 AM
View All 54 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...