improve handling of compressed content in mc.ext

[ossi]

this may be controversial - there is a related thread at ​https://mail.gnome.org/archives/mc/2005-June/thread.html#00022, though one would expect it to be outdated by now. i've been using this code for a very long time.

[andrew_b]

Branch: 4128_mc.ext_compressed_content
changeset:8857423e4ebb770b6f0ea3103abf5d35c85fcbe8

[andrew_b]

Thanks!

Merged to master: [eeb14d3f482c7a37e4a2391b951a84b451b3d6a7].

[zaytsev]

It would be awesome if someone did something to ts - right now it is always regarded as a video file. This might have been reasonable before the advent of TypeScript, but now most of TS-files are actually code.

Unfortunately file is not perfect:

zaytsev@parallels:~/src/app/src$ file main.ts 
main.ts: Java source, ASCII text

zaytsev@parallels:~/src/app/src$ file typings.d.ts 
typings.d.ts: ASCII text

zaytsev@parallels:~/src/app/src$ file test.ts 
test.ts: Java source, UTF-8 Unicode text

Maybe match on text before video?

[ossi]

that's one hell of an overloaded extension:

> file qt/translations/qt_ru.ts
qt/translations/qt_ru.ts: XML 1.0 document, UTF-8 Unicode text, with very long lines

(this extension predates typescript by over a decade.)

we can keep the order, but need to switch to content detection:

...
# does anyone have a sample file to verify that?
type/MPEG
    Include=video
...
type/XML
    Open=linguist %f
...
include/editor
...

[zaytsev]

zaytsev@parallels:~/src$ file sample_640x360.ts 
sample_640x360.ts: data

zaytsev@parallels:~/src$ file ed24p_10.ts 
ed24p_10.ts: data

[ossi]

heh, FAIL.
then we need some coding, to enable compound conditions like this:

shell/i/.ts
type/^data$
    Open=...

(so basically AND conditions by stacking.)

i'd suggest to create a new ticket (and possibly delete the kinda off-topic comments here).

[zaytsev]

Done in #4141.