Do not create the list of supported mime types on demand
and use a static stringlist instead.
Order alphabetically where required
Details
Details
- Reviewers
bruns - Commits
- R286:0375eee8a128: move supported mimetypes to static string list
Diff Detail
Diff Detail
- Repository
- R286 KFileMetaData
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
| src/extractors/epubextractor.cpp | ||
|---|---|---|
| 39 | The idea behind using anonymous namespaces is to neither "pollute" the class namespace nor the global namespace. Remove the "EPubExtractor::" qualifier, see fetchMetadata. | |
| src/extractors/epubextractor.cpp | ||
|---|---|---|
| 39 | All the stringlists are currently implemented as a private static member of the class... would you like to change that to a local static variable? | |
| src/extractors/epubextractor.cpp | ||
|---|---|---|
| 39 | I consider this cleaner, yes. | |
| src/extractors/taglibextractor.cpp | ||
|---|---|---|
| 71 | w < x | |