You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Vincent Lefevre <vi...@vinc17.org> on 2004/06/18 23:05:56 UTC
text MIME types (was: Is XML text?)
On 2004-06-13 16:25:46 -0700, Ben Reser wrote:
> I think the logical thing to do is make this a configuration option,
> allowing users to provide patterns that are "text" MIME-Types. I'd
> say we leave the default the same as what we have now (no
> configuration variable). I think our current default is good for
> most users, because this issue doesn't come up much.
I've just seen that message/* should probably be regarded as text by
default.
--
Vincent Lefèvre <vi...@vinc17.org> - Web: <http://www.vinc17.org/>
100% validated (X)HTML - Acorn / RISC OS / ARM, free software, YP17,
Championnat International des Jeux Mathématiques et Logiques, etc.
Work: CR INRIA - computer arithmetic / SPACES project at LORIA
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: [PATCH] messages treated non binary (was Re: text MIME types)
Posted by Benjamin Pflugmann <be...@pflugmann.de>.
On Sun 2004-06-20 at 17:36:03 +0200, Pierre THIERRY wrote:
> > I've just seen that message/* should probably be regarded as text by
> > default.
>
> Index: subversion/libsvn_subr/validate.c
> ===================================================================
> --- subversion/libsvn_subr/validate.c (revision 10034)
> +++ subversion/libsvn_subr/validate.c (working copy)
> @@ -68,5 +68,6 @@
> return ((strncmp (mime_type, "text/", 5) != 0)
> && (strcmp (mime_type, "image/x-xbitmap") != 0)
> && (strcmp (mime_type, "image/x-xpixmap") != 0)
> + && (strcmp (mime_type, "message/") != 0)
> );
> }
Without commenting on whether that change would be a good idea, I
think the patch is broken. Shouldn't it be
+ && (strncmp (mime_type, "message/", 8) != 0)
Bye,
Benjamin.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
[PATCH] messages treated non binary (was Re: text MIME types)
Posted by Pierre THIERRY <pi...@moine-fou.org>.
> I've just seen that message/* should probably be regarded as text by
> default.
Index: subversion/libsvn_subr/validate.c
===================================================================
--- subversion/libsvn_subr/validate.c (revision 10034)
+++ subversion/libsvn_subr/validate.c (working copy)
@@ -68,5 +68,6 @@
return ((strncmp (mime_type, "text/", 5) != 0)
&& (strcmp (mime_type, "image/x-xbitmap") != 0)
&& (strcmp (mime_type, "image/x-xpixmap") != 0)
+ && (strcmp (mime_type, "message/") != 0)
);
}
Quickly,
Nowhere man
--
pierre.thierry@moine-fou.org
OpenPGP 0xD9D50D8A