diff options
| author | Tor Lillqvist <tlillqvist@suse.com> | 2012-02-14 13:06:10 +0200 |
|---|---|---|
| committer | Tor Lillqvist <tml@iki.fi> | 2012-02-15 16:20:01 +0200 |
| commit | 3a2cbe739db31139e03beacd4c886a486acf6922 (patch) | |
| tree | e7a663fff8e138c68275cada71320ce854977ea4 /sc | |
| parent | 4eaf1feb56632e8e4da34c9a5275bd409395f009 (diff) | |
Add comment wondering why this loadFile() isn't in osl
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/qa/unit/helper/qahelper.hxx | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/sc/qa/unit/helper/qahelper.hxx b/sc/qa/unit/helper/qahelper.hxx index c679c73a4494..0974e7053946 100644 --- a/sc/qa/unit/helper/qahelper.hxx +++ b/sc/qa/unit/helper/qahelper.hxx @@ -38,6 +38,12 @@ #include <osl/detail/android-bootstrap.h> +// Why is this here and not in osl, and using the already existing file +// handling APIs? Do we really want to add arbitrary new file handling +// wrappers here and there (and then having to handle the Android (and +// eventually perhaps iOS) special cases here, too)? Please move this to osl, +// it sure looks gemerally useful. Or am I missing something? + void loadFile(const rtl::OUString& aFileName, std::string& aContent) { rtl::OString aOFileName = rtl::OUStringToOString(aFileName, RTL_TEXTENCODING_UTF8); |