Differential D12787 Diff 33920 src/file/fileexcludefilters.cpp

Changeset View

Standalone View

View Options

src/file/fileexcludefilters.cpp

Show First 20 Lines • Show All 58 Lines • ▼ Show 20 Line(s)23const char* const s_defaultFileExcludeFilters[] = {
59 "*.nvram", 59​ "*.nvram",
60 "*.rcore", 60​ "*.rcore",
61 "*.swap", 61​ "*.swap",
62 "lzo", 62​ "lzo",
63 "litmain.sh", 63​ "litmain.sh",
64 "*.orig", 64​ "*.orig",
65 ".histfile.*", 65​ ".histfile.*",
66 ".xsession-errors*", 66​ ".xsession-errors*",
67​ "*.js.map",
68​ "*.css.map,"
69​ "*.so",
broulikUnsubmitted
Done

Don't we ignore blobs already? If not, we should also add stuff like qmlc and jsc

broulik: Don't we ignore blobs already? If not, we should also add stuff like `qmlc` and `jsc`
ngrahamAuthorUnsubmitted
Done

As far as I can tell, we do not, and they have to be manually listed. I've added qmlc and jsc. Any more you can think of?

ngraham: As far as I can tell, we do not, and they have to be manually listed. I've added `qmlc` and…
brunsUnsubmitted
Done

Static library - .a

bruns: Static library - `.a`
70​ "*.db",
71​ "*.pluginmap",
67 72​
68 // Compiled files 73​ // Compiled files
brunsUnsubmitted
Done

Probably Bytecode - we have .o above, which is also compiled

bruns: Probably `Bytecode` - we have `.o` above, which is also compiled
69 "*.class", // Java 74​ "*.class", // Java
70 "*.pyc", // Python 75​ "*.pyc", // Python
brunsUnsubmitted
Done

For python2, there is also .pyo (Python3 is covered by the __pycache__ directory filter)

bruns: For python2, there is also `.pyo` (Python3 is covered by the `__pycache__` directory filter)
71 "*.elc", // Emacs Lisp 76​ "*.elc", // Emacs Lisp
77​ "*.qrc", // QML
broulikUnsubmitted
Done

qrc is a Qt resource file, not a QML file

broulik: `qrc` is a Qt resource file, not a QML file
72 78​
73 // end of list 79​ // end of list
brunsUnsubmitted
Done

Thats not what I meant (I am not aware of anything generating a Bytecode file literally).
I meant changing the // Compiled files comment to // Bytecode files, which all the ones below are.

bruns: Thats not what I meant (I am not aware of anything generating a `Bytecode` file literally). I…
ngrahamAuthorUnsubmitted
Done

Heh, oops.

ngraham: Heh, oops.
74 nullptr 80​ nullptr
75}; 81​};
76 82​
77const int s_defaultFileExcludeFiltersVersion = 2; 83​const int s_defaultFileExcludeFiltersVersion = 3;
78 84​
79const char* const s_defaultFolderExcludeFilters[] = { 85​const char* const s_defaultFolderExcludeFilters[] = {
80 "po", 86​ "po",
81 87​
82 // VCS 88​ // VCS
83 "CVS", 89​ "CVS",
84 ".svn", 90​ ".svn",
85 ".git", 91​ ".git",
86 "_darcs", 92​ "_darcs",
87 ".bzr", 93​ ".bzr",
88 ".hg", 94​ ".hg",
89 95​
90 // development 96​ // development
91 "CMakeFiles", 97​ "CMakeFiles",
92 "CMakeTmp", 98​ "CMakeTmp",
93 "CMakeTmpQmake", 99​ "CMakeTmpQmake",
94 ".moc", 100​ ".moc",
95 ".obj", 101​ ".obj",
96 ".pch", 102​ ".pch",
97 ".uic", 103​ ".uic",
104​ ".npm",
105​ ".yarn",
106​ ".yarn-cache",
98 "__pycache__", 107​ "__pycache__",
108​ "node_modules",
109​ "node_packages",
110​ "nbproject",
99 111​
100 //misc 112​ //misc
101 "core-dumps", 113​ "core-dumps",
102 "lost+found", 114​ "lost+found",
103 115​
104 // end of list 116​ // end of list
105 nullptr 117​ nullptr
106}; 118​};
107 119​
108const int s_defaultFolderExcludeFiltersVersion = 1; 120​const int s_defaultFolderExcludeFiltersVersion = 2;
109 121​
110const char* const s_sourceCodeMimeTypes[] = { 122​const char* const s_sourceCodeMimeTypes[] = {
111 "text/css", 123​ "text/css",
112 "text/x-c++src", 124​ "text/x-c++src",
113 "text/x-c++hdr", 125​ "text/x-c++hdr",
114 "text/x-csrc", 126​ "text/x-csrc",
115 "text/x-chdr", // c header files 127​ "text/x-chdr", // c header files
116 "text/x-python", 128​ "text/x-python",
117 "text/x-assembly", 129​ "text/x-assembly",
118 "text/x-java", 130​ "text/x-java",
119 "text/x-objsrc", 131​ "text/x-objsrc",
120 "text/x-ruby", 132​ "text/x-ruby",
121 "text/x-scheme", 133​ "text/x-scheme",
122 "text/x-pascal", 134​ "text/x-pascal",
135​ "text/x-fortran",
136​ "text/x-erlang",
137​ "text/x-cmake",
138​ "text/x-lua",
123 "text/x-yacc", 139​ "text/x-yacc",
124 "text/x-sed", 140​ "text/x-sed",
125 "text/x-haskell", 141​ "text/x-haskell",
142​ "text/x-copying", // COPYING files
143​ "text/x-readme", // README files
126 "text/asp", 144​ "text/asp",
145​ "text/jsx",
146​ "text/csx",
147​ "text/vnd.trolltech.linguist",
cfeckUnsubmitted
Done

,

cfeck: ,
127 "application/x-awk", 148​ "application/x-awk",
128 "application/x-cgi", 149​ "application/x-cgi",
brunsUnsubmitted
Done

Hm, not to sure about this one - SVG typically has RDF metadata, and also everything in <tspan> tags qualifies as "content".
Do we have a generalized XML extractor?

bruns: Hm, not to sure about this one - SVG typically has RDF metadata, and also everything in…
ngrahamAuthorUnsubmitted
Done

My impression is that Baloo is really intended for user files; SVGs only get their content indexed by accident, because they happen to be textual. I don't think there's any textual content inside an SVG file that you'd actually want to have indexed.

ngraham: My impression is that Baloo is really intended for user files; SVGs only get their content…
brunsUnsubmitted
Done

SVGs are user files, and anything inside <tspan> is textual content. You can have several paragraphs with text inside SVGs.
We index the RDF metadata (author, title, ...) for PDFs, EPUB, ... so we should for SVG.
Of course it is pointless to index e.g. the tags itself, or the content of any non-textual tag, thats the reason I asked for an XML extractor.

bruns: SVGs are user files, and anything inside `<tspan>` is textual content. You can have several…
129 "application/x-csh", 150​ "application/x-csh",
130 "application/x-ipynb+json", 151​ "application/x-ipynb+json",
131 "application/x-java", 152​ "application/x-java",
132 "application/x-javascript", 153​ "application/x-javascript",
133 "application/x-perl", 154​ "application/x-perl",
134 "application/x-php", 155​ "application/x-php",
135 "application/x-python", 156​ "application/x-python",
136 "application/x-sh", 157​ "application/x-sh",
158​ "application/xml",
159​ "application/javascript",
160​ "application/json",
161​ "application/geo+json",
162​ "application/json-patch+json",
163​ "application/ld+json",
164​ "application/x-ipynb+json", // Jupyter notebooks
137 165​
138 // end of list 166​ // end of list
139 nullptr 167​ nullptr
140}; 168​};
141const int s_sourceCodeMimeTypesVersion = 1; 169​const int s_sourceCodeMimeTypesVersion = 2;
142} 170​}
143 171​
144 172​
145QStringList Baloo::defaultExcludeFilterList() 173​QStringList Baloo::defaultExcludeFilterList()
146{ 174​{
147 QStringList l; 175​ QStringList l;
148 for (int i = 0; s_defaultFileExcludeFilters[i]; ++i) 176​ for (int i = 0; s_defaultFileExcludeFilters[i]; ++i)
149 l << QLatin1String(s_defaultFileExcludeFilters[i]); 177​ l << QLatin1String(s_defaultFileExcludeFilters[i]);
Show All 30 Lines