| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
| |
<netinet/tcp_var.h>'s prerequisites. Prerequistes should not grow for
userland headers, and <netinet/tcp_var.h> is unfortunately still needed
in userland.
|
|
|
|
| |
Approved by: rwatson (mentor)
|
|
|
|
| |
Approved by: arved (mentor)
|
|
|
|
| |
Use WARNS?= instead of WARNS=.
|
| |
|
| |
|
|
|
|
|
|
|
| |
Use %ju and an (uintmax_t) cast to print size_t values.
Use %jd and an (intmax_t) cast to print off_t values.
Use off_t variables to hold the difference between two off_t values.
Don't bother with unsigned char where char will do nicely.
|
|
|
|
|
| |
format. The old code tried to produce the exact same output as the
pre-libfetch implementation, but I no longer see any value in this.
|
|
|
|
|
|
| |
1) Remove some unused variables.
2) Mark some things aas static or __unused.
3) Cast to make sure we're comparing the same types.
|
| |
|
|
|
|
|
|
|
| |
1) Include string.h for strcpy.
2) Don't make duplicate declaration of dump_file, we now include extern.h.
3) Help out with some constness.
4) Cast to slightly better types in some comparisons.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
read EOF from STDIN_FILENO, write zero bytes into the pseudo-terminal;
this is interpreted as an EOF by the program being scripted.
I've tested this with two non-interactive scripts:
# echo 5 | script foo sh -c 'read x; sleep $x; echo bar'
# echo bar | xargs script foo echo
and one interactive program:
# script foo more /etc/passwd
and everything seems to work properly...
PR: bin/56166, bin/57414, ports/57415, ports/60534
Approved by: rwatson (mentor)
MFC after: 2 weeks
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
parentheses if the continuation indent is exactly half of the main
indent. Indenting one contination indent for every level of
parentheses gives bad results in most cases and is not what is done
in about 90% of properly hand-formatted KNF code (sys/kern/*.c,
nvi/common/*.c). The main advantage of the non-default KNF options
-nlp -ci4 is that continuation lines don't accidentally line up with
the next main indentation level or march to the right, and increasing
their indentation defeats this.
This behaviour change is limited to when the continuation indent is
exactly half of the main indent to avoid adding yet another option.
|
| |
|
|
|
|
| |
Approved by: bms (mentor)
|
|
|
|
|
|
|
|
|
|
|
|
| |
The default is to be backwards compatible and non-KNF (use the same
indentation for locals as for globals; -ldi0 gives KNF indentation
for locals (none)). The indentation for globals also applies to struct
member names in local declatations. The indentation of variable names
in multi-line declarations is broken in various ways and this commit
gives some new variations.
indent.1:
Also clarified the description of -di<N>.
|
|
|
|
|
|
|
|
| |
nonzero (so that the 1-char indentation given by -di0 is never rendered
by a tab).
Removed garbage commented out code for setting the indentation of variable
names.
|
|
|
|
|
| |
message and wasn't quite ready (it avoided indenting the names of local
variables).
|
|
|
|
|
|
|
| |
worse than useless because it caused things like mangling of
"^int\tfoo" to "int foo" (this for N = 8). This quick fix breaks
the invariant that characters between s_code and e_code have width 1,
but nothing seems to depend on this.
|
|
|
|
|
|
|
|
|
|
| |
was mangled to "struct foo * bar". There should be an option to control
this, but no space is normal. This finishes fixing the bugs in rev.1.4.
indent(1) still doesn't really understand types in parameter lists. It
thinks keywords inside parentheses are for casts or sizeofs. This works
accidentally for scalar types and this quick fix makes it work similarly
but not so accidentally for struct/union/enum types.
|
|
|
|
|
|
|
|
|
|
| |
the number of typedef-names is not so limited. Same as in rev.1.4.
Added the "const" and "volatile" to the keyword table. Rev.1.4 added
these but they were misclassified so they were not formatted as types.
indent still doesn't really understand them. E.g., it mangles
"char * const *foo" and "char *const *foo". This change mainly stops
it mangling "char const foo" to "char<declaration-indent>const foo".
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
properly. Of the 3 changes mentioned in the log message for rev.1.4,
the first (implementing -[n]fcb) was correct but didn't touch this
file, the second (no-space-after-sizeof) was not actually done (it is
the default and is controlled by the undcoumented -[n]bs options), and
the third (no-space-after 'struct foo *') was very buggy and was reduced
to wrong comments and other style bugs by backing out the main part
of it in rev.1.6. Rev.1.4 had 2 changes which were not mentioned in
its commit log: expand specials[] so that more than -83 typedef-names
can be specified (this was the one working change in rev.1.4), and add
"const" and "volatile" to specials[] (this was buggy).
|
|
|
|
| |
- Require the "directory" argument with the -d option.
|
| |
|
| |
|
|
|
|
| |
Obtained from: KAME
|
|
|
|
| |
Obtained from: KAME
|
|
|
|
|
|
| |
- support AES XCBC MAC/AES counter mode.
Obtained from: KAME
|
|
|
|
| |
Obtained from: KAME
|
|
|
|
|
|
|
|
| |
- goodbye RC5.
- use %llu directly.
- KNF.
Obtained from: KAME
|
|
|
|
| |
Prodded by: des
|
| |
|
|
|
|
| |
Approved by: simon(mentor)
|
|
|
|
| |
Approved by: joerg (mentor)
|
| |
|
| |
|
|
|
|
| |
other systems do that in the old way.
|
| |
|
|
|
|
|
|
| |
These appear to have been inserted in the PR text by gnatsweb.
Noticed by: des (maintainer)
|
|
|
|
|
| |
Requested by: phk
Reviewed by: imp (mentor), ru
|
|
|
|
|
|
|
|
|
| |
re-breaks non-interactive portupgrade (or at least old versions of
portupgrade); I'll see if I can put together a solution which avoids
breaking anything later.
Approved by: rwatson (mentor)
Noticed by: Stefan Farfeleder, Joshua Goodall
|
|
|
|
|
| |
PR: 61354
Submitted by: Eugeny Grosbein <eugen (at) kuzbass.ru>
|
|
|
|
|
|
| |
effective user id, instead of the real user id.
MFC after: 2 weeks
|
|
|
|
|
|
| |
read fetch(3) as well.
Requested by: des (maintainer)
|
|
|
|
| |
what I intended to do). Really remove my middle name.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
the double quotes ("" and '') as a separate argument.
Reported by: ache
The fix in this and previous revisions combined is functionally
equivalent to the below patch against rev. 1.27 but the code is
now much easier to follow:
%%%
Index: str.c
===================================================================
RCS file: /home/ncvs/src/usr.bin/make/str.c,v
retrieving revision 1.27
diff -u -r1.27 str.c
--- str.c 28 Oct 2002 23:33:57 -0000 1.27
+++ str.c 25 Jan 2004 12:09:21 -0000
@@ -168,7 +168,7 @@
inquote = (char) ch;
/* Don't miss "" or '' */
if (start == NULL && p[1] == inquote) {
- start = t + 1;
+ start = t;
break;
}
}
%%%
|
|
|
|
|
| |
man pages (though not from copyright notices). While I'm here, add email
addresses where appropriate.
|
| |
|