summaryrefslogtreecommitdiff
path: root/dtrans/source/cnttype/mcnttype.hxx
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2018-10-11 17:31:30 +0200
committerCaolán McNamara <caolanm@redhat.com>2018-10-19 10:35:04 +0200
commit90173417577338ba837cd1ed51276b3e3c8a50df (patch)
tree919f313cf8d860278d954f62ac84e380172d1dbd /dtrans/source/cnttype/mcnttype.hxx
parente3e7f45ea4b07e2cbeaad75d46edc1b8fb9160fc (diff)
tdf#120158: Base CMimeContentType on INetMIME::scanContentType
...instead of using yet another local implementation of parsing media types. CMimeContentType is the implementation of the UNO css.datatransfer.XMimeContentType interface. One observable change in behavior is that type, subtype, and parameter names will now always be reported in lower case instead of with the casing from the input preserved (but those differences in casing are functionally equivalent per the media type specification). Also, parameter names supplied to the hasParameter and getParameterValue functions are now also treated case-insensitive. The upside of this change is that INetMIME::scanContentType (via "The encoding of rMediaType should be US-ASCII, but any Unicode values in the range U+0080.. U+FFFF are interpreted 'as appropriate.'") already implicitly supports RFC 6532 "Internationalized Email Headers" extensions for media types, allowing quoted- string parameter values to contain non-ASCII Unicode characters. That means that tfd#120158 "Impossible to paste special in Writer from Calc in Libreoffice 6.1.x in some UI languages - the dialogue caption says 'unknown source'" can be fixed by just allowing non-ASCII typename parameters being generated in ImplGetParameterString in svtools/source/misc/transfer.cxx, and reverting the problematic (see the comments there) previous fix <https://gerrit.libreoffice.org/61601> "tdf#120158: fix ImplGetParameterString for typename". (Which will be done in a follow-up commit, to ease potential backporting, as that previous fix has already been backported to some versions but not to others.) Reviewed-on: https://gerrit.libreoffice.org/61684 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com> (cherry picked from commit b75e3ded1f73c943fb09f117e476e367ac0c3413) Conflicts: dtrans/source/cnttype/mcnttype.cxx Change-Id: I5d4d3586e8046f288a97605b000e262a8db5a4e9 Reviewed-on: https://gerrit.libreoffice.org/61702 Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'dtrans/source/cnttype/mcnttype.hxx')
-rw-r--r--dtrans/source/cnttype/mcnttype.hxx3
1 files changed, 0 insertions, 3 deletions
diff --git a/dtrans/source/cnttype/mcnttype.hxx b/dtrans/source/cnttype/mcnttype.hxx
index 35fb37fbdba1..68c4bff5a258 100644
--- a/dtrans/source/cnttype/mcnttype.hxx
+++ b/dtrans/source/cnttype/mcnttype.hxx
@@ -66,10 +66,7 @@ private:
::osl::Mutex m_aMutex;
OUString m_MediaType;
OUString m_MediaSubtype;
- OUString m_ContentType;
std::map< OUString, OUString > m_ParameterMap;
- sal_Int32 m_nPos;
- OUString m_nxtSym;
};
#endif