[PATCH] D148821: [SystemZ][z/OS] Create utility functions for converting between EBCDIC and UTF-8.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/lib/Support/ConvertEBCDIC.cpp:100
+
+std::error_code ConverterEBCDIC::convertToUTF8(StringRef Source,
+                                               SmallVectorImpl<char> &Result) {
----------------
Maybe convertToUTF8 should return void, instead of an std::error_code, because it can't fail.  (The only reason I can think of to keep it is consistency with conversions which can fail.  But that doesn't seem compelling given an explicit entry point like this.)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148821/new/

https://reviews.llvm.org/D148821