[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gnu-arch-users] [BUG] mini patch for pfs-dav.c and libneon GNU TLS

From: Aleix Conchillo Flaque
Subject: Re: [Gnu-arch-users] [BUG] mini patch for pfs-dav.c and libneon GNU TLS work
Date: Sun, 26 Sep 2004 09:45:03 +0200
User-agent: Gnus/5.1006 (Gnus v5.10.6) Emacs/21.3 (gnu/linux)

John Meinel <address@hidden> writes:

> Meaning, default to 80, unless we are sure this should be 443. The way
> the patch was submitted states "default to 443" unless we know we
> should be on 80.
> I don't know for sure. But I would say, given any port to pick as
> default, go with 80.
> I don't know what strings would exist in dav_scheme, though, to really
> recommend a specific fix.

As I see it, I treat it like a switch case, where the "default" is 80.

Anyway, I think it doesn't matter, because only http and https strings are
allowed, so there are just two cases. See this function in pfs-dav.c:

arch_pfs_dav_supported_protocol (t_uchar * uri)
  if (!str_cmp_prefix ("http:", uri) ||
      !str_cmp_prefix ("https:", uri))
    return 1;
    return 0;

Eventhough, the patch is so small that the merger can fix it in some
other way. Right now, I'm changing it so it doesn't have to compare
the strings again, it's already done at the beginning of the
arch_pfs_dav_connect function (see next message).

Best regards,


Attachment: pgptwxVp_r721.pgp
Description: PGP signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]