diff --git a/src/emu/debug/debugcmd.cpp b/src/emu/debug/debugcmd.cpp
index c690dad5bd3..ee5ea1f8b0f 100644
--- a/src/emu/debug/debugcmd.cpp
+++ b/src/emu/debug/debugcmd.cpp
@@ -699,7 +699,7 @@ static void execute_print(running_machine &machine, int ref, int params, const c
/* then print each one */
for (i = 0; i < params; i++)
- debug_console_printf(machine, "%s", core_i64_hex_format(values[i], 0));
+ debug_console_printf(machine, "%X", values[i]);
debug_console_printf(machine, "\n");
}
@@ -1329,7 +1329,7 @@ static void execute_bplist(running_machine &machine, int ref, int params, const
/* loop over the breakpoints */
for (device_debug::breakpoint *bp = device->debug()->breakpoint_first(); bp != nullptr; bp = bp->next())
{
- buffer = string_format("%c%4X @ %s", bp->enabled() ? ' ' : 'D', bp->index(), core_i64_hex_format(bp->address(), device->debug()->logaddrchars()));
+ buffer = string_format("%c%4X @ %*X", bp->enabled() ? ' ' : 'D', bp->index(), device->debug()->logaddrchars(), bp->address());
if (std::string(bp->condition()).compare("1") != 0)
buffer.append(string_format(" if %s", bp->condition()));
if (std::string(bp->action()).compare("") != 0)
@@ -1496,9 +1496,9 @@ static void execute_wplist(running_machine &machine, int ref, int params, const
/* loop over the watchpoints */
for (device_debug::watchpoint *wp = device->debug()->watchpoint_first(spacenum); wp != nullptr; wp = wp->next())
{
- buffer = string_format("%c%4X @ %s-%s %s", wp->enabled() ? ' ' : 'D', wp->index(),
- core_i64_hex_format(wp->space().byte_to_address(wp->address()), wp->space().addrchars()),
- core_i64_hex_format(wp->space().byte_to_address_end(wp->address() + wp->length()) - 1, wp->space().addrchars()),
+ buffer = string_format("%c%4X @ %*X-%*X %s", wp->enabled() ? ' ' : 'D', wp->index(),
+ wp->space().addrchars(), wp->space().byte_to_address(wp->address()),
+ wp->space().addrchars(), wp->space().byte_to_address_end(wp->address() + wp->length()) - 1,
types[wp->type() & 3]);
if (std::string(wp->condition()).compare("1") != 0)
buffer.append(string_format(" if %s", wp->condition()));
@@ -1818,7 +1818,7 @@ static void execute_load(running_machine &machine, int ref, int params, const ch
if ( i == offset)
debug_console_printf(machine, "Length specified too large, load failed\n");
else
- debug_console_printf(machine, "Data loaded successfully to memory : 0x%s to 0x%s\n", core_i64_hex_format(offset,0), core_i64_hex_format(i-1,0));
+ debug_console_printf(machine, "Data loaded successfully to memory : 0x%X to 0x%X\n", offset, i-1);
}
@@ -1867,13 +1867,14 @@ static void execute_dump(running_machine &machine, int ref, int params, const ch
}
/* now write the data out */
+ util::ovectorstream output;
+ output.reserve(200);
for (i = offset; i <= endoffset; i += 16)
{
- char output[200];
- int outdex = 0;
+ output.clear();
/* print the address */
- outdex += sprintf(&output[outdex], "%s: ", core_i64_hex_format((UINT32)space->byte_to_address(i), space->logaddrchars()));
+ util::stream_format(output, "%*X: ", space->logaddrchars(), (UINT32)space->byte_to_address(i));
/* print the bytes */
for (j = 0; j < 16; j += width)
@@ -1884,34 +1885,35 @@ static void execute_dump(running_machine &machine, int ref, int params, const ch
if (debug_cpu_translate(*space, TRANSLATE_READ_DEBUG, &curaddr))
{
UINT64 value = debug_read_memory(*space, i + j, width, TRUE);
- outdex += sprintf(&output[outdex], " %s", core_i64_hex_format(value, width * 2));
+ util::stream_format(output, " %*X", width * 2, value);
}
else
- outdex += sprintf(&output[outdex], " %.*s", (int)width * 2, "****************");
+ util::stream_format(output, " %.*s", width * 2, "****************");
}
else
- outdex += sprintf(&output[outdex], " %*s", (int)width * 2, "");
+ util::stream_format(output, " %*s", width * 2, "");
}
/* print the ASCII */
if (ascii)
{
- outdex += sprintf(&output[outdex], " ");
+ util::stream_format(output, " ");
for (j = 0; j < 16 && (i + j) <= endoffset; j++)
{
offs_t curaddr = i + j;
if (debug_cpu_translate(*space, TRANSLATE_READ_DEBUG, &curaddr))
{
UINT8 byte = debug_read_byte(*space, i + j, TRUE);
- outdex += sprintf(&output[outdex], "%c", (byte >= 32 && byte < 127) ? byte : '.');
+ util::stream_format(output, "%c", (byte >= 32 && byte < 127) ? byte : '.');
}
else
- outdex += sprintf(&output[outdex], " ");
+ util::stream_format(output, " ");
}
}
/* output the result */
- fprintf(f, "%s\n", output);
+ auto const &text = output.vec();
+ fprintf(f, "%.*s\n", int(unsigned(text.size())), &text[0]);
}
/* close the file */
@@ -2292,6 +2294,7 @@ static void execute_cheatlist(running_machine &machine, int ref, int params, con
}
/* write the cheat list */
+ util::ovectorstream output;
for (cheatindex = 0; cheatindex < cheat.cheatmap.size(); cheatindex += 1)
{
if (cheat.cheatmap[cheatindex].state == 1)
@@ -2302,14 +2305,27 @@ static void execute_cheatlist(running_machine &machine, int ref, int params, con
if (params > 0)
{
active_cheat++;
- fprintf(f, " \n", active_cheat, core_i64_hex_format(address, space->logaddrchars()), core_i64_hex_format(value, cheat.width * 2));
- fprintf(f, " \n");
- fprintf(f, " \n\n");
+ output.clear();
+ stream_format(
+ output,
+ " \n"
+ " \n"
+ " \n\n",
+ active_cheat, space->logaddrchars(), address, cheat.width * 2, value,
+ cpu->tag(), spaceletter, sizeletter, space->logaddrchars(), address, cheat.width * 2, cheat_byte_swap(&cheat, cheat.cheatmap[cheatindex].first_value) & sizemask);
+ auto const &text(output.vec());
+ fprintf(f, "%.*s", int(unsigned(text.size())), &text[0]);
}
else
- debug_console_printf(machine, "Address=%s Start=%s Current=%s\n", core_i64_hex_format(address, space->logaddrchars()), core_i64_hex_format(cheat_byte_swap(&cheat, cheat.cheatmap[cheatindex].first_value) & sizemask, cheat.width * 2), core_i64_hex_format(value, cheat.width * 2));
+ {
+ debug_console_printf(
+ machine, "Address=%*X Start=%*X Current=%*X\n",
+ space->logaddrchars(), address,
+ cheat.width * 2, cheat_byte_swap(&cheat, cheat.cheatmap[cheatindex].first_value) & sizemask,
+ cheat.width * 2, value);
+ }
}
}
if (params > 0)
@@ -2436,7 +2452,7 @@ static void execute_find(running_machine &machine, int ref, int params, const ch
if (match)
{
found++;
- debug_console_printf(machine, "Found at %s\n", core_i64_hex_format((UINT32)space->byte_to_address(i), space->addrchars()));
+ debug_console_printf(machine, "Found at %*X\n", space->addrchars(), (UINT32)space->byte_to_address(i));
}
}
@@ -2490,17 +2506,19 @@ static void execute_dasm(running_machine &machine, int ref, int params, const ch
}
/* now write the data out */
+ util::ovectorstream output;
+ output.reserve(512);
for (UINT64 i = 0; i < length; )
{
int pcbyte = space->address_to_byte(offset + i) & space->bytemask();
- char output[512], disasm[200];
+ char disasm[200];
const char *comment;
offs_t tempaddr;
- int outdex = 0;
int numbytes = 0;
+ output.clear();
/* print the address */
- outdex += sprintf(&output[outdex], "%s: ", core_i64_hex_format((UINT32)space->byte_to_address(pcbyte), space->logaddrchars()));
+ stream_format(output, "%*X: ", space->logaddrchars(), (UINT32)space->byte_to_address(pcbyte));
/* make sure we can translate the address */
tempaddr = pcbyte;
@@ -2522,38 +2540,37 @@ static void execute_dasm(running_machine &machine, int ref, int params, const ch
/* print the bytes */
if (bytes)
{
- int startdex = outdex;
+ auto const startdex = output.tellp();
numbytes = space->address_to_byte(numbytes);
for (j = 0; j < numbytes; j += minbytes)
- outdex += sprintf(&output[outdex], "%s ", core_i64_hex_format(debug_read_opcode(*decrypted_space, pcbyte + j, minbytes), minbytes * 2));
- if (outdex - startdex < byteswidth)
- outdex += sprintf(&output[outdex], "%*s", byteswidth - (outdex - startdex), "");
- outdex += sprintf(&output[outdex], " ");
+ stream_format(output, "%*X ", minbytes * 2, debug_read_opcode(*decrypted_space, pcbyte + j, minbytes));
+ if ((output.tellp() - startdex) < byteswidth)
+ stream_format(output, "%*s", byteswidth - (output.tellp() - startdex), "");
+ stream_format(output, " ");
}
/* add the disassembly */
- sprintf(&output[outdex], "%s", disasm);
+ stream_format(output, "%s", disasm);
/* attempt to add the comment */
comment = space->device().debug()->comment_text(tempaddr);
if (comment != nullptr)
{
/* somewhat arbitrary guess as to how long most disassembly lines will be [column 60] */
- if (strlen(output) < 60)
+ if (output.tellp() < 60)
{
/* pad the comment space out to 60 characters and null-terminate */
- for (outdex = (int)strlen(output); outdex < 60; outdex++)
- output[outdex] = ' ' ;
- output[outdex] = 0 ;
+ while (output.tellp() < 60) output.put(' ');
- sprintf(&output[strlen(output)], "// %s", comment) ;
+ stream_format(output, "// %s", comment);
}
else
- sprintf(&output[strlen(output)], "\t// %s", comment) ;
+ stream_format(output, "\t// %s", comment);
}
/* output the result */
- fprintf(f, "%s\n", output);
+ auto const &text(output.vec());
+ fprintf(f, "%.*s\n", int(unsigned(text.size())), &text[0]);
}
/* close the file */
@@ -2684,7 +2701,7 @@ static void execute_history(running_machine &machine, int ref, int params, const
char buffer[200];
debug->disassemble(buffer, pc, opbuf, argbuf);
- debug_console_printf(machine, "%s: %s\n", core_i64_hex_format(pc, space->logaddrchars()), buffer);
+ debug_console_printf(machine, "%*X: %s\n", space->logaddrchars(), pc, buffer);
}
}
@@ -2890,10 +2907,15 @@ static void execute_map(running_machine &machine, int ref, int params, const cha
if (debug_cpu_translate(*space, intention, &taddress))
{
const char *mapname = space->get_handler_string((intention == TRANSLATE_WRITE_DEBUG) ? ROW_WRITE : ROW_READ, taddress);
- debug_console_printf(machine, "%7s: %s logical == %s physical -> %s\n", intnames[intention & 3], core_i64_hex_format(address, space->logaddrchars()), core_i64_hex_format(space->byte_to_address(taddress), space->addrchars()), mapname);
+ debug_console_printf(
+ machine, "%7s: %*X logical == %*X physical -> %s\n",
+ intnames[intention & 3],
+ space->logaddrchars(), address,
+ space->addrchars(), space->byte_to_address(taddress),
+ mapname);
}
else
- debug_console_printf(machine, "%7s: %s logical is unmapped\n", intnames[intention & 3], core_i64_hex_format(address, space->logaddrchars()));
+ debug_console_printf(machine, "%7s: %*X logical is unmapped\n", intnames[intention & 3], space->logaddrchars(), address);
}
}
@@ -2977,7 +2999,7 @@ static void execute_symlist(running_machine &machine, int ref, int params, const
UINT64 value = entry->value();
/* only display "register" type symbols */
- debug_console_printf(machine, "%s = %s", namelist[symnum], core_i64_hex_format(value, 0));
+ debug_console_printf(machine, "%s = %X", namelist[symnum], value);
if (!entry->is_lval())
debug_console_printf(machine, " (read-only)");
debug_console_printf(machine, "\n");
diff --git a/src/emu/debug/dvbpoints.cpp b/src/emu/debug/dvbpoints.cpp
index 1606f596ddf..e0fd474d5c3 100644
--- a/src/emu/debug/dvbpoints.cpp
+++ b/src/emu/debug/dvbpoints.cpp
@@ -12,6 +12,8 @@
#include "debugger.h"
#include "dvbpoints.h"
+#include
+
// Sorting functors for the qsort function
@@ -100,8 +102,8 @@ static const int tableBreaks[] = { 5, 9, 31, 45, 63, 80 };
//-------------------------------------------------
debug_view_breakpoints::debug_view_breakpoints(running_machine &machine, debug_view_osd_update_func osdupdate, void *osdprivate)
- : debug_view(machine, DVT_BREAK_POINTS, osdupdate, osdprivate),
- m_sortType(cIndexAscending)
+ : debug_view(machine, DVT_BREAK_POINTS, osdupdate, osdprivate)
+ , m_sortType(cIndexAscending)
{
// fail if no available sources
enumerate_sources();
@@ -188,9 +190,11 @@ void debug_view_breakpoints::view_click(const int button, const debug_view_xy& p
}
-void debug_view_breakpoints::pad_astring_to_length(std::string& str, int len)
+void debug_view_breakpoints::pad_ostream_to_length(std::ostream& str, int len)
{
- str = string_format("%-*s", std::max(len, 0), str);
+ auto const current = str.tellp();
+ if (current < decltype(current)(len))
+ str << std::setw(current - decltype(current)(len)) << "";
}
@@ -228,41 +232,43 @@ void debug_view_breakpoints::view_update()
m_total.y = 10;
// Draw
- debug_view_char *dest = &m_viewdata[0];
- std::string linebuf;
+ debug_view_char *dest = &m_viewdata[0];
+ util::ovectorstream linebuf;
+ linebuf.reserve(ARRAY_LENGTH(tableBreaks) - 1);
// Header
if (m_visible.y > 0)
{
linebuf.clear();
- linebuf.append("ID");
- if (m_sortType == &cIndexAscending) linebuf.push_back('\\');
- else if (m_sortType == &cIndexDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[0]);
- linebuf.append("En");
- if (m_sortType == &cEnabledAscending) linebuf.push_back('\\');
- else if (m_sortType == &cEnabledDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[1]);
- linebuf.append("CPU");
- if (m_sortType == &cCpuAscending) linebuf.push_back('\\');
- else if (m_sortType == &cCpuDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[2]);
- linebuf.append("Address");
- if (m_sortType == &cAddressAscending) linebuf.push_back('\\');
- else if (m_sortType == &cAddressDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[3]);
- linebuf.append("Condition");
- if (m_sortType == &cConditionAscending) linebuf.push_back('\\');
- else if (m_sortType == &cConditionDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[4]);
- linebuf.append("Action");
- if (m_sortType == &cActionAscending) linebuf.push_back('\\');
- else if (m_sortType == &cActionDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[5]);
+ linebuf << "ID";
+ if (m_sortType == &cIndexAscending) linebuf.put('\\');
+ else if (m_sortType == &cIndexDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[0]);
+ linebuf << "En";
+ if (m_sortType == &cEnabledAscending) linebuf.put('\\');
+ else if (m_sortType == &cEnabledDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[1]);
+ linebuf << "CPU";
+ if (m_sortType == &cCpuAscending) linebuf.put('\\');
+ else if (m_sortType == &cCpuDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[2]);
+ linebuf << "Address";
+ if (m_sortType == &cAddressAscending) linebuf.put('\\');
+ else if (m_sortType == &cAddressDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[3]);
+ linebuf << "Condition";
+ if (m_sortType == &cConditionAscending) linebuf.put('\\');
+ else if (m_sortType == &cConditionDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[4]);
+ linebuf << "Action";
+ if (m_sortType == &cActionAscending) linebuf.put('\\');
+ else if (m_sortType == &cActionDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[5]);
+ auto const &text(linebuf.vec());
for (UINT32 i = m_topleft.x; i < (m_topleft.x + m_visible.x); i++, dest++)
{
- dest->byte = (i < linebuf.length()) ? linebuf[i] : ' ';
+ dest->byte = (i < text.size()) ? text[i] : ' ';
dest->attrib = DCA_ANCILLARY;
}
}
@@ -275,23 +281,25 @@ void debug_view_breakpoints::view_update()
{
device_debug::breakpoint *const bp = m_buffer[bpi];
- linebuf = string_format("%2X", bp->index());
- pad_astring_to_length(linebuf, tableBreaks[0]);
- linebuf.push_back(bp->enabled() ? 'X' : 'O');
- pad_astring_to_length(linebuf, tableBreaks[1]);
- linebuf.append(bp->debugInterface()->device().tag());
- pad_astring_to_length(linebuf, tableBreaks[2]);
- linebuf.append(core_i64_hex_format(bp->address(), bp->debugInterface()->logaddrchars()));
- pad_astring_to_length(linebuf, tableBreaks[3]);
+ linebuf.clear();
+ util::stream_format(linebuf, "%2X", bp->index());
+ pad_ostream_to_length(linebuf, tableBreaks[0]);
+ linebuf.put(bp->enabled() ? 'X' : 'O');
+ pad_ostream_to_length(linebuf, tableBreaks[1]);
+ linebuf << bp->debugInterface()->device().tag();
+ pad_ostream_to_length(linebuf, tableBreaks[2]);
+ util::stream_format(linebuf, "%0*X", bp->debugInterface()->logaddrchars(), bp->address());
+ pad_ostream_to_length(linebuf, tableBreaks[3]);
if (strcmp(bp->condition(), "1"))
- linebuf.append(bp->condition());
- pad_astring_to_length(linebuf, tableBreaks[4]);
- linebuf.append(bp->action());
- pad_astring_to_length(linebuf, tableBreaks[5]);
+ linebuf << bp->condition();
+ pad_ostream_to_length(linebuf, tableBreaks[4]);
+ linebuf << bp->action();
+ pad_ostream_to_length(linebuf, tableBreaks[5]);
+ auto const &text(linebuf.vec());
for (UINT32 i = m_topleft.x; i < (m_topleft.x + m_visible.x); i++, dest++)
{
- dest->byte = (i < linebuf.length()) ? linebuf[i] : ' ';
+ dest->byte = (i < text.size()) ? text[i] : ' ';
dest->attrib = DCA_NORMAL;
// Color disabled breakpoints red
diff --git a/src/emu/debug/dvbpoints.h b/src/emu/debug/dvbpoints.h
index cd9d0b8c22a..7e52e7d4cae 100644
--- a/src/emu/debug/dvbpoints.h
+++ b/src/emu/debug/dvbpoints.h
@@ -42,7 +42,7 @@ protected:
private:
// internal helpers
void enumerate_sources();
- void pad_astring_to_length(std::string& str, int len);
+ void pad_ostream_to_length(std::ostream& str, int len);
void gather_breakpoints();
diff --git a/src/emu/debug/dvwpoints.cpp b/src/emu/debug/dvwpoints.cpp
index 504a1cd19fa..25dda306972 100644
--- a/src/emu/debug/dvwpoints.cpp
+++ b/src/emu/debug/dvwpoints.cpp
@@ -11,6 +11,8 @@
#include "emu.h"
#include "dvwpoints.h"
+#include
+
static int cIndexAscending(const void* a, const void* b)
@@ -211,9 +213,11 @@ void debug_view_watchpoints::view_click(const int button, const debug_view_xy& p
}
-void debug_view_watchpoints::pad_astring_to_length(std::string& str, int len)
+void debug_view_watchpoints::pad_ostream_to_length(std::ostream& str, int len)
{
- str = string_format("%-*s", std::max(len, 0), str);
+ auto const current = str.tellp();
+ if (current < decltype(current)(len))
+ str << std::setw(current - decltype(current)(len)) << "";
}
@@ -254,49 +258,51 @@ void debug_view_watchpoints::view_update()
m_total.y = 10;
// Draw
- debug_view_char *dest = &m_viewdata[0];
- std::string linebuf;
+ debug_view_char *dest = &m_viewdata[0];
+ util::ovectorstream linebuf;
+ linebuf.reserve(ARRAY_LENGTH(tableBreaks) - 1);
// Header
if (m_visible.y > 0)
{
linebuf.clear();
- linebuf.append("ID");
- if (m_sortType == &cIndexAscending) linebuf.push_back('\\');
- else if (m_sortType == &cIndexDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[0]);
- linebuf.append("En");
- if (m_sortType == &cEnabledAscending) linebuf.push_back('\\');
- else if (m_sortType == &cEnabledDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[1]);
- linebuf.append("CPU");
- if (m_sortType == &cCpuAscending) linebuf.push_back('\\');
- else if (m_sortType == &cCpuDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[2]);
- linebuf.append("Space");
- if (m_sortType == &cSpaceAscending) linebuf.push_back('\\');
- else if (m_sortType == &cSpaceDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[3]);
- linebuf.append("Addresses");
- if (m_sortType == &cAddressAscending) linebuf.push_back('\\');
- else if (m_sortType == &cAddressDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[4]);
- linebuf.append("Type");
- if (m_sortType == &cTypeAscending) linebuf.push_back('\\');
- else if (m_sortType == &cTypeDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[5]);
- linebuf.append("Condition");
- if (m_sortType == &cConditionAscending) linebuf.push_back('\\');
- else if (m_sortType == &cConditionDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[6]);
- linebuf.append("Action");
- if (m_sortType == &cActionAscending) linebuf.push_back('\\');
- else if (m_sortType == &cActionDescending) linebuf.push_back('/');
- pad_astring_to_length(linebuf, tableBreaks[7]);
+ linebuf << "ID";
+ if (m_sortType == &cIndexAscending) linebuf.put('\\');
+ else if (m_sortType == &cIndexDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[0]);
+ linebuf << "En";
+ if (m_sortType == &cEnabledAscending) linebuf.put('\\');
+ else if (m_sortType == &cEnabledDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[1]);
+ linebuf << "CPU";
+ if (m_sortType == &cCpuAscending) linebuf.put('\\');
+ else if (m_sortType == &cCpuDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[2]);
+ linebuf << "Space";
+ if (m_sortType == &cSpaceAscending) linebuf.put('\\');
+ else if (m_sortType == &cSpaceDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[3]);
+ linebuf << "Addresses";
+ if (m_sortType == &cAddressAscending) linebuf.put('\\');
+ else if (m_sortType == &cAddressDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[4]);
+ linebuf << "Type";
+ if (m_sortType == &cTypeAscending) linebuf.put('\\');
+ else if (m_sortType == &cTypeDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[5]);
+ linebuf << "Condition";
+ if (m_sortType == &cConditionAscending) linebuf.put('\\');
+ else if (m_sortType == &cConditionDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[6]);
+ linebuf << "Action";
+ if (m_sortType == &cActionAscending) linebuf.put('\\');
+ else if (m_sortType == &cActionDescending) linebuf.put('/');
+ pad_ostream_to_length(linebuf, tableBreaks[7]);
+ auto const &text(linebuf.vec());
for (UINT32 i = m_topleft.x; i < (m_topleft.x + m_visible.x); i++, dest++)
{
- dest->byte = (i < linebuf.length()) ? linebuf[i] : ' ';
+ dest->byte = (i < text.size()) ? text[i] : ' ';
dest->attrib = DCA_ANCILLARY;
}
}
@@ -310,29 +316,31 @@ void debug_view_watchpoints::view_update()
static char const *const types[] = { "unkn ", "read ", "write", "r/w " };
device_debug::watchpoint *const wp = m_buffer[wpi];
- linebuf = string_format("%2X", wp->index());
- pad_astring_to_length(linebuf, tableBreaks[0]);
- linebuf.push_back(wp->enabled() ? 'X' : 'O');
- pad_astring_to_length(linebuf, tableBreaks[1]);
- linebuf.append(wp->debugInterface()->device().tag());
- pad_astring_to_length(linebuf, tableBreaks[2]);
- linebuf.append(wp->space().name());
- pad_astring_to_length(linebuf, tableBreaks[3]);
- linebuf.append(core_i64_hex_format(wp->space().byte_to_address(wp->address()), wp->space().addrchars()));
- linebuf.push_back('-');
- linebuf.append(core_i64_hex_format(wp->space().byte_to_address_end(wp->address() + wp->length()) - 1, wp->space().addrchars()));
- pad_astring_to_length(linebuf, tableBreaks[4]);
- linebuf.append(types[wp->type() & 3]);
- pad_astring_to_length(linebuf, tableBreaks[5]);
+ linebuf.clear();
+ util::stream_format(linebuf, "%2X", wp->index());
+ pad_ostream_to_length(linebuf, tableBreaks[0]);
+ linebuf.put(wp->enabled() ? 'X' : 'O');
+ pad_ostream_to_length(linebuf, tableBreaks[1]);
+ linebuf << wp->debugInterface()->device().tag();
+ pad_ostream_to_length(linebuf, tableBreaks[2]);
+ linebuf << wp->space().name();
+ pad_ostream_to_length(linebuf, tableBreaks[3]);
+ util::stream_format(linebuf, "%0*X", wp->space().addrchars(), wp->space().byte_to_address(wp->address()));
+ linebuf.put('-');
+ util::stream_format(linebuf, "%0*X", wp->space().addrchars(), wp->space().byte_to_address_end(wp->address() + wp->length()) - 1);
+ pad_ostream_to_length(linebuf, tableBreaks[4]);
+ linebuf << types[wp->type() & 3];
+ pad_ostream_to_length(linebuf, tableBreaks[5]);
if (strcmp(wp->condition(), "1"))
- linebuf.append(wp->condition());
- pad_astring_to_length(linebuf, tableBreaks[6]);
- linebuf.append(wp->action());
- pad_astring_to_length(linebuf, tableBreaks[7]);
+ linebuf << wp->condition();
+ pad_ostream_to_length(linebuf, tableBreaks[6]);
+ linebuf << wp->action();
+ pad_ostream_to_length(linebuf, tableBreaks[7]);
+ auto const &text(linebuf.vec());
for (UINT32 i = m_topleft.x; i < (m_topleft.x + m_visible.x); i++, dest++)
{
- dest->byte = (i < linebuf.length()) ? linebuf[i] : ' ';
+ dest->byte = (i < text.size()) ? text[i] : ' ';
dest->attrib = DCA_NORMAL;
// Color disabled watchpoints red
diff --git a/src/emu/debug/dvwpoints.h b/src/emu/debug/dvwpoints.h
index b7d899e7717..544828598d7 100644
--- a/src/emu/debug/dvwpoints.h
+++ b/src/emu/debug/dvwpoints.h
@@ -42,7 +42,7 @@ protected:
private:
// internal helpers
void enumerate_sources();
- void pad_astring_to_length(std::string& str, int len);
+ void pad_ostream_to_length(std::ostream& str, int len);
void gather_watchpoints();
diff --git a/src/lib/util/corestr.h b/src/lib/util/corestr.h
index cfd30d2c653..bd08da1483c 100644
--- a/src/lib/util/corestr.h
+++ b/src/lib/util/corestr.h
@@ -64,7 +64,6 @@ int core_strwildcmp(const char *sp1, const char *sp2);
/* I64 printf helper */
char *core_i64_format(UINT64 value, UINT8 mindigits, bool is_octal);
char *core_i64_hex_format(UINT64 value, UINT8 mindigits);
-char *core_i64_oct_format(UINT64 value, UINT8 mindigits);
int strcatvprintf(std::string &str, const char *format, va_list args);
diff --git a/src/lib/util/vecstream.h b/src/lib/util/vecstream.h
index 9c7c5d5b4e0..4427a532b0c 100644
--- a/src/lib/util/vecstream.h
+++ b/src/lib/util/vecstream.h
@@ -322,7 +322,7 @@ public:
void vec(const vector_type &content) { rdbuf()->vec(content); }
void vec(vector_type &&content) { rdbuf()->vec(std::move(content)); }
void clear() { rdbuf()->clear(); }
- void reserve(typename vector_type::size_type size) { rdbuf->reserve(size); }
+ void reserve(typename vector_type::size_type size) { rdbuf()->reserve(size); }
private:
basic_vectorbuf m_rdbuf;
@@ -344,7 +344,7 @@ public:
void vec(const vector_type &content) { rdbuf()->vec(content); }
void vec(vector_type &&content) { rdbuf()->vec(std::move(content)); }
void clear() { rdbuf()->clear(); }
- void reserve(typename vector_type::size_type size) { rdbuf->reserve(size); }
+ void reserve(typename vector_type::size_type size) { rdbuf()->reserve(size); }
private:
basic_vectorbuf m_rdbuf;