Iconv: Change convert() return type to const std::string& (#1099)

`JDLIB::Iconv`クラスで使うバッファを`std::vector<char>`から
`std::string`に変更します。`convert()`の引数と戻り値を整理して
内部バッファのconst参照を直接返すように修正します。合わせて
`convert()`を呼び出すコードを整理します。

src/dbtree/boardbase.cpp

@@ -1079,17 +1079,16 @@ void BoardBase::receive_data( const char* data, size_t size )
     if( byte_in != std::string::npos ) {
         byte_in += 1; // 改行まで含める
 
-        int byte_out;
-        const char* rawdata_utf8 = m_iconv->convert( &*m_rawdata_left.begin(), byte_in, byte_out );
+        const std::string& rawdata_utf8 = m_iconv->convert( m_rawdata_left.data(), byte_in );
 
-        parse_subject( rawdata_utf8 );
+        parse_subject( rawdata_utf8.c_str() );
 
         // 残りを先頭に移動
         m_rawdata_left.erase( 0, byte_in );
 
 #ifdef _DEBUG
         std::cout << "BoardBase::receive_data rawdata.size = " << m_rawdata.size() << " size = " << size
-                  << " byte_in = " << byte_in << " byte_out = " << byte_out
+                  << " byte_in = " << byte_in << " byte_out = " << rawdata_utf8.size()
                   << " rawdata_left.size = " << m_rawdata_left.size() << std::endl;
 #endif
     }

src/dbtree/nodetree2chcompati.cpp

@@ -12,6 +12,9 @@
 
 #include "config/globalconf.h"
 
+#include <cstring>
+
+
 using namespace DBTREE;
 
 
@@ -115,8 +118,7 @@ char* NodeTree2chCompati::skip_status_line( char* pos, int status )
 
     // エラー
     if( status != 1 ){
-        int byte;
-        std::string ext_err = std::string( m_iconv->convert( pos_msg, pos - pos_msg, byte ) );
+        const std::string& ext_err = m_iconv->convert( pos_msg, pos - pos_msg );
         set_ext_err( ext_err );
         MISC::ERRMSG( ext_err );
     }
@@ -137,7 +139,9 @@ const char* NodeTree2chCompati::raw2dat( char* rawlines, int& byte )
     assert( m_iconv != nullptr );
 
     // バッファ自体はiconvクラスの中で持っているのでポインタだけもらう
-    return  m_iconv->convert( rawlines, strlen( rawlines ), byte );
+    const std::string& result = m_iconv->convert( rawlines, std::strlen( rawlines ) );
+    byte = result.size();
+    return result.c_str();
 }
 
 

src/dbtree/nodetreejbbs.cpp

@@ -13,6 +13,7 @@
 
 #include "global.h"
 
+#include <cstring>
 #include <sstream>
 
 
@@ -22,7 +23,7 @@ constexpr std::size_t kBufSizeDecodedLines = 512 * 1024;
 
 #define APPEND_SECTION( num ) do {\
 if( lng_sec[ num ] ){ \
-m_decoded_lines.append( lines + pos_sec[ num ], lng_sec[ num ] ); \
+m_decoded_lines.append( lines.data() + pos_sec[ num ], lng_sec[ num ] ); \
 } } while( 0 )
 
 
@@ -134,33 +135,32 @@ const char* NodeTreeJBBS::raw2dat( char* rawlines, int& byte )
 {
     assert( m_iconv != nullptr );
 
-    int byte_lines;
-    const char* lines = m_iconv->convert( rawlines, strlen( rawlines ), byte_lines );
+    const std::string& lines = m_iconv->convert( rawlines, std::strlen( rawlines ) );
 
     int number = id_header() + 1;
 
 #ifdef _DEBUG
-    std::cout << "NodeTreeJBBS::raw2dat : byte_lines = " << byte_lines << std::endl;
+    std::cout << "NodeTreeJBBS::raw2dat : byte_lines = " << lines.size() << std::endl;
 #endif
 
     // セクション分けして再合成する
     m_decoded_lines.clear();
     byte = 0;
-    int pos = 0;
+    std::size_t pos = 0;
     int section = 0;
-    int pos_sec[ MAX_SECTION ];
+    std::size_t pos_sec[ MAX_SECTION ];
     int lng_sec[ MAX_SECTION ];
     memset( lng_sec, 0, sizeof( int ) * MAX_SECTION );
 
-    while( pos < byte_lines ){
+    while( pos < lines.size() ){
 
         // セクション分け
         pos_sec[ section ] = pos;
-        while( !( lines[ pos ] == '<' && lines[ pos +1 ] == '>' ) && lines[ pos ] != '\n' && pos < byte_lines ) ++pos;
+        while( pos < lines.size() && lines.compare( pos, 2, "<>", 2 ) != 0 && lines[pos] != '\n' ) ++pos;
         lng_sec[ section ] = pos - pos_sec[ section ];
 
         // 最後の行で、かつ壊れている場合
-        if( pos >= byte_lines ){
+        if( pos >= lines.size() ){
             set_broken( true );
             break;
         }
@@ -174,7 +174,7 @@ const char* NodeTreeJBBS::raw2dat( char* rawlines, int& byte )
                 // 透明あぼーんの判定
                 char number_str[ 64 ];
                 memset( number_str, 0, 64 );
-                memcpy( number_str, lines + pos_sec[ 0 ], MIN( lng_sec[ 0 ], 64 -1 ) );
+                memcpy( number_str, lines.data() + pos_sec[ 0 ], MIN( lng_sec[ 0 ], 64 -1 ) );
                 int number_in = atoi( number_str );
 
                 while( number_in > number ){
@@ -203,7 +203,7 @@ const char* NodeTreeJBBS::raw2dat( char* rawlines, int& byte )
 
                     m_decoded_lines.append( " ID:" );
 
-                    m_decoded_lines.append( lines + pos_sec[ i ], lng_sec[ i ] );
+                    m_decoded_lines.append( lines.data() + pos_sec[ i ], lng_sec[ i ] );
                 }
                 m_decoded_lines.append( "<>" );
 
@@ -239,7 +239,7 @@ const char* NodeTreeJBBS::raw2dat( char* rawlines, int& byte )
                 set_broken( true );
 
                 // その行は飛ばす
-                while( pos < byte_lines && lines[ pos ] != '\n' ) ++pos;
+                while( pos < lines.size() && lines[ pos ] != '\n' ) ++pos;
                 ++pos;
                 section = 0;
                 memset( lng_sec, 0, sizeof( int ) * MAX_SECTION );

src/dbtree/nodetreemachi.cpp

@@ -240,8 +240,7 @@ const char* NodeTreeMachi::raw2dat( char* rawlines, int& byte )
     std::string buffer;
 
     // 文字コード変換
-    int byte_lines;
-    const char* str_lines = m_iconv->convert( rawlines, strlen( rawlines ), byte_lines );
+    const std::string& str_lines = m_iconv->convert( rawlines, std::strlen( rawlines ) );
 
     std::list< std::string > lines = MISC::get_lines( str_lines );
     for( std::string& line : lines ) {
@@ -357,8 +356,7 @@ void NodeTreeMachi::receive_finish()
     if( m_buffer_for_200.size() >= 2 && m_buffer_for_200[ 0 ] == '<'
             && ( m_buffer_for_200[ 1 ] == 'E' || m_buffer_for_200[ 1 ] == 'h' ) ) {
 
-        int byte_lines;
-        std::string str_lines( m_iconv->convert( &*m_buffer_for_200.begin(), m_buffer_for_200.size(), byte_lines ) );
+        const std::string& str_lines = m_iconv->convert( m_buffer_for_200.data(), m_buffer_for_200.size() );
 
 #ifdef _DEBUG    
         std::cout << str_lines << std::endl;

src/dbtree/root.cpp

@@ -399,8 +399,7 @@ void Root::receive_finish()
 
     // 文字コードを変換してXML作成
     JDLIB::Iconv libiconv{ "UTF-8", "MS932" };
-    int byte_out;
-    const std::string rawdata_utf8 = libiconv.convert( &*m_rawdata.begin(), m_rawdata.size(),  byte_out );
+    const std::string& rawdata_utf8 = libiconv.convert( m_rawdata.data(), m_rawdata.size() );
     bbsmenu2xml( rawdata_utf8 );
 
     if( m_xml_document.hasChildNodes() )

src/jdlib/jdiconv.cpp

@@ -88,17 +88,19 @@ Iconv::~Iconv()
  * @details UTF-8から別のエンコーディングに変換するときは表現できない文字をHTML数値文字参照(10進数表記)に置き換える。
  * @param[in] str_in    変換するテキストへのポインター
  * @param[in] size_in   変換するテキストの長さ
- * @param[out] size_out 変換した結果の長さ
- * @return 変換したテキストへのポインター。
- * @note 返り値は Iconv オブジェクトを破棄、または convert() を再び呼び出すとdangling pointerになる。
+ * @return 変換したテキストへの参照
+ * @note 返り値は Iconv オブジェクトを破棄、または convert() を再び呼び出すとdangling referenceになる。
  */
-const char* Iconv::convert( char* str_in, int size_in, int& size_out )
+const std::string& Iconv::convert( char* str_in, std::size_t size_in )
 {
 #ifdef _DEBUG
     std::cout << "Iconv::convert size_in = " << size_in << std::endl;
 #endif
 
-    if( m_cd == ( GIConv ) -1 ) return nullptr;
+    if( m_cd == ( GIConv ) -1 ) {
+        m_buf.clear();
+        return m_buf;
+    }
 
     if( const std::size_t size_in_x2 = size_in * 2;
             size_in_x2 >= m_buf.size() ) {
@@ -345,11 +347,11 @@ const char* Iconv::convert( char* str_in, int size_in, int& size_out )
 
     } while( buf_in_tmp < buf_in_end );
 
-    size_out = buf_out_tmp - m_buf.data();
-    *buf_out_tmp = '\0';
+    const std::size_t size_out = buf_out_tmp - m_buf.data();
+    m_buf.resize( size_out );
 
 #ifdef _DEBUG
     std::cout << "Iconv::convert size_out = " << size_out << std::endl;
 #endif
-    return m_buf.data();
+    return m_buf;
 }

src/jdlib/jdiconv.h

@@ -16,7 +16,7 @@ namespace JDLIB
     {
         GIConv m_cd; ///< iconv実装は環境で違いがあるためGlibのラッパーAPIを利用する
 
-        std::vector<char> m_buf; ///< 出力バッファ
+        std::string m_buf; ///< 出力バッファ
 
         std::string m_coding_from; ///< 変換元の文字エンコーディング
         bool m_coding_to_is_utf8; ///< 変換先の文字エンコーディングがUTF-8ならtrue
@@ -29,7 +29,7 @@ namespace JDLIB
         ~Iconv();
 
         // テキストの文字エンコーディングを変換する
-        const char* convert( char* str_in, int size_in, int& size_out );
+        const std::string& convert( char* str_in, std::size_t size_in );
     };
 }
 

src/jdlib/miscutil.cpp

@@ -1489,9 +1489,7 @@ std::string MISC::Iconv( const std::string& str, const std::string& coding_to, c
     std::string str_bk = str;
 
     JDLIB::Iconv libiconv( coding_to, coding_from );
-    int byte_out;
-
-    std::string str_enc = libiconv.convert( &*str_bk.begin(), str_bk.size(), byte_out );
+    std::string str_enc = libiconv.convert( str_bk.data(), str_bk.size() );
 
     return str_enc;
 }

src/message/messageviewbase.cpp

@@ -913,8 +913,7 @@ void MessageViewBase::slot_switch_page( Gtk::Widget*, guint page )
             // 対応していないUnicode文字を文字参照の形式(&#nnnn;)で表示する
             if( DBTREE::get_unicode( get_url() ) == "change" ){
                 // MS932等に無い文字を数値文字参照にするために文字コードを変換する
-                int byte_out;
-                const std::string str_enc = m_iconv->convert( msg.data(), msg.size(), byte_out );
+                const std::string& str_enc = m_iconv->convert( msg.data(), msg.size() );
                 msg = MISC::Iconv( str_enc, "UTF-8", DBTREE::board_charset( get_url() ) );
             }
             else{
@@ -994,8 +993,7 @@ void MessageViewBase::show_status()
     }
     else if( m_text_changed )
     {
-        int byte_out;
-        std::string str_enc = m_iconv->convert( message.data(), message.size(), byte_out );
+        const std::string& str_enc = m_iconv->convert( message.data(), message.size() );
         m_lng_str_enc = str_enc.length();
 
         // 特殊文字の文字数を計算

src/message/post.cpp

@@ -239,8 +239,7 @@ void Post::receive_finish()
     {
         const std::string charset = DBTREE::board_charset( m_url );
         JDLIB::Iconv libiconv( "UTF-8", charset );
-        int byte_out;
-        m_return_html = libiconv.convert( &*m_rawdata.begin(), m_rawdata.size(), byte_out );
+        m_return_html = libiconv.convert( m_rawdata.data(), m_rawdata.size() );
     }
 
 #ifdef _DEBUG

src/skeleton/textloader.cpp

@@ -173,8 +173,7 @@ void TextLoader::receive_finish()
 
     // UTF-8に変換しておく
     JDLIB::Iconv libiconv( "UTF-8", get_charset() );
-    int byte_out;
-    m_data = libiconv.convert( &*m_rawdata.begin(), m_rawdata.size(),  byte_out );
+    m_data = libiconv.convert( m_rawdata.data(), m_rawdata.size() );
     clear();
 
     receive_cookies();