diff options
author | Jay Berkenbilt <ejb@ql.org> | 2020-04-03 18:06:23 +0200 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2020-04-03 18:17:57 +0200 |
commit | 54726930dffa78f84ab3447cbe464b587694674e (patch) | |
tree | e4ce1c5f37eabd7dc0107206929e5d3e2e46cf53 | |
parent | da1258e2e679bc8b9471b5d9ff35d8473e76e062 (diff) | |
download | qpdf-54726930dffa78f84ab3447cbe464b587694674e.tar.zst |
Remove redundant methods in QUtil
This was being saved until we had to break ABI.
-rw-r--r-- | include/qpdf/QUtil.hh | 10 | ||||
-rw-r--r-- | libqpdf/QUtil.cc | 22 |
2 files changed, 2 insertions, 30 deletions
diff --git a/include/qpdf/QUtil.hh b/include/qpdf/QUtil.hh index e44f95a2..fa6a76aa 100644 --- a/include/qpdf/QUtil.hh +++ b/include/qpdf/QUtil.hh @@ -313,17 +313,11 @@ namespace QUtil // Filename is UTF-8 encoded, even on Windows, as described in the // comments for safe_fopen. QPDF_DLL - std::list<std::string> read_lines_from_file(char const* filename); - // ABI: make preserve_eol an optional arg and remove single-arg version - QPDF_DLL std::list<std::string> read_lines_from_file( - char const* filename, bool preserve_eol); - QPDF_DLL - std::list<std::string> read_lines_from_file(std::istream&); - // ABI: make preserve_eol an optional arg and remove single-arg version + char const* filename, bool preserve_eol = false); QPDF_DLL std::list<std::string> read_lines_from_file( - std::istream&, bool preserve_eol); + std::istream&, bool preserve_eol = false); QPDF_DLL std::list<std::string> read_lines_from_file( FILE*, bool preserve_eol = false); diff --git a/libqpdf/QUtil.cc b/libqpdf/QUtil.cc index 5c4bdeed..04aaf51f 100644 --- a/libqpdf/QUtil.cc +++ b/libqpdf/QUtil.cc @@ -1060,18 +1060,6 @@ static bool read_char_from_FILE(char& ch, FILE* f) } std::list<std::string> -QUtil::read_lines_from_file(char const* filename) -{ - // ABI: remove this method - std::list<std::string> lines; - FILE* f = safe_fopen(filename, "rb"); - FileCloser fc(f); - auto next_char = [&f](char& ch) { return read_char_from_FILE(ch, f); }; - read_lines_from_file(next_char, lines, false); - return lines; -} - -std::list<std::string> QUtil::read_lines_from_file(char const* filename, bool preserve_eol) { std::list<std::string> lines; @@ -1083,16 +1071,6 @@ QUtil::read_lines_from_file(char const* filename, bool preserve_eol) } std::list<std::string> -QUtil::read_lines_from_file(std::istream& in) -{ - // ABI: remove this method - std::list<std::string> lines; - auto next_char = [&in](char& ch) { return (in.get(ch)) ? true: false; }; - read_lines_from_file(next_char, lines, false); - return lines; -} - -std::list<std::string> QUtil::read_lines_from_file(std::istream& in, bool preserve_eol) { std::list<std::string> lines; |