TextSearch won't grep XHTML files

[humdinger]

This is hrev49934.

XHTML has unfortunately a MIME type of application/xhtml+xml. You'd have to uncheck TextSearch's setting "Text files only" (which is what you usually want) to make it work.

TextSearch examines files, if they have a "text" supertype for this.
Would introducing this exception in code be acceptable:

// Make an exception for XHTML files
if (strcmp("application/xhtml+xml", mimeTypeString) == 0)
	return true;

(inserted here: ​http://xref.plausible.coop/source/xref/haiku/src/apps/text_search/FileIterator.cpp#61)

[waddlesplash]

deassigning various things from me

[humdinger]

So... is the above solution OK for now?
I know an even better idea would be to check the "supported types" of TextSearch. OTOH, there probably aren't that many more text files that aren't of the MIME text/* in the future. Plus, as TextSearch comes from a package and is therefore read-only, the user can't add more supported types (which is something that might be addressed after R1, I guess).

[pulkomandy]

Yes, that sounds like the right way to go.

[humdinger]

Applied in hrev50568.