diff options
Diffstat (limited to 'protocols/Omegle/src')
| -rw-r--r-- | protocols/Omegle/src/chat.cpp | 29 | ||||
| -rw-r--r-- | protocols/Omegle/src/client.h | 12 | ||||
| -rw-r--r-- | protocols/Omegle/src/communication.cpp | 14 | ||||
| -rw-r--r-- | protocols/Omegle/src/connection.cpp | 2 | ||||
| -rw-r--r-- | protocols/Omegle/src/dialogs.cpp | 11 | ||||
| -rw-r--r-- | protocols/Omegle/src/proto.cpp | 5 | ||||
| -rw-r--r-- | protocols/Omegle/src/theme.cpp | 22 | ||||
| -rw-r--r-- | protocols/Omegle/src/utils.cpp | 12 | ||||
| -rw-r--r-- | protocols/Omegle/src/utils.h | 12 | 
9 files changed, 40 insertions, 79 deletions
diff --git a/protocols/Omegle/src/chat.cpp b/protocols/Omegle/src/chat.cpp index cf98c66a32..fcc7706690 100644 --- a/protocols/Omegle/src/chat.cpp +++ b/protocols/Omegle/src/chat.cpp @@ -37,7 +37,7 @@ void OmegleProto::UpdateChat(const TCHAR *name, const TCHAR *message, bool addto  		name = TranslateT("Server");
  		gce.bIsMe = false;
  	}
 -	else gce.bIsMe = !_tcscmp(name, this->facy.nick_);
 +	else gce.bIsMe = !mir_tstrcmp(name, this->facy.nick_);
  	if (addtolog)
  		gce.dwFlags  |= GCEF_ADDTOLOG;
 @@ -83,7 +83,7 @@ int OmegleProto::OnChatEvent(WPARAM wParam,LPARAM lParam)  			if (!stricmp(command.c_str(), "new"))
  			{
  				facy.spy_mode_ = false;
 -				facy.question_ = "";
 +				facy.question_.clear();
  				ForkThread(&OmegleProto::NewChatWorker, NULL);
  				break;
 @@ -96,7 +96,7 @@ int OmegleProto::OnChatEvent(WPARAM wParam,LPARAM lParam)  			else if (!stricmp(command.c_str(), "spy"))
  			{
  				facy.spy_mode_ = true;
 -				facy.question_ = "";
 +				facy.question_.clear();
  				ForkThread(&OmegleProto::NewChatWorker, NULL);
  				break;
 @@ -117,13 +117,13 @@ int OmegleProto::OnChatEvent(WPARAM wParam,LPARAM lParam)  					}
  				} else {
  					// Save actual question as last question
 -					if (strlen(params.c_str()) >= OMEGLE_QUESTION_MIN_LENGTH)
 +					if (params.length() >= OMEGLE_QUESTION_MIN_LENGTH)
  					{
  						setU8String( OMEGLE_KEY_LAST_QUESTION, params.c_str());
  					}
  				}
 -				if (strlen(params.c_str()) < OMEGLE_QUESTION_MIN_LENGTH)
 +				if (params.length() < OMEGLE_QUESTION_MIN_LENGTH)
  				{
  					UpdateChat(NULL, TranslateT("Your question is too short."), false);
  					break;
 @@ -184,7 +184,7 @@ int OmegleProto::OnChatEvent(WPARAM wParam,LPARAM lParam)  	case GC_USER_LEAVE:
  	case GC_SESSION_TERMINATE:
 -		mir_free( facy.nick_ );
 +		facy.nick_ = NULL;
  		ForkThread(&OmegleProto::StopChatWorker, NULL);
  		break;
  	}
 @@ -234,7 +234,7 @@ void OmegleProto::AddChatContact(const TCHAR *name)  	if (name == NULL)
  		gce.bIsMe = false;
  	else 
 -		gce.bIsMe = !_tcscmp(name, this->facy.nick_);
 +		gce.bIsMe = mir_tstrcmp(name, this->facy.nick_);
  	if (gce.bIsMe)
  		gce.ptszStatus = _T("Admin");
 @@ -255,7 +255,7 @@ void OmegleProto::DeleteChatContact(const TCHAR *name)  	if (name == NULL)
  		gce.bIsMe = false;
  	else 
 -		gce.bIsMe = !_tcscmp(name, this->facy.nick_);
 +		gce.bIsMe = mir_tstrcmp(name, this->facy.nick_);
  	CallServiceSync(MS_GC_EVENT,0,reinterpret_cast<LPARAM>(&gce));
  }
 @@ -326,20 +326,13 @@ void OmegleProto::SetChatStatus(int status)  	if(status == ID_STATUS_ONLINE)
  	{		
 -		// Free previously loaded name
 -		mir_free(facy.nick_);
 -		
  		// Load actual name from database
 -		DBVARIANT dbv;
 -		if ( !db_get_ts(NULL, m_szModuleName, OMEGLE_KEY_NAME, &dbv))
 -		{
 -			facy.nick_ = mir_tstrdup(dbv.ptszVal);
 -			db_free(&dbv);
 -		} else {
 +		facy.nick_ = db_get_tsa(NULL, m_szModuleName, OMEGLE_KEY_NAME);
 +		if (facy.nick_ == NULL) {
  			facy.nick_ = mir_tstrdup(TranslateT("You"));
  			db_set_ts(NULL, m_szModuleName, OMEGLE_KEY_NAME, facy.nick_);
  		}
 -		
 +
  		// Add self contact
  		AddChatContact(facy.nick_);
 diff --git a/protocols/Omegle/src/client.h b/protocols/Omegle/src/client.h index a6e28c5a2c..8506c14556 100644 --- a/protocols/Omegle/src/client.h +++ b/protocols/Omegle/src/client.h @@ -37,7 +37,6 @@ public:  	// Client definition
  	Omegle_client( )
  	{
 -		chat_id_ = server_ = question_ = "";
  		nick_ = NULL;
  		//msgid_ = 0;
  		send_message_lock_ = NULL;
 @@ -63,7 +62,7 @@ public:  	std::string chat_id_;
  	std::string server_;
  	std::string question_;
 -	TCHAR *nick_;
 +	ptrT nick_;
  	HANDLE send_message_lock_;
  	//int	msgid_;
 @@ -75,7 +74,6 @@ public:  	bool spy_mode_;
  	// Data storage
 -	std::map< std::string, std::string >    headers;
  	void    store_headers( http::response* resp, NETLIBHTTPHEADER* headers, int headers_count );
  	std::string get_server( bool not_last = false );
 @@ -84,9 +82,9 @@ public:  	// Connection handling
  	unsigned int error_count_;
 -	bool    handle_entry( std::string method );
 -	bool    handle_success( std::string method );
 -	bool    handle_error( std::string method, bool force_disconnect = false );
 +	bool    handle_entry(const std::string &method);
 +	bool    handle_success(const std::string &method);
 +	bool    handle_error(const std::string &method, bool force_disconnect = false );
  	void __inline increment_error( ) { error_count_++; }
  	void __inline decrement_error( ) { if ( error_count_ > 0 ) error_count_--; }
 @@ -102,7 +100,7 @@ public:  	std::string get_page( int );
 -	bool    send_message( std::string message_text );
 +	bool    send_message(const std::string &message_text );
  	// HTTP communication
  	http::response  flap( const int request_type, std::string* request_data = NULL, std::string* get_data = NULL );
 diff --git a/protocols/Omegle/src/communication.cpp b/protocols/Omegle/src/communication.cpp index 377a9e6e53..fb2641a88b 100644 --- a/protocols/Omegle/src/communication.cpp +++ b/protocols/Omegle/src/communication.cpp @@ -96,20 +96,20 @@ http::response Omegle_client::flap( const int request_type, std::string* request  	return resp;
  }
 -bool Omegle_client::handle_entry( std::string method )
 +bool Omegle_client::handle_entry(const std::string &method )
  {
  	parent->debugLogA("   >> Entering %s()", method.c_str());
  	return true;
  }
 -bool Omegle_client::handle_success( std::string method )
 +bool Omegle_client::handle_success(const std::string &method )
  {
  	parent->debugLogA("   << Quitting %s()", method.c_str());
  	reset_error();
  	return true;
  }
 -bool Omegle_client::handle_error( std::string method, bool force_disconnect )
 +bool Omegle_client::handle_error(const std::string &method, bool force_disconnect )
  {
  	bool result;
  	increment_error();
 @@ -138,7 +138,7 @@ std::string Omegle_client::get_server( bool not_last )  {
  	BYTE q = not_last ? 1 : 0;	
 -	BYTE server = db_get_b(NULL, parent->m_szModuleName, OMEGLE_KEY_SERVER, 0);
 +	int server = db_get_b(NULL, parent->m_szModuleName, OMEGLE_KEY_SERVER, 0);
  	if (server < 0 || server >= (SIZEOF(servers)-q))
  		server = 0;
 @@ -152,7 +152,7 @@ std::string Omegle_client::get_server( bool not_last )  std::string Omegle_client::get_language()
  {
 -	BYTE language = db_get_b(NULL, parent->m_szModuleName, OMEGLE_KEY_LANGUAGE, 0);
 +	int language = db_get_b(NULL, parent->m_szModuleName, OMEGLE_KEY_LANGUAGE, 0);
  	if (language < 0 || language >= (SIZEOF(languages)))
  		language = 0;
 @@ -304,7 +304,7 @@ NETLIBHTTPHEADER* Omegle_client::get_request_headers( int request_type, int* hea  void Omegle_client::store_headers( http::response* resp, NETLIBHTTPHEADER* headers, int headersCount )
  {
 -	for ( int i = 0; i < headersCount; i++ )
 +	for ( size_t i = 0; i < headersCount; i++ )
  	{
  		std::string header_name = headers[i].szName;
  		std::string header_value = headers[i].szValue;
 @@ -740,7 +740,7 @@ bool Omegle_client::events( )  	}
  }
 -bool Omegle_client::send_message( std::string message_text )
 +bool Omegle_client::send_message(const std::string &message_text )
  {
  	handle_entry( "send_message" );
 diff --git a/protocols/Omegle/src/connection.cpp b/protocols/Omegle/src/connection.cpp index 46625c6eea..4037abadb1 100644 --- a/protocols/Omegle/src/connection.cpp +++ b/protocols/Omegle/src/connection.cpp @@ -100,7 +100,7 @@ void OmegleProto::StopChat(bool disconnect)  	}
  	facy.state_ = STATE_INACTIVE;
 -	facy.chat_id_ = "";
 +	facy.chat_id_.clear();
  }
  void OmegleProto::NewChat()
 diff --git a/protocols/Omegle/src/dialogs.cpp b/protocols/Omegle/src/dialogs.cpp index 01dbe75516..5678d6c50b 100644 --- a/protocols/Omegle/src/dialogs.cpp +++ b/protocols/Omegle/src/dialogs.cpp @@ -38,12 +38,9 @@ static BOOL StoreDBCheckState(OmegleProto* ppro, HWND hwnd, int idCtrl, const ch  static void LoadDBText(OmegleProto* ppro, HWND hwnd, int idCtrl, const char* szSetting)
  {
 -	DBVARIANT dbv;
 -	if ( !db_get_ts(NULL, ppro->m_szModuleName, szSetting, &dbv))
 -	{
 -		SetDlgItemText(hwnd, idCtrl, dbv.ptszVal);
 -		db_free(&dbv);
 -	}
 +	ptrT tstr( db_get_tsa(NULL, ppro->m_szModuleName, szSetting));
 +	if (tstr)
 +		SetDlgItemText(hwnd, idCtrl, tstr);
  }
  static void StoreDBText(OmegleProto* ppro, HWND hwnd, int idCtrl, const char* szSetting)
 @@ -51,7 +48,7 @@ static void StoreDBText(OmegleProto* ppro, HWND hwnd, int idCtrl, const char* sz  	TCHAR tstr[250+1];
  	GetDlgItemText(hwnd, idCtrl, tstr, SIZEOF(tstr));
 -	if ( _tcsclen( tstr ) > 0 ) {
 +	if (tstr[0] != '\0') {
  		db_set_ts(NULL, ppro->m_szModuleName, szSetting, tstr);
  	} else {
  		db_unset(NULL, ppro->m_szModuleName, szSetting);
 diff --git a/protocols/Omegle/src/proto.cpp b/protocols/Omegle/src/proto.cpp index e8cc0c4d42..bc785a9c44 100644 --- a/protocols/Omegle/src/proto.cpp +++ b/protocols/Omegle/src/proto.cpp @@ -46,9 +46,6 @@ OmegleProto::OmegleProto(const char* proto_name, const TCHAR* username) :  	NETLIBUSER nlu = {sizeof(nlu)};
  	nlu.flags = NUF_INCOMING | NUF_OUTGOING | NUF_HTTPCONNS | NUF_TCHAR;
  	nlu.szSettingsModule = m_szModuleName;
 -	char module[512];
 -	mir_snprintf(module,SIZEOF(module),"%sAv",m_szModuleName);
 -	nlu.szSettingsModule = module;
  	mir_sntprintf(descr,SIZEOF(descr),TranslateT("%s server connection"),m_tszUserName);
  	nlu.ptszDescriptiveName = descr;
  	m_hNetlibUser = (HANDLE)CallService(MS_NETLIB_REGISTERUSER,0,(LPARAM)&nlu);
 @@ -76,8 +73,6 @@ OmegleProto::~OmegleProto( )  	CloseHandle( this->facy.send_message_lock_ );
  	CloseHandle( this->events_loop_lock_ );
  	CloseHandle( this->facy.connection_lock_ );
 -
 -	mir_free( this->facy.nick_ );
  }
  //////////////////////////////////////////////////////////////////////////////
 diff --git a/protocols/Omegle/src/theme.cpp b/protocols/Omegle/src/theme.cpp index c9b41d93ec..bfc7389230 100644 --- a/protocols/Omegle/src/theme.cpp +++ b/protocols/Omegle/src/theme.cpp @@ -45,25 +45,3 @@ HANDLE GetIconHandle(const char* name)  	}
  	return 0;
  }
 -
 -// Helper functions
 -static OmegleProto * GetInstanceByHContact(MCONTACT hContact)
 -{
 -	char *proto = GetContactProto(hContact);
 -	if (!proto)
 -		return 0;
 -
 -	for(int i=0; i<g_Instances.getCount(); i++)
 -		if (!strcmp(proto,g_Instances[i].m_szModuleName))
 -			return &g_Instances[i];
 -
 -	return 0;
 -}
 -
 -template<int (__cdecl OmegleProto::*Fcn)(WPARAM,LPARAM)>
 -INT_PTR GlobalService(WPARAM wParam,LPARAM lParam)
 -{
 -	OmegleProto *proto = GetInstanceByHContact(reinterpret_cast<HANDLE>(wParam));
 -	return proto ? (proto->*Fcn)(wParam,lParam) : 0;
 -}
 -
 diff --git a/protocols/Omegle/src/utils.cpp b/protocols/Omegle/src/utils.cpp index bc7bac240a..09ced8f1db 100644 --- a/protocols/Omegle/src/utils.cpp +++ b/protocols/Omegle/src/utils.cpp @@ -27,7 +27,7 @@ std::string utils::url::encode(const std::string &s)  	return (char*)ptrA( mir_urlEncode( s.c_str()));
  }
 -void utils::text::replace_first( std::string* data, std::string from, std::string to )
 +void utils::text::replace_first( std::string* data, const std::string &from, const std::string &to )
  {
  	std::string::size_type position = 0;
 @@ -38,7 +38,7 @@ void utils::text::replace_first( std::string* data, std::string from, std::strin  	}
  }
 -void utils::text::replace_all( std::string* data, std::string from, std::string to )
 +void utils::text::replace_all( std::string* data, const std::string &from, const std::string &to )
  {
  	std::string::size_type position = 0;
 @@ -49,7 +49,7 @@ void utils::text::replace_all( std::string* data, std::string from, std::string  	}
  }
 -void utils::text::treplace_all(std::tstring* data, std::tstring from, std::tstring to)
 +void utils::text::treplace_all(std::tstring* data, const std::tstring &from, const std::tstring &to)
  {
  	std::tstring::size_type position = 0;
 @@ -72,7 +72,7 @@ std::string utils::text::special_expressions_decode( std::string data )  }
 -std::string utils::text::slashu_to_utf8( std::string data )
 +std::string utils::text::slashu_to_utf8(const std::string &data )
  {
  	std::string new_string = "";
 @@ -108,7 +108,7 @@ std::string utils::text::slashu_to_utf8( std::string data )  	return new_string;
  }
 -std::string utils::text::trim( std::string data )
 +std::string utils::text::trim(const std::string &data )
  {
  	std::string spaces = " \t\r\n";
  	std::string::size_type begin = data.find_first_not_of( spaces );
 @@ -117,7 +117,7 @@ std::string utils::text::trim( std::string data )  	return (begin != std::string::npos) ? data.substr( begin, end - begin ) : "";
  }
 -int utils::debug::log(std::string file_name, std::string text)
 +int utils::debug::log(const std::string &file_name, const std::string &text)
  {
  	char szFile[MAX_PATH];
  	GetModuleFileNameA(g_hInstance, szFile, SIZEOF(szFile));
 diff --git a/protocols/Omegle/src/utils.h b/protocols/Omegle/src/utils.h index dc22deffbb..420f736ff8 100644 --- a/protocols/Omegle/src/utils.h +++ b/protocols/Omegle/src/utils.h @@ -31,17 +31,17 @@ namespace utils  	namespace text
  	{
 -		void replace_first( std::string* data, std::string from, std::string to );
 -		void replace_all( std::string* data, std::string from, std::string to );
 -		void treplace_all(std::tstring* data, std::tstring from, std::tstring to);
 +		void replace_first( std::string* data, const std::string &from, const std::string &to );
 +		void replace_all( std::string* data, const std::string &from, const std::string &to );
 +		void treplace_all(std::tstring* data, const std::tstring &from, const std::tstring &to);
  		std::string special_expressions_decode( std::string data );
 -		std::string slashu_to_utf8( std::string data );
 -		std::string trim( std::string data );
 +		std::string slashu_to_utf8(const std::string &data );
 +		std::string trim(const std::string &data );
  	};
  	namespace debug
  	{
 -		int log(std::string file_name, std::string text);
 +		int log(const std::string &file_name, const std::string &text);
  	};
  	namespace mem
  | 
