Page MenuHomeFreeBSD
Differential D7610

bspatch: apply style(9)
ClosedPublic
Actions

Authored by emaste on Aug 23 2016, 12:36 PM.
Tags
None
Referenced Files
F110787500: D7610.diff
Sun, Feb 23, 3:37 AM
F110758779: D7610.id19590.diff
Sat, Feb 22, 5:59 PM
Unknown Object (File)
Sat, Feb 22, 1:20 PM
Unknown Object (File)
Sat, Feb 22, 9:34 AM
Unknown Object (File)
Sat, Feb 22, 6:28 AM
Unknown Object (File)
Fri, Feb 21, 3:55 PM
Unknown Object (File)
Fri, Feb 21, 3:31 PM
Unknown Object (File)
Mon, Feb 17, 3:13 PM
View All Files
Subscribers
None

Details

Reviewers
allanjude
cperciva
oshogbo
Commits
rS304691: bspatch: apply style(9)
Summary

Make style changes now in order to reduce noise in diffs for future capsicum changes (D7358 or D7603).

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste updated this revision to Diff 19580.Aug 23 2016, 12:36 PM
emaste retitled this revision from to bspatch: apply style(9).
emaste updated this object.
emaste edited the test plan for this revision. (Show Details)
emaste added reviewers: allanjude, cperciva, oshogbo.
emaste added inline comments.
usr.bin/bsdiff/bspatch/bspatch.c
148–151 ↗(On Diff #19580)

this change (and a similar one for newfd) is not specifically a style(9) change but is a small refactoring also to reduce later capsicum changes

allanjude edited edge metadata.Aug 23 2016, 2:23 PM

Thank you for doing this.

usr.bin/bsdiff/bspatch/bspatch.c
117 ↗(On Diff #19580)

space in the wrong position

emaste updated this revision to Diff 19587.Aug 23 2016, 2:36 PM
emaste edited edge metadata.
  • fix typo spotted by @allanjude
  • remove unnecessary brackets
oshogbo added inline comments.Aug 23 2016, 5:14 PM
usr.bin/bsdiff/bspatch/bspatch.c
36 ↗(On Diff #19587)

Maybe:

#include <bzlib.h>
#include <err.h>
#include <fcntl.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>

58 ↗(On Diff #19587)

return (y);

71 ↗(On Diff #19587)

I know that is not direct connected by maybe:

FILE *f, *cpf, *dpf, *epf;
BZFILE *cpfbz2, *dpfbz2, *epfbz2;

227 ↗(On Diff #19587)

return (0);

emaste updated this revision to Diff 19590.Aug 23 2016, 5:20 PM

tweaks suggested by @oshogbo

emaste added a comment.Aug 23 2016, 5:26 PM

On IRC @cperciva said no objection

oshogbo accepted this revision.Aug 23 2016, 5:32 PM
oshogbo edited edge metadata.
This revision is now accepted and ready to land.Aug 23 2016, 5:32 PM
Closed by commit rS304691: bspatch: apply style(9) (authored by emaste). · Explain WhyAug 23 2016, 5:42 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
usr.bin/
bsdiff/
bspatch/
112 lines

Diff 19591

View Options

head/usr.bin/bsdiff/bspatch/bspatch.c

Loading...