Merge pull request #5283 from djrm/remove_prints

Removed lots of prints

core/print_string.cpp

@@ -44,18 +44,15 @@ void add_print_handler(PrintHandlerList *p_handler) {
 
 void remove_print_handler(PrintHandlerList *p_handler) {
 
-	OS::get_singleton()->print("pre-removing print handler...\n");
 	_global_lock();
 
 	PrintHandlerList *prev = NULL;
 	PrintHandlerList *l = print_handler_list;
 
-	OS::get_singleton()->print("removing print handler...\n");
 	while(l) {
 
 		if (l==p_handler) {
 
-			OS::get_singleton()->print("found\n");
 			if (prev)
 				prev->next=l->next;
 			else

drivers/unix/packet_peer_udp_posix.cpp

@@ -124,7 +124,6 @@ Error PacketPeerUDPPosix::listen(int p_port, int p_recv_buffer_size){
 		close();
 		return ERR_UNAVAILABLE;
 	}
-	printf("UDP Connection listening on port %i  bufsize %i \n", p_port,p_recv_buffer_size);
 	rb.resize(nearest_shift(p_recv_buffer_size));
 	return OK;
 }

drivers/unix/tcp_server_posix.cpp

@@ -67,7 +67,6 @@ void TCPServerPosix::make_default() {
 
 Error TCPServerPosix::listen(uint16_t p_port,const List<String> *p_accepted_hosts) {
 
-	printf("********* listening on port %i\n", p_port);
 	int sockfd;
 	sockfd = socket(AF_INET, SOCK_STREAM, 0);
 	ERR_FAIL_COND_V(sockfd == -1, FAILED);
@@ -80,8 +79,7 @@ Error TCPServerPosix::listen(uint16_t p_port,const List<String> *p_accepted_host
 
 	int reuse=1;
 	if(setsockopt(sockfd, SOL_SOCKET, SO_REUSEADDR, (char *)&reuse, sizeof(reuse)) < 0) {
-
-		printf("REUSEADDR failed!");
+		WARN_PRINT("REUSEADDR failed!")
 	}
 
 	struct sockaddr_in my_addr;
@@ -103,14 +101,11 @@ Error TCPServerPosix::listen(uint16_t p_port,const List<String> *p_accepted_host
 	};
 
 	if (listen_sockfd != -1) {
-
-		printf("FAILED\n");
 		stop();
 	};
 
 	listen_sockfd = sockfd;
 
-	printf("OK! %i\n", listen_sockfd);
 	return OK;
 };
 
@@ -129,7 +124,6 @@ bool TCPServerPosix::is_connection_available() const {
 	ERR_FAIL_COND_V(ret < 0, FAILED);
 
 	if (ret && (pfd.revents & POLLIN)) {
-		printf("has connection!\n");
 		return true;
 	};
 
@@ -164,7 +158,6 @@ Ref<StreamPeerTCP> TCPServerPosix::take_connection() {
 void TCPServerPosix::stop() {
 
 	if (listen_sockfd != -1) {
-		print_line("CLOSING CONNECTION");
 		int ret = close(listen_sockfd);
 		ERR_FAIL_COND(ret!=0);
 	};

modules/gdscript/gd_editor.cpp

@@ -497,11 +497,6 @@ static Ref<Reference> _get_parent_class(GDCompletionContext& context) {
 
 				int base_idx = GDScriptLanguage::get_singleton()->get_global_map()[base];
 				native = GDScriptLanguage::get_singleton()->get_global_array()[base_idx];
-				if (!native.is_valid()) {
-
-					print_line("Global not a class: '"+base+"'");
-
-				}
 				return native;
 			}
 
@@ -2124,10 +2119,8 @@ Error GDScriptLanguage::complete_code(const String& p_code, const String& p_base
 	switch(p.get_completion_type()) {
 
 		case GDParser::COMPLETION_NONE: {
-			print_line("No completion");
 		} break;
 		case GDParser::COMPLETION_BUILT_IN_TYPE_CONSTANT: {
-			print_line("Built in type constant");
 			List<StringName> constants;
 			Variant::get_numeric_constants_for_type(p.get_completion_built_in_constant(),&constants);
 			for(List<StringName>::Element *E=constants.front();E;E=E->next()) {
@@ -2143,7 +2136,6 @@ Error GDScriptLanguage::complete_code(const String& p_code, const String& p_base
 			_find_identifiers(context,p.get_completion_line(),isfunction,options);
 		} break;
 		case GDParser::COMPLETION_PARENT_FUNCTION: {
-			print_line("parent function");
 
 		} break;
 		case GDParser::COMPLETION_METHOD:

tools/editor/editor_node.cpp

@@ -1158,7 +1158,6 @@ void EditorNode::_dialog_action(String p_file) {
 		} break;
 		case FILE_RUN_SCRIPT: {
 
-			print_line("RUN: "+p_file);
 			Ref<Script> scr = ResourceLoader::load(p_file,"Script",true);
 			if (scr.is_null()) {
 				add_io_error("Script Failed to Load:\n"+p_file);
@@ -1302,7 +1301,6 @@ void EditorNode::_dialog_action(String p_file) {
 			ret = unzGoToFirstFile(pkg);
 
 			EditorProgress p("ltask",TTR("Loading Export Templates"),fc);
-			print_line("BEGIN IMPORT");
 
 			fc=0;
 
@@ -1332,7 +1330,6 @@ void EditorNode::_dialog_action(String p_file) {
 				file=file.get_file();
 
 				p.step(TTR("Importing:")+" "+file,fc);
-				print_line("IMPORT "+file);
 
 				FileAccess *f = FileAccess::open(EditorSettings::get_singleton()->get_settings_path()+"/templates/"+file,FileAccess::WRITE);
 
@@ -3551,7 +3548,6 @@ Error EditorNode::load_scene(const String& p_scene, bool p_ignore_broken_deps,bo
 	load_errors->clear();
 	String lpath = Globals::get_singleton()->localize_path(p_scene);
 
-	print_line("LOCAL PATH: "+lpath+" from "+p_scene);
 	if (!lpath.begins_with("res://")) {
 
 		current_option=-1;
@@ -4198,7 +4194,6 @@ void EditorNode::_dock_select_input(const InputEvent& p_input) {
 				dock_slot[dock_popup_selected]->set_current_tab(0);
 			}
 
-			print_line("performing reparent");
 			dock_slot[nrect]->add_child(dock);
 			dock_popup_selected=nrect;
 			dock_slot[nrect]->set_current_tab(dock_slot[nrect]->get_tab_count()-1);
@@ -5172,7 +5167,6 @@ EditorNode::EditorNode() {
 		EditorSettings::create();
 	{
 		int dpi_mode = EditorSettings::get_singleton()->get("global/hidpi_mode");
-		print_line("DPI MODE: "+itos(dpi_mode));
 		if (dpi_mode==0) {
 			editor_set_hidpi( OS::get_singleton()->get_screen_dpi(0) > 150 );
 		} else if (dpi_mode==2) {

tools/editor/editor_settings.cpp

@@ -63,7 +63,6 @@ bool EditorSettings::_set(const StringName& p_name, const Variant& p_value) {
 
 		Array arr=p_value;
 		ERR_FAIL_COND_V(arr.size() && arr.size()&1,true);
-		print_line("shortcuts: "+Variant(arr).get_construct_string());
 		for(int i=0;i<arr.size();i+=2) {
 
 			String name = arr[i];
@@ -399,14 +398,11 @@ String EditorSettings::get_settings_path() const {
 void EditorSettings::setup_language() {
 
 	String lang = get("global/editor_language");
-	print_line("LANG IS "+lang);
 	if (lang=="en")
 		return; //none to do
 
 	for(int i=0;i<translations.size();i++) {
-		print_line("TESTING "+translations[i]->get_locale());
 		if (translations[i]->get_locale()==lang) {
-			print_line("ok translation");
 			TranslationServer::get_singleton()->set_tool_translation(translations[i]);
 			break;
 		}
@@ -711,7 +707,6 @@ void EditorSettings::notify_changes() {
 		sml = OS::get_singleton()->get_main_loop()->cast_to<SceneTree>();
 
 	if (!sml) {
-		print_line("not SML");
 		return;
 	}
 

tools/editor/plugins/script_editor_plugin.cpp

@@ -2201,7 +2201,7 @@ void ScriptEditor::_editor_stop() {
 
 void ScriptEditor::_add_callback(Object *p_obj, const String& p_function, const StringArray& p_args) {
 
-	print_line("add callback! hohoho");
+	//print_line("add callback! hohoho"); kinda sad to remove this
 	ERR_FAIL_COND(!p_obj);
 	Ref<Script> script = p_obj->get_script();
 	ERR_FAIL_COND( !script.is_valid() );
@@ -2243,7 +2243,6 @@ void ScriptEditor::_add_callback(Object *p_obj, const String& p_function, const
 
 void ScriptEditor::_editor_settings_changed() {
 
-	print_line("settings changed");
 	trim_trailing_whitespace_on_save = EditorSettings::get_singleton()->get("text_editor/trim_trailing_whitespace_on_save");
 	float autosave_time = EditorSettings::get_singleton()->get("text_editor/autosave_interval_secs");
 	if (autosave_time>0) {
@@ -2284,7 +2283,6 @@ void ScriptEditor::_editor_settings_changed() {
 
 void ScriptEditor::_autosave_scripts() {
 
-	print_line("autosaving");
 	save_all_scripts();
 }