Bug 11259 - New entries for text/html magic
Summary: New entries for text/html magic
Status: RESOLVED FIXED
Alias: None
Product: shared-mime-info
Classification: Unclassified
Component: freedesktop.org.xml (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: medium normal
Assignee: Jonathan Blandford
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-13 14:39 UTC by David Faure
Modified: 2007-06-13 15:06 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
better patch (1.27 KB, patch)
2007-06-13 14:44 UTC, David Faure
Details | Splinter Review

Description David Faure 2007-06-13 14:39:42 UTC
"<script>foo</script>" needs to be recognized as text/html  (KDE bug 61483).

Here's a patch to add that. It removes the entries for TITLE and title because there is already an entry a few lines above, which has 0:64 as offset, so those lines with only "0" as offset are redundant.

diff -u -p -r1.246 freedesktop.org.xml.in
--- freedesktop.org.xml.in      1 Jun 2007 15:27:01 -0000       1.246
+++ freedesktop.org.xml.in      13 Jun 2007 21:33:25 -0000
@@ -3378,8 +3378,8 @@ command to generate the output files.
       <match type="string" value="&lt;HTML" offset="0:64"/>
       <match type="string" value="&lt;BODY" offset="0"/>
       <match type="string" value="&lt;body" offset="0"/>
-      <match type="string" value="&lt;TITLE" offset="0"/>
-      <match type="string" value="&lt;title" offset="0"/>
+      <match type="string" value="&lt;SCRIPT" offset="0"/>
+      <match type="string" value="&lt;script" offset="0"/>
       <match type="string" value="&lt;!--" offset="0"/>
       <match type="string" value="&lt;h1" offset="0"/>
       <match type="string" value="&lt;H1" offset="0"/>
Comment 1 David Faure 2007-06-13 14:44:37 UTC
Created attachment 10301 [details] [review]
better patch

Doh, I meant to use a range instead. Let's also put uppercase first lowercase second in all those range entries for good order.
Comment 2 Christian - Manny Calavera - Neumair 2007-06-13 15:06:10 UTC
Committed, thanks. Closing.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.