Change function argument type for MISC::get_hostname() (JDimproved#1293)

引数のURLからホスト名を取得する関数の引数型を`std::string`から
`std::string_view`に変更します。

修正前は`std::string_view`を使いたい処理の中で`get_hostname()`を
呼び出す部分があると一時的な`std::string`を作る必要があったため
`std::string_view`が利用にしくい問題がありました。

src/jdlib/miscutil.cpp

@@ -1764,27 +1764,28 @@ std::string MISC::tolower_str( const std::string& str )
 
 
 
-//
-// path からホスト名だけ取り出す
-//
-// protocol = false のときはプロトコルを除く
-//
-std::string MISC::get_hostname( const std::string& path, bool protocol )
+/** @brief path からホスト名だけ取り出してコピーを返す
+ *
+ * @param[in] path     ホスト名を取り出す文字列
+ * @param[in] protocol false のときはプロトコルを除く
+ * @return ホスト名の文字列 (protocolがtrueのときは先頭にプロトコルが付く)
+ */
+std::string MISC::get_hostname( std::string_view path, bool protocol )
 {
-    int lng = 0;
+    std::size_t lng = 0;
     if( path.rfind( "http://", 0 ) == 0 ) lng = strlen( "http://" );
     else if( path.rfind( "https://", 0 ) == 0 ) lng = strlen( "https://" );
     else if( path.rfind( "ftp://", 0 ) == 0 ) lng = strlen( "ftp://" );
     if( !lng ) return std::string();
 
-    int pos = 0;
+    std::size_t pos = 0;
     if( ! protocol ) pos = lng;
 
     size_t i = path.find( '/', lng );
 
-    if( i == std::string::npos ) return path.substr( pos );
+    if( i == std::string_view::npos ) return std::string{ path.substr( pos ) };
 
-    return path.substr( pos, i - pos );
+    return std::string{ path.substr( pos, i - pos ) };
 }
 
 

src/jdlib/miscutil.h

@@ -232,7 +232,7 @@ namespace MISC
 
     // path からホスト名だけ取り出す
     // protocol = false のときはプロトコルを除く
-    std::string get_hostname( const std::string& path, const bool protocol = true );
+    std::string get_hostname( std::string_view path, const bool protocol = true );
 
     // path からファイル名だけ取り出す
     std::string get_filename( const std::string& path );