Page MenuHomeFreeBSD

bspatch: apply style(9)
ClosedPublic

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
F110740645: D7610.diff
Sat, Feb 22, 1:20 PM
F110726805: D7610.id19587.diff
Sat, Feb 22, 9:34 AM
F110716061: D7610.id.diff
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
Subscribers
None

Details

Summary

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

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

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

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

Thank you for doing this.

usr.bin/bsdiff/bspatch/bspatch.c
117

space in the wrong position

emaste edited edge metadata.
  • fix typo spotted by @allanjude
  • remove unnecessary brackets
usr.bin/bsdiff/bspatch/bspatch.c
36

Maybe:

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

58

return (y);

71

I know that is not direct connected by maybe:

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

227

return (0);

oshogbo edited edge metadata.
This revision is now accepted and ready to land.Aug 23 2016, 5:32 PM
This revision was automatically updated to reflect the committed changes.