[PATCH] remove libz/libbz2 entry from contributing page

2013-09-13 Thread rekado
According to [1], the preferred way is to remove support for unzipping from the exec translator and implement this feature through other translators. [1]: http://lists.gnu.org/archive/html/bug-hurd/2013-08/msg00049.html --- contributing.mdwn | 1 - 1 file changed, 1 deletion(-) diff --git a/cont

Re: [PATCH 3/7] name_equal: return early from function if src null-terminates in the loop

2013-09-13 Thread Marin Ramesa
On 13.09.2013 15:41:47, Neal H. Walfield wrote: > At Fri, 13 Sep 2013 15:06:33 +0200, > Marin Ramesa wrote: > > > > On 13.09.2013 14:42:44, Neal H. Walfield wrote: > > > At Fri, 13 Sep 2013 13:31:53 +0200, > > > Marin Ramesa wrote: > > > > diff --git a/device/dev_name.c b/device/dev_name.c > > > >

Re: [PATCH 3/7] name_equal: return early from function if src null-terminates in the loop

2013-09-13 Thread Neal H. Walfield
At Fri, 13 Sep 2013 15:06:33 +0200, Marin Ramesa wrote: > > On 13.09.2013 14:42:44, Neal H. Walfield wrote: > > At Fri, 13 Sep 2013 13:31:53 +0200, > > Marin Ramesa wrote: > > > diff --git a/device/dev_name.c b/device/dev_name.c > > > index bf541df..6ce4b19 100644 > > > --- a/device/dev_name.c > >

Re: [PATCH 3/7] name_equal: return early from function if src null-terminates in the loop

2013-09-13 Thread Marin Ramesa
On 13.09.2013 14:42:44, Neal H. Walfield wrote: > At Fri, 13 Sep 2013 13:31:53 +0200, > Marin Ramesa wrote: > > diff --git a/device/dev_name.c b/device/dev_name.c > > index bf541df..6ce4b19 100644 > > --- a/device/dev_name.c > > +++ b/device/dev_name.c > > @@ -69,9 +69,12 @@ name_equal(src, len, ta

Re: [PATCH 3/7] name_equal: return early from function if src null-terminates in the loop

2013-09-13 Thread Neal H. Walfield
At Fri, 13 Sep 2013 13:31:53 +0200, Marin Ramesa wrote: > diff --git a/device/dev_name.c b/device/dev_name.c > index bf541df..6ce4b19 100644 > --- a/device/dev_name.c > +++ b/device/dev_name.c > @@ -69,9 +69,12 @@ name_equal(src, len, target) > int len; > char*target; > { > -

[PATCH 6/7] use boolean_t instead of an int

2013-09-13 Thread Marin Ramesa
Variable 'found' already receives values TRUE and FALSE, so why not make it a boolean. * device/dev_name.c: Use boolean_t instead of an int. --- device/dev_name.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/device/dev_name.c b/device/dev_name.c index a9056ff..dc5f835 1006

[PATCH 2/7] small style changes for consistency

2013-09-13 Thread Marin Ramesa
* device/dev_name: Changes in coding style. --- device/dev_name.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/device/dev_name.c b/device/dev_name.c index 98a0e27..bf541df 100644 --- a/device/dev_name.c +++ b/device/dev_name.c @@ -65,9 +65,9 @@ nomap() */

[PATCH 4/7] another small change in style for consistency

2013-09-13 Thread Marin Ramesa
* device/dev_name.c: Change in coding style. --- device/dev_name.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/device/dev_name.c b/device/dev_name.c index 6ce4b19..7bdbf38 100644 --- a/device/dev_name.c +++ b/device/dev_name.c @@ -170,7 +170,7 @@ boolean_t dev_name_loo

[PATCH 3/7] name_equal: return early from function if src null-terminates in the loop

2013-09-13 Thread Marin Ramesa
Return early from function if src null-terminates in the loop before the target does. This is to prevent pointers going to addresses beyond null-termination. * device/dev_name.c (name_equal): Check if src null-terminates in the loop. --- device/dev_name.c | 5 - 1 file changed, 4 insertions(

Small cleanup of device name routines

2013-09-13 Thread Marin Ramesa
What follows is a small cleanup of device name routines. Patches 2, 4 and 7 can be applied at the same time. They are just changes in style for consistency. I didn't know how to combine them in one commit, so I'm sending them this way, I hope this is not a problem. I'm not sure about patch 3 - I a

[PATCH 7/7] another small change in style for consistency

2013-09-13 Thread Marin Ramesa
* device/dev_name.c: Change in coding style. --- device/dev_name.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/device/dev_name.c b/device/dev_name.c index dc5f835..2e3b0ab 100644 --- a/device/dev_name.c +++ b/device/dev_name.c @@ -219,14 +219,14 @@ boolean_t dev_change

[PATCH 5/7] remove preprocessor comments

2013-09-13 Thread Marin Ramesa
The first one is a message that name lookup has been called, which I think is safe to remove, or maybe add #if DEBUG. Second one is a alternate calculation that I doubt it will ever be used, so I think it's safe to remove it. * device/dev_name.c: Remove preprocessor comments. --- device/dev_name

[PATCH 1/7] remove register qualifiers

2013-09-13 Thread Marin Ramesa
* device/dev_name.c: Remove register qualifiers. --- device/dev_name.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/device/dev_name.c b/device/dev_name.c index cef1efd..98a0e27 100644 --- a/device/dev_name.c +++ b/device/dev_name.c @@ -65,9 +65,9 @@ nomap

Re: glib2.0 test gio/tests/socket fails, BUG?

2013-09-13 Thread Justus Winter
Hi, Quoting Svante Signell (2013-09-13 00:01:20) > Another puzzling thing is how to find out which version is used for > which situation? For *_io_restrict_auth() there are plenty to choose > from: > > grep -r _io_restrict_auth ../../hurd/hurd-20130727/ > pfinet/io-ops.c:S_io_restrict_auth > pflo