On Tue, Apr 14, 2015 at 04:39:44PM +0100, Dimitris Papastamos wrote:
> On Tue, Apr 14, 2015 at 04:33:56PM +0100, Connor Lane Smith wrote:
> > On 14 April 2015 at 13:50, Dimitris Papastamos wrote:
> > > dst and src are required to be valid objects even if n is 0, otherwise
> > > this is undefined b
On Fri, Apr 10, 2015 at 05:44:45PM +0200, Roberto E. Vargas Caballero wrote:
> Applied with small style change. Thanks.
>
>
> Regards,
>
>
Ah, I was going by the surrounding style in the function and didn't
realize that it was wrong ;) Anyways, thanks!
--
Omar
Cool. My concern is about adding "unstable" code on a "stable" distribution.
The disk cache is disabled by default, so it is probably not an issue. If
someone enables it, it is their decision.
I am not the maintainer, so it isn't my call, but if the maintainer wants my
input as the original su
Quoth tauto...@gmail.com:
> On those platforms that need that define set before the include, does the -D
> argument actually work?
Yep, the cache seems to work fine.
On those platforms that need that define set before the include, does the -D
argument actually work?
Ben
Original Message
From: Nick
Sent: Tuesday, April 14, 2015 2:21 AM
To: dev mail list
Reply To: dev mail list
Subject: Re: [dev] [surf] [PATCH] Add NODISKCACHE option to disable the disk
ca
On Tue, Apr 14, 2015 at 04:33:56PM +0100, Connor Lane Smith wrote:
> On 14 April 2015 at 13:50, Dimitris Papastamos wrote:
> > dst and src are required to be valid objects even if n is 0, otherwise
> > this is undefined behaviour.
>
> I looked this up in C11. Seems to be the case:
>
> > 7.24.2.1
Well, that sucks. Thank you for bringing that to my attention.
I think an attachment is better in this case...
On Tue, Apr 14, 2015 at 5:29 PM, Eric Pruitt wrote:
> On Tue, Apr 14, 2015 at 05:25:15PM +0200, Willem van de Krol wrote:
>> - Text *data = text_load(filename && access(filename, F_
On 14 April 2015 at 13:50, Dimitris Papastamos wrote:
> dst and src are required to be valid objects even if n is 0, otherwise
> this is undefined behaviour.
I looked this up in C11. Seems to be the case:
> 7.24.2.1.2. The memcpy function copies n characters from the object
> pointed to by s2 in
On Tue, Apr 14, 2015 at 05:25:15PM +0200, Willem van de Krol wrote:
> - Text *data = text_load(filename && access(filename, F_OK) == 0 ?
> filename : NULL);
Your MUA munged the patch by wrapping a line.
Eric
Permission errors are handled in text_load(). To detect these errors,
and other open() errors (opening a directory, for example), check the
return value of text_load().
This fixes a segmentation fault when opening a directory.
Also, opening a file you are not permitted to read, will now give an
er
On Tue, Apr 14, 2015 at 01:43:56PM +0200, Silvan Jegen wrote:
> On Tue, Apr 14, 2015 at 12:55 PM, Gregor Best wrote:
> > The cause seems to be that for bot `term.line` and `term.alt` are NULL
> > at this point. While this does mean that even with a `len` parameter of
> > 0, the `dst` pointer gets
On Tue, Apr 14, 2015 at 12:55 PM, Gregor Best wrote:
> The cause seems to be that for bot `term.line` and `term.alt` are NULL
> at this point. While this does mean that even with a `len` parameter of
> 0, the `dst` pointer gets touched, I don't think it's ever right to call
> either `memcpy` or `m
This commit seems to cause st to segfault on start on OpenBSD
(-current). A similar segfault has been observed in #suckless by user
mvdan, using GNU libc.
The cause seems to be that for bot `term.line` and `term.alt` are NULL
at this point. While this does mean that even with a `len` parameter of
Not always is desirable to create a pseudo terminal, and some times
we want to open a terminal emulator over a tty line. With this new
patch is possible to do someting like:
$ st -l /dev/ttyS0 115200
Without this option was needed to launch another terminal emulator
over st (for example m
>> It compiles fine with that patch on Wheezy, too.
>
> Not even all of Roberto's patch seems necessary. The attached patch
> works for me.
Maybe we should add these lines to the base version, at least
until the cache code becomes more stable.
Regards,
Quoth Nick:
> Quoth Roberto E. Vargas Caballero:
> > I compiled it in jessie with this patch:
>
> It compiles fine with that patch on Wheezy, too.
Not even all of Roberto's patch seems necessary. The attached patch
works for me.
diff --git a/surf.c b/surf.c
index 87c10ef..0c2d580 100644
--- a/s
Quoth Roberto E. Vargas Caballero:
> > It looks like you can use SOUP_CHECK_VERSION(2, 34, 0) to test for the
> > version that supports disk cache. May you please try your patch with that
> > code, and send me an updated patch? I will then test on my own system with
> > support, and verify th
17 matches
Mail list logo