added a few more missing \n in logging / cleanups (nw)

src/emu/diimage.c

@@ -1005,9 +1005,9 @@ done:
 		if (!m_init_phase)
 		{
 			if (device().machine().phase() == MACHINE_PHASE_RUNNING)
-				popmessage("Error: Unable to %s image '%s': %s\n", is_create ? "create" : "load", path, error());
+				popmessage("Error: Unable to %s image '%s': %s", is_create ? "create" : "load", path, error());
 			else
-				mame_printf_error("Error: Unable to %s image '%s': %s", is_create ? "create" : "load", path, error());
+				mame_printf_error("Error: Unable to %s image '%s': %s\n", is_create ? "create" : "load", path, error());
 		}
 		clear();
 	}

src/emu/input.c

@@ -867,7 +867,7 @@ input_device::input_device(input_class &_class, int devindex, const char *name,
 			m_joymap.parse(joystick_map_8way);
 		}
 		else if (mapstring != joystick_map_8way)
-			mame_printf_verbose("Input: Default joystick map = %s", m_joymap.to_string(tempstr));
+			mame_printf_verbose("Input: Default joystick map = %s\n", m_joymap.to_string(tempstr));
 	}
 }
 
@@ -2127,7 +2127,7 @@ bool input_manager::set_global_joystick_map(const char *mapstring)
 		return false;
 
 	astring tempstr;
-	mame_printf_verbose("Input: Changing default joystick map = %s", map.to_string(tempstr));
+	mame_printf_verbose("Input: Changing default joystick map = %s\n", map.to_string(tempstr));
 
 	// iterate over joysticks and set the map
 	for (int joynum = 0; joynum <= m_joystick_class.maxindex(); joynum++)

src/emu/ioport.c

@@ -3529,7 +3529,7 @@ time_t ioport_manager::playback_init()
 	mame_printf_info("INP version %d.%d\n", header[0x10], header[0x11]);
 	time_t basetime = header[0x08] | (header[0x09] << 8) | (header[0x0a] << 16) | (header[0x0b] << 24) |
 						((UINT64)header[0x0c] << 32) | ((UINT64)header[0x0d] << 40) | ((UINT64)header[0x0e] << 48) | ((UINT64)header[0x0f] << 56);
-	mame_printf_info("Created %s", ctime(&basetime));
+	mame_printf_info("Created %s\n", ctime(&basetime));
 	mame_printf_info("Recorded using %s\n", header + 0x20);
 
 	// verify the header against the current game

src/emu/render.c

@@ -1639,7 +1639,7 @@ bool render_target::load_layout_file(const char *dirname, const char *filename)
 		if (filename[0] != '<')
 			mame_printf_warning("Error in XML file '%s': %s\n", filename, err.string());
 		else
-			mame_printf_warning("Error in XML string: %s", err.string());
+			mame_printf_warning("Error in XML string: %s\n", err.string());
 		result = false;
 	}
 

src/emu/rendlay.c

@@ -1167,9 +1167,9 @@ void layout_element::component::load_bitmap()
 
 		// log an error
 		if (!m_alphafile[0])
-			mame_printf_warning("Unable to load component bitmap '%s'", m_imagefile[0].cstr());
+			mame_printf_warning("Unable to load component bitmap '%s'\n", m_imagefile[0].cstr());
 		else
-			mame_printf_warning("Unable to load component bitmap '%s'/'%s'", m_imagefile[0].cstr(), m_alphafile[0].cstr());
+			mame_printf_warning("Unable to load component bitmap '%s'/'%s'\n", m_imagefile[0].cstr(), m_alphafile[0].cstr());
 	}
 }
 

src/osd/sdl/netdev_pcap_osx.c

@@ -69,7 +69,7 @@ netdev_pcap::netdev_pcap(const char *name, class device_network_interface *ifdev
 	}
 	if(pcap_set_datalink(m_p, DLT_EN10MB) == -1)
 	{
-		mame_printf_verbose("Unable to set %s to ethernet", name);
+		mame_printf_verbose("Unable to set %s to ethernet\n", name);
 		pcap_close(m_p);
 		m_p = NULL;
 		return;
@@ -89,10 +89,10 @@ void netdev_pcap::set_mac(const char *mac)
 	if(!m_p) return;
 	sprintf(filter, "not ether src %.2X:%.2X:%.2X:%.2X:%.2X:%.2X and (ether dst %.2X:%.2X:%.2X:%.2X:%.2X:%.2X or ether multicast or ether broadcast or ether dst 09:00:07:ff:ff:ff)", (unsigned char)mac[0], (unsigned char)mac[1], (unsigned char)mac[2],(unsigned char)mac[3], (unsigned char)mac[4], (unsigned char)mac[5], (unsigned char)mac[0], (unsigned char)mac[1], (unsigned char)mac[2],(unsigned char)mac[3], (unsigned char)mac[4], (unsigned char)mac[5]);
 	if(pcap_compile(m_p, &fp, filter, 1, 0) == -1) {
-		mame_printf_verbose("Error with pcap_compile");
+		mame_printf_verbose("Error with pcap_compile\n");
 	}
 	if(pcap_setfilter(m_p, &fp) == -1) {
-		mame_printf_verbose("Error with pcap_setfilter");
+		mame_printf_verbose("Error with pcap_setfilter\n");
 	}
 }
 

src/osd/windows/drawd3d.c

@@ -977,7 +977,7 @@ int renderer::device_create_resources()
 				D3DPOOL_DEFAULT, &m_vertexbuf);
 	if (result != D3D_OK)
 	{
-		printf("Error creating vertex buffer (%08X)", (UINT32)result);
+		mame_printf_error("Error creating vertex buffer (%08X)\n", (UINT32)result);
 		return 1;
 	}
 
@@ -986,7 +986,7 @@ int renderer::device_create_resources()
 		d3dintf->post_fx_available) ? D3DFVF_XYZW : D3DFVF_XYZRHW)));
 	if (result != D3D_OK)
 	{
-		mame_printf_error("Error setting vertex format (%08X)", (UINT32)result);
+		mame_printf_error("Error setting vertex format (%08X)\n", (UINT32)result);
 		return 1;
 	}
 
@@ -1174,7 +1174,7 @@ int renderer::device_test_cooperative()
 		// if it didn't work, punt to GDI
 		if (result != D3D_OK)
 		{
-			printf("Unable to reset, result %08x\n", (UINT32)result);
+			mame_printf_error("Unable to reset, result %08x\n", (UINT32)result);
 			return 1;
 		}
 
@@ -1880,7 +1880,7 @@ void renderer::primitive_flush_pending()
 
 		if (vertnum + m_poly[polynum].get_vertcount() > m_numverts)
 		{
-			printf("Error: vertnum (%d) plus poly vertex count (%d) > %d\n", vertnum, m_poly[polynum].get_vertcount(), m_numverts);
+			mame_printf_error("Error: vertnum (%d) plus poly vertex count (%d) > %d\n", vertnum, m_poly[polynum].get_vertcount(), m_numverts);
 			fflush(stdout);
 		}
 
@@ -2102,7 +2102,7 @@ texture_info::texture_info(texture_manager *manager, const render_texinfo* texso
 
 error:
 	d3dintf->post_fx_available = false;
-	printf("Direct3D: Critical warning: A texture failed to allocate. Expect things to get bad quickly.\n");
+	mame_printf_error("Direct3D: Critical warning: A texture failed to allocate. Expect things to get bad quickly.\n");
 	if (m_d3dsurface != NULL)
 		(*d3dintf->surface.release)(m_d3dsurface);
 	if (m_d3dtex != NULL)