Bug #240
closedPatch line limit removal
0%
Description
Hi all,
http://leaf.dragonflybsd.org/~joerg/patch.diff
contains the last missing piece to remove unnecessary line limits in
patch(1). Please test or review, I'd like to have this in the tree soon
:-)
Joerg
Updated by dillon over 18 years ago
:Hi all,
:http://leaf.dragonflybsd.org/~joerg/patch.diff
:contains the last missing piece to remove unnecessary line limits in
:patch(1). Please test or review, I'd like to have this in the tree soon
::-)
:
:Joerg
This is kinda messy:
#define __STDC_LIMIT_MACROS
Is there any way to not use those extensions ?
-Matt
Matthew Dillon
<dillon@backplane.com>
Updated by dillon over 18 years ago
:
:Hi all,
:http://leaf.dragonflybsd.org/~joerg/patch.diff
:contains the last missing piece to remove unnecessary line limits in
:patch(1). Please test or review, I'd like to have this in the tree soon
::-)
:
:Joerg
Also, full_fgets() can be cleaned up a bit. Just add a local
resize_len variable instead of copying 'buf_len * 2' all over
the place, and also clean up the last bit to read something like
this:
ret = fgets(resize_buf + buf_len, resize_len - buf_len, fp);
buf = resize_buf;
buf_len = resize_len;
-Matt
Matthew Dillon
<dillon@backplane.com>
Updated by joerg over 18 years ago
On Fri, Jul 14, 2006 at 10:29:06AM -0700, Matthew Dillon wrote:
:Hi all,
:http://leaf.dragonflybsd.org/~joerg/patch.diff
:contains the last missing piece to remove unnecessary line limits in
:patch(1). Please test or review, I'd like to have this in the tree soon
::-)
:
:JoergThis is kinda messy:
#define __STDC_LIMIT_MACROS
Is there any way to not use those extensions ?
This is a standard macro and part of C99.
Joerg
Updated by dillon over 18 years ago
:On Fri, Jul 14, 2006 at 10:29:06AM 0700, Matthew Dillon wrote:)
:>
:> :Hi all,
:> :http://leaf.dragonflybsd.org/~joerg/patch.diff
:> :contains the last missing piece to remove unnecessary line limits in
:> :patch(1). Please test or review, I'd like to have this in the tree soon
:> ::
:> :
:> :Joerg
:>
:> This is kinda messy:
:>
:> #define __STDC_LIMIT_MACROS
:>
:> Is there any way to not use those extensions ?
:
:This is a standard macro and part of C99.
:
:Joerg
I really dislike having to pre-define anything prior to doing
an #include. There are a few places where we have no choice,
but if something is going to be 'standard C' then I just do not
accept that it must require a pre-defined macro to operate properly.
I suppose if you really, really want to use the macro you can make
it a -D option in the compile line. That is, fix the Makefile rather
then put the #define in the source code.
-Matt
Matthew Dillon
<dillon@backplane.com>
Updated by tuxillo over 15 years ago
The patch on leaf seems to be no longer available. Also there was a patch(1)
sync with OpenBSD after the creation date of the submitted patch.
How can I check if the line limit is still there?
Updated by corecode over 15 years ago
I think this can be closed. New patch uses mmap, AFAIR
Updated by ahuete.devel over 15 years ago
plan_a() function uses indeed mmap() and comments state "/* Try
keeping everything in memory. */"
Is that what you meant?
2009/8/26 Simon 'corecode' Schubert (via DragonFly issue tracker)
<sinknull@leaf.dragonflybsd.org>:
Simon 'corecode' Schubert <corecode@fs.ei.tum.de> added the comment:
I think this can be closed. New patch uses mmap, AFAIR
_____________________________________________
DragonFly issue tracker <bugs@lists.dragonflybsd.org>
<http://bugs.dragonflybsd.org/issue240>
_____________________________________________