Matthew Dillon [Mon, 21 Mar 2005 05:46:40 +0000 (05:46 +0000)]
Add Windoz autorun support to the CD to automatically pop up a browser with a
nice DragonFly information page and hotlinks to various DFly resources.
Submitted-by: Eduardo Tongson <propolice@gmail.com>
Chris Pressey [Mon, 21 Mar 2005 04:52:34 +0000 (04:52 +0000)]
Punctuation, formatting, grammar, and spelling nits.
Chris Pressey [Mon, 21 Mar 2005 04:07:13 +0000 (04:07 +0000)]
Update the information regarding the installer and the pfi
boot-configuration system.
Chris Pressey [Mon, 21 Mar 2005 03:58:04 +0000 (03:58 +0000)]
Update installer to version 1.1.5. Highlights include:
- ESC key can be used to return to previous form or menu.
- Better error dialog when network can't be configured as specified.
- Fixed a case where "Configure an Installed System" could fail due
to the device node not existing (added an appropriate MAKEDEV)
- A friendly 'splash screen' (index.html) installed for CGI frontend.
Jeffrey Hsu [Mon, 21 Mar 2005 01:56:22 +0000 (01:56 +0000)]
When doing Limited Transmit, don't retract snd_nxt if it was previously
the same as snd_max.
Submitted by: Noritoshi Demizu <demizu@dd.iij4u.or.jp>
Jeffrey Hsu [Mon, 21 Mar 2005 01:49:43 +0000 (01:49 +0000)]
Ensure that Limited Transmit always sends new data, even after a
retransmission timeout.
Reported by: Noritoshi Demizu <demizu@dd.iij4u.or.jp>
Joerg Sonnenberger [Sun, 20 Mar 2005 18:45:15 +0000 (18:45 +0000)]
Override _kevent, not kevent. This should fix the DNS issue, since
the libc resolver was calling _kevent directly, which prevented
libc_r from switching to another thread.
Suggested-by: Jonathan Lemon
Problem-reports: Brock Johnson, Laurent Sartran and others
Special-thanks-to: Laurent Sartran for tracking down the timeframe
Joerg Sonnenberger [Sun, 20 Mar 2005 17:46:31 +0000 (17:46 +0000)]
Add kqueue overwrite for libc_r. We have to trace the opened descriptor.
Obtained-from: FreeBSD
Joerg Sonnenberger [Sun, 20 Mar 2005 14:57:28 +0000 (14:57 +0000)]
Minor style changes.
Liam J. Foy [Sun, 20 Mar 2005 14:07:43 +0000 (14:07 +0000)]
- Style(9)
- Bump WARNS to 6
Submitted by: Sepherosa Ziehau <sepherosa@softhome.net>
Max Okumoto [Sun, 20 Mar 2005 12:25:40 +0000 (12:25 +0000)]
patch-7.163
Use VarParse() instead of Var_Parse() wapper within var.c
Max Okumoto [Sun, 20 Mar 2005 12:25:19 +0000 (12:25 +0000)]
patch-7.162
Use the same names for variables passed to Var_Parse().
Max Okumoto [Sun, 20 Mar 2005 12:24:40 +0000 (12:24 +0000)]
patch-7.161
Use same code structure for code using VarFind().
Max Okumoto [Sun, 20 Mar 2005 12:24:13 +0000 (12:24 +0000)]
tch-7.160
Removed some extra parans.
Max Okumoto [Sun, 20 Mar 2005 12:23:53 +0000 (12:23 +0000)]
patch-7.158
- Change VarGetPattern() to return a Buffer object.
- Change struct VarPattern to store Buffer objects instead of
a 'char *' and len.
Max Okumoto [Sun, 20 Mar 2005 12:23:25 +0000 (12:23 +0000)]
patch-7.157
Use lhs field in VarPattern object instead of variable re.
Max Okumoto [Sun, 20 Mar 2005 12:22:46 +0000 (12:22 +0000)]
patch-7.156
Merge struct VarPattern and struct VarREPattern. This will help
factor out more common code.
Chris Pressey [Sun, 20 Mar 2005 03:54:01 +0000 (03:54 +0000)]
Add the pfi_curses_escdelay variable. When using the curses frontend,
this variable determines the amount of time, in milliseconds, that
must pass after ESC is pressed, in order for it to be recognized as a
plain ESC keystroke and not as the start of an escape code. See the
ESCDELAY entry in the ncurses manual page for more information.
Max Okumoto [Sun, 20 Mar 2005 00:44:45 +0000 (00:44 +0000)]
Clean up Var_SetEnv().
YONETANI Tomokazu [Sat, 19 Mar 2005 19:49:05 +0000 (19:49 +0000)]
Remove unused version.
YONETANI Tomokazu [Sat, 19 Mar 2005 19:38:31 +0000 (19:38 +0000)]
Update ACPI build wrappers to use new ACPICA(
20050309) code.
The previously imported version(
20050211) still had
AE_AML_BUFFER_LIMIT issue, which was fixed in the release
made just after the import.
YONETANI Tomokazu [Sat, 19 Mar 2005 19:24:31 +0000 (19:24 +0000)]
Import acpica-
20050309 from Intel.
http://developer.intel.com/technology/iapc/acpi/downloads/acpica-unix-
20050309.tar.gz
Sascha Wildner [Sat, 19 Mar 2005 18:57:34 +0000 (18:57 +0000)]
Fix order for SI_SUB_TUNABLES.
Liam J. Foy [Sat, 19 Mar 2005 17:43:18 +0000 (17:43 +0000)]
- Make sure we call exit() after clnt_pcreateerror()
- Better parsing of passed options
- Fix (xdrproc_t) warnings
- Bump WARNS 6
Joerg Sonnenberger [Sat, 19 Mar 2005 14:56:08 +0000 (14:56 +0000)]
Add /etc/firmware, remove /etc/gnats.
Joerg Sonnenberger [Sat, 19 Mar 2005 14:54:50 +0000 (14:54 +0000)]
Generic firmware support. Currently implemented is loading from
/etc/firmware, support kernel builtin images and kernel modules
will follow later.
Written-by: Johannes Hofmann and Joerg Sonnenberger
Max Okumoto [Sat, 19 Mar 2005 10:44:07 +0000 (10:44 +0000)]
Reindent stripvarname() to use 8 space tabs.
Max Okumoto [Sat, 19 Mar 2005 00:21:19 +0000 (00:21 +0000)]
patch-7.155
Sort function proto types
Max Okumoto [Sat, 19 Mar 2005 00:20:55 +0000 (00:20 +0000)]
patch-7.154
Fix indent VarPattern in var.h
Max Okumoto [Sat, 19 Mar 2005 00:19:55 +0000 (00:19 +0000)]
patch-7.151
- Use Buf_Data() to simplify code.
- Use Buf_Peel() to simplify code.
Max Okumoto [Sat, 19 Mar 2005 00:19:13 +0000 (00:19 +0000)]
patch-7.150
- Use Buf_Data() to simplify code.
- Use Buf_Peel() to simplify code.
Max Okumoto [Sat, 19 Mar 2005 00:18:50 +0000 (00:18 +0000)]
patch-7.149
- Use Buf_Data() to simplify code.
- Use Buf_Peel() to simplify code.
Max Okumoto [Sat, 19 Mar 2005 00:18:28 +0000 (00:18 +0000)]
patch-7.148
Removed warning by using Buf_Peel() to simplify code.
Max Okumoto [Sat, 19 Mar 2005 00:18:01 +0000 (00:18 +0000)]
patch-7.147
Sort function declarations in buf.h
Max Okumoto [Sat, 19 Mar 2005 00:17:40 +0000 (00:17 +0000)]
patch-7.146
Add new functions:
Buf_Data() - return a reference to the data contained in the buffer.
Buf_AppendBuf(a, b) - append a copy of thecontent of b into a.
Max Okumoto [Sat, 19 Mar 2005 00:17:07 +0000 (00:17 +0000)]
patch-7.145
Replace redundent code with calls to VarGetPattern().
Max Okumoto [Sat, 19 Mar 2005 00:16:19 +0000 (00:16 +0000)]
patch-7.144
VarGetPattern()
Removed unreachable code, VAR_NOSUBST is never set.
Max Okumoto [Sat, 19 Mar 2005 00:15:44 +0000 (00:15 +0000)]
patch-7.143
VarGetPattern()
- Update vp->ptr instead of using temp variable.
- Renamed variable pattern to patt.
Sascha Wildner [Fri, 18 Mar 2005 23:20:34 +0000 (23:20 +0000)]
Forced commit to supply a proper message for the previous change.
1) Clean up:
* Traditional function declarations -> prototype form
* Remove unnecessary void casts
* Use '\0' where '\0' is meant
* Add some extra spaces
* include <stdlib.h> for exit()
* Add $DragonFly$ to Makefile
2) Raise WARNS to 6
Max Okumoto [Fri, 18 Mar 2005 22:12:47 +0000 (22:12 +0000)]
FreeBSD-Date: 2005/03/18 15:23:49
FreeBSD-Date: 2005/03/18 15:25:23
Author: harti
o Replace a bogus use of Lst_Find with explicite LST_FOREACH calls
and remove the unneeded second argument to ReadMakefile.
o Replace Lst_Find calls with LST_FOREACH loops. This helps in
constification and simplifies the code because the one-liner
predicates can be inlined into the code.
Liam J. Foy [Fri, 18 Mar 2005 22:08:08 +0000 (22:08 +0000)]
- Correct usage of gethostname(3)
- strlcpy
Max Okumoto [Fri, 18 Mar 2005 18:39:07 +0000 (18:39 +0000)]
Fix dangling ref to auto variable.
Matthew Dillon [Fri, 18 Mar 2005 18:29:05 +0000 (18:29 +0000)]
Add a sysctl "net.inet.tcp.aggregate_acks" which controls the tcp ack
aggregation feature, default to on. This feature was added a while
back and most systems will want to leave it on. The feature only effects
situations where TCP packets are aggregated by e.g. a GigE interface or a
router talking to a GigE interface and allows the tcp stack to send just
one ack after receiving a large burst (~8-10) of tcp packets rather then
sending four or five back-to-back acks. The single ack actually gets the
sequence space updated more quickly on the other end as well as greatly
reduces the PPS rate and bandwidth used by the return channel. Protocol
stacks on both ends are also made more efficient due to the fewer acks that
need to be sent and received.
The feature has one side-effect which warrents having the sysctl. The
slow-start exponential ramp-up takes longer to ramp-up due to the lower
ack rate, reducing tcp bandwidth at the beginning of the connection (but
not once it gets going). This ONLY effects links which huge bandwidth-delay
products (like 100ms x 100MBits), and then only for a short time, and
even in such cases the difference is not usually noticeable except when
explicitly tested for. LANs and most WAN links are NOT effected. The
positive aspects greatly outweight the negative ones in most cases and
sysops will almost always want to leave the feature on.
Delayed-Ack-Analysis-by: Noritoshi Demizu <demizu@dd.iij4u.or.jp>
Max Okumoto [Fri, 18 Mar 2005 18:21:10 +0000 (18:21 +0000)]
Patch originally from Matt, then modified by harti.
FreeBSD-Date: 2005/03/18 15:16:09
Author: matt, harti
Fix a bug in matching suffixes. Under certain circumstances the code
would access memory before the beginning of the string to match (the
suffix match starts at the end of both the string and the suffix and
proceedes to the begin until either the start of the suffix is hit
or the character does not match). This could lead to a memcpy copying
into random memory. Fix this by checking the length of the string to
match too and replacing the Lst_Find calls with LST_FOREACH loops
(last part by harti).
Sascha Wildner [Fri, 18 Mar 2005 17:42:46 +0000 (17:42 +0000)]
../../../ppt.msg
Chris Pressey [Fri, 18 Mar 2005 01:57:58 +0000 (01:57 +0000)]
Raise WARNS to 6:
- Constify some char *'s.
- Mark parameters as unused in callback function.
Joerg Sonnenberger [Fri, 18 Mar 2005 00:57:10 +0000 (00:57 +0000)]
WARNS=6.
Partly-submitted-by: Samuel J. Greear <sjg@evilcode.net>
Joerg Sonnenberger [Fri, 18 Mar 2005 00:52:49 +0000 (00:52 +0000)]
WARNS=6.
Submitted-by: Samuel J. Greear <sjg@evilcode.net>
Matthew Dillon [Thu, 17 Mar 2005 22:25:46 +0000 (22:25 +0000)]
Add RCNG support for setting the negative attribute cache timeout, make
the boot-time display of nfs client parameters more readable, and properly
document the nfs_access_cache and nfs_neg_cache RCNG configuration variables.
Increase the default NFS attribute cache timeout from 2 to 5 seconds.
Matthew Dillon [Thu, 17 Mar 2005 21:41:16 +0000 (21:41 +0000)]
Ignore additional stty control characters like ^Y which interfere with
the editor's operation.
Reported-by: Jasse Jansson <jasse@hornet.ac>
David Rhodus [Thu, 17 Mar 2005 20:59:34 +0000 (20:59 +0000)]
Testing a new commits monitoring script.
David Rhodus [Thu, 17 Mar 2005 20:53:55 +0000 (20:53 +0000)]
Unbreak the pkg_add -r which the last commit broke.
Matthew Dillon [Thu, 17 Mar 2005 17:28:46 +0000 (17:28 +0000)]
Clean up a number of caching edge cases in NFS, rework the code to be
a bit more readable, document some bits, and fix some cache coherency
detection issues. The caching cleanups should allow the NFS client to
retain more of the NFS cache when doing complex operations on a file.
* Properly check and update the mtime using WCC records in the NFS response.
This record gives us the 'before' and 'after' mtime. The 'before' mtime
must match our existing idea of the mtime, if it doesn't we flag the
nfsnode as having been modified by the server. Our notion of the mtime
is then set to the 'after time. This was not being done properly for
several edge cases.
This required extending the nfsm macros a bit in order to be able to tell
loadattrcache how to handle the mtime data. This also required rearranging
(really fixing) the sequence in nfs_open(), nfs_write(), etc.
* Rearrange the flags a bit. NSIZECHANGED -> NRMODIFIED (nfsnode modified
by server), NMODIFIED -> NLMODIFIED (nfsnode modified by client). Do
not clear NRMODIFIED until we have actually invalidated the cache (this
fixes a problem where programs using mmap() were not properly clearing
the cache after a file was modified on the server).
* Don't code NRMODIFIED as an exception to NLMODIFIED. Recode the flags so
they (mostly) operate in tandem.
* When appending to a file, use nfs_flush() instead of nfs_vinvalbuf().
There is no need to destroy our data cache for the file. This makes
appends considerably more efficient.
* Hopefully fix the last problem associated with attribute timeouts.
* Clear the attribute cache when a file is opened for write in nfs_open()
BEFORE doing other checks rather then after.
* Document some of the nastier cache coherency hacks.
Joerg Sonnenberger [Thu, 17 Mar 2005 15:39:34 +0000 (15:39 +0000)]
Fix warnings. Raise WARNS to 6.
Submitted-by: Samuel J. Greear <sjg@evilcode.net>
Add some explicit XXX comments about the signal handlers,
they have to be rewritten.
Joerg Sonnenberger [Thu, 17 Mar 2005 15:09:01 +0000 (15:09 +0000)]
Use a common source for the string to integer conversion. Add
strtoimax and strtoumax.
Inspired-by: wide char versions.
Sascha Wildner [Thu, 17 Mar 2005 08:22:38 +0000 (08:22 +0000)]
Now that make no longer needs it, remove the ispc98 sysctl.
YONETANI Tomokazu [Thu, 17 Mar 2005 07:11:15 +0000 (07:11 +0000)]
Now that const'ification of users of dktypenames also const'ified
most of users of fstypenames, finish the last bit of const'ification.
Max Okumoto [Thu, 17 Mar 2005 02:34:31 +0000 (02:34 +0000)]
Remove pc98 support as a gift to Sascha Wildner :-)
Matthew Dillon [Thu, 17 Mar 2005 02:17:54 +0000 (02:17 +0000)]
Close a possible security hole by using strvis() when displaying td_comm.
Matthew Dillon [Thu, 17 Mar 2005 02:01:12 +0000 (02:01 +0000)]
Fix a potential security hole by outputing non-printable characters related
to file names or program commands as '?'.
Max Okumoto [Wed, 16 Mar 2005 22:50:06 +0000 (22:50 +0000)]
patch-7.142
modifier_C()
Update vp->ptr instead of using temp variable.
Max Okumoto [Wed, 16 Mar 2005 22:49:51 +0000 (22:49 +0000)]
patch-7.141
modifier_S()
Update vp->ptr instead of using temp variable.
Max Okumoto [Wed, 16 Mar 2005 22:49:01 +0000 (22:49 +0000)]
patch-7.140
- Removed comment about assuming that Var_Parse() is always
passed '$' as first char. Sometimes is is the last character
of a conditional (e.g., defined(DragonFly), etc.)
- Clean up some comments.
- Moved common code out of if-statement.
Max Okumoto [Wed, 16 Mar 2005 22:48:47 +0000 (22:48 +0000)]
patch-7.139
Split out sysVvarsub() from ParseModifier()
Max Okumoto [Wed, 16 Mar 2005 22:48:31 +0000 (22:48 +0000)]
patch-7.138
- renamed freePtr to freeResult.
Max Okumoto [Wed, 16 Mar 2005 22:48:15 +0000 (22:48 +0000)]
patch-7.137
- Cleanup comments
- Moved code from bottom of ParseModifier() in to calling function.
Max Okumoto [Wed, 16 Mar 2005 22:47:47 +0000 (22:47 +0000)]
patch-7.136
Clean up handling of VAR_JUNK.
Matthew Dillon [Wed, 16 Mar 2005 22:17:59 +0000 (22:17 +0000)]
Fix a bugaboo in the last commit. We cannot safely modify n_mtime based
on the server indicating that it has changed without also flushing the
vnode's buffers. Otherwise we end up with stale data in the buffer cache
and the client unaware that it is stale. Since we do not want to flush
the buffers in the getattr code we instead simply mark the nfs node
NSIZECHANGED which will flush the buffers on the next read.
Chris Pressey [Wed, 16 Mar 2005 21:26:01 +0000 (21:26 +0000)]
Raise WARNS to 6.
Chris Pressey [Wed, 16 Mar 2005 21:21:34 +0000 (21:21 +0000)]
Raise WARNS to 6:
- Rename local variables to avoid conflict with globals.
Max Okumoto [Wed, 16 Mar 2005 20:03:00 +0000 (20:03 +0000)]
FreeBSD-Date: 2005/03/16 08:04:45
FreeBSD-Date: 2005/03/16 08:06:28
FreeBSD-Date: 2005/03/16 12:22:15
FreeBSD-Date: 2005/03/16 16:02:09
FreeBSD-Date: 2005/03/16 16:11:11
Author: harti
o Get rid of another bunch of Lst_ForEach in favour of LST_FOREACH
and simplify code accordingly.
o Fix a comment.
o Replace another bunch of Lst_ForEachs by LST_FOREACHs and simplify
code.
o Replace Lst_ForEach by LST_FOREACH.
o Now that there are no users of Lst_ForEach and Lst_ForEachFrom are
left delete these two macros and all the associated stuff.
YONETANI Tomokazu [Wed, 16 Mar 2005 17:54:59 +0000 (17:54 +0000)]
Constify all users of dktypenames to unbreak buildworld.
Reported-by: walt
David Xu [Wed, 16 Mar 2005 14:30:31 +0000 (14:30 +0000)]
Add types prgregset_t and psaddr_t which will be used by
thread debugging code.
Chris Pressey [Wed, 16 Mar 2005 08:02:05 +0000 (08:02 +0000)]
Style(9):
- Spell NULL as NULL, and '\0' as '\0'.
- Explicitly check non-booleans against 0, '\0', or NULL.
Chris Pressey [Wed, 16 Mar 2005 07:30:33 +0000 (07:30 +0000)]
Style(9):
- Braces around multiline blocks.
- Spell NULL as NULL, not 0.
- Don't treat pointers and ints as booleans; use explicit tests.
- Minor whitespace adjustments.
Joerg Sonnenberger [Wed, 16 Mar 2005 07:22:07 +0000 (07:22 +0000)]
Remove redundant declarations. Convert to ANSI C function declarations.
Convert dates to ISO style.
Chris Pressey [Wed, 16 Mar 2005 07:16:14 +0000 (07:16 +0000)]
Style(9):
- Braces around multiline blocks.
- Space after 'switch'.
- 'errno' is not a boolean, use explicit test.
- Add explanatory comment for an issue which has caused confusion at
least once in the past.
Chris Pressey [Wed, 16 Mar 2005 07:08:04 +0000 (07:08 +0000)]
Style(9):
- Braces around multiline block.
- Space after 'switch'.
- stat(2), lstat(2), link(2), unlink(2) and symlink(2) don't return
boolean values; use explicit checks against 0.
Joerg Sonnenberger [Wed, 16 Mar 2005 06:56:20 +0000 (06:56 +0000)]
Add some more path definitions as used by the Citrus framework.
Joerg Sonnenberger [Wed, 16 Mar 2005 06:54:41 +0000 (06:54 +0000)]
Add Citrus files not conflicting with the current rune implementation.
Obtained-from: NetBSD / Citrus Project
Joerg Sonnenberger [Wed, 16 Mar 2005 06:42:34 +0000 (06:42 +0000)]
More Citrus code.
Obtained-from: NetBSD
Joerg Sonnenberger [Wed, 16 Mar 2005 06:38:26 +0000 (06:38 +0000)]
More stuff for Citrus.
Chris Pressey [Wed, 16 Mar 2005 06:35:48 +0000 (06:35 +0000)]
We have _DIAGASSERT now; might as well enable it here.
Chris Pressey [Wed, 16 Mar 2005 06:33:11 +0000 (06:33 +0000)]
Followup to previous commit: use uint8_t instead of u_char, for a real
guarantee of byte-ness.
Pointed-out-by: joerg
Joerg Sonnenberger [Wed, 16 Mar 2005 06:29:03 +0000 (06:29 +0000)]
Make dktypnames const.
Joerg Sonnenberger [Wed, 16 Mar 2005 06:27:17 +0000 (06:27 +0000)]
s/index/idx. Add an explicit default case to teach GCC that idx is
used correctly.
Chris Pressey [Wed, 16 Mar 2005 06:26:49 +0000 (06:26 +0000)]
De-spaghettify:
- fail() consists of two statements and is called only once; inline it.
- The point of finish() is to reap the child, so call it reap().
- Make the child's pid local to main(), and pass it to reap().
- done() is always called after reap(), so calling done() from inside
reap() is not necessary; let main() handle it instead.
Joerg Sonnenberger [Wed, 16 Mar 2005 06:24:36 +0000 (06:24 +0000)]
Fix some warnings.
Joerg Sonnenberger [Wed, 16 Mar 2005 06:19:09 +0000 (06:19 +0000)]
idx is always used, shut GCC up by initialising it to 0.
Chris Pressey [Wed, 16 Mar 2005 06:18:56 +0000 (06:18 +0000)]
Raise WARNS to 6.
Joerg Sonnenberger [Wed, 16 Mar 2005 06:13:24 +0000 (06:13 +0000)]
Fix warnings.
Chris Pressey [Wed, 16 Mar 2005 06:08:06 +0000 (06:08 +0000)]
When dumping the contents of a struct sockaddr with an unknown address
family, dump individual bytes (u_chars) instead of u_shorts. This
avoids endianness issues while also simplifying the code.
In-discussion-with: joerg
Joerg Sonnenberger [Wed, 16 Mar 2005 05:27:27 +0000 (05:27 +0000)]
WARNS=6.
Submitted-by: Samuel J. Greear <sjg@evilcode.net>
Joerg Sonnenberger [Wed, 16 Mar 2005 05:19:11 +0000 (05:19 +0000)]
Raise WARNS to 6:
- correctly initialize globals completely
- don't shadow globals
- change some locals to match the type in the ng headers
Submitted-by: Samuel J. Greear <sjg@evilcode.net>
Chris Pressey [Wed, 16 Mar 2005 04:47:00 +0000 (04:47 +0000)]
When rendering the routename() or netname() of a struct sockaddr
with an unknown address family, use a more straightforward loop
for dumping the contents of sa_data as hex:
- start at sa_data, end at sa + sa_len (NOT sa_data + sa_len);
- don't make assumptions about the size of u_short.
Inspired-by-patch-by: Sepherosa Ziehau <sepherosa_at_softhome_dot_net>
Joerg Sonnenberger [Wed, 16 Mar 2005 04:45:07 +0000 (04:45 +0000)]
Pass const ** as second argument to the command functions,
explicitly cast this away in calls to getopt. Change some
local variables to const char * as well, to match this.
Chris Pressey [Wed, 16 Mar 2005 04:18:35 +0000 (04:18 +0000)]
Raise WARNS to 6:
- Rename local variables to avoid conflicts with globals;
- Make local functions static;
- Add missing function prototypes;
- Remove stale function prototypes and struct declarations;
- Un-nest nested extern declarations;
- Constify some char *'s where appropriate;
- Resolve comparisons between signed and unsigned;
- Copy some strings into work buffers instead of returning
a const char * from a function defined to return a char *.
Submitted-by: Sepherosa Ziehau <sepherosa_at_softhome_dot_net>
Some-tweaks-by: cpressey
Max Okumoto [Wed, 16 Mar 2005 00:15:21 +0000 (00:15 +0000)]
patch-7.135
Don't pass endc since we can calucluate it in the function that
uses it.
Max Okumoto [Wed, 16 Mar 2005 00:15:04 +0000 (00:15 +0000)]
patch-7.134
Cleanup calculation of consumed.
Max Okumoto [Wed, 16 Mar 2005 00:14:43 +0000 (00:14 +0000)]
patch-7.133
ParseModifier()
- substitute in delim
- Use shorter variable name patt instead of pattern.
- Move calc of consumed into subroutine.