Fix #994: Values wider than 8 bytes show as 0x0 in TTD memory widget (#996)

The original display code had undefined behavior when event.size > 8:
`1ULL << (event.size * 8)` with size=16 shifts by 128 bits, which is
undefined behavior in C++. On many platforms this wraps to shift by 0,
making the mask 0, causing the value to display as 0x0.

TTD always returns 8 bytes in the Value field regardless of access size,
so we need to mask to show the correct number of bytes. The fix:
- For sizes < 8: Apply the mask (safe, shift is < 64 bits)
- For sizes >= 8: Display the full 8-byte value (avoids UB)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: xusheng6

ui/ttdmemorywidget.cpp

@@ -436,9 +436,20 @@ void TTDMemoryQueryWidget::performQuery()
 
 			// Size
 			m_resultsTable->setItem(i, 4, new NumericalTableWidgetItem(QString::number(event.size), event.size));
-			
-			// Value truncated to the number of bytes specified by size
-			QString valueStr = QString("0x%1").arg(event.value & ((1ULL << (event.size * 8)) - 1), 0, 16);
+
+			// Value - TTD always returns 8 bytes, so mask to the actual access size
+			// Note: For sizes > 8 bytes, TTD only provides the lower 8 bytes in the Value field
+			QString valueStr;
+			if (event.size < 8)
+			{
+				uint64_t mask = (1ULL << (event.size * 8)) - 1;
+				valueStr = QString("0x%1").arg(event.value & mask, 0, 16);
+			}
+			else
+			{
+				// For sizes >= 8, display the full 8-byte value
+				valueStr = QString("0x%1").arg(event.value, 0, 16);
+			}
 			m_resultsTable->setItem(i, 5, new NumericalTableWidgetItem(valueStr, event.value));
 
 			// Thread ID