On Mon, Nov 28, 2016 at 08:39:21AM +0100, Markus Armbruster wrote:
> The "{disas, slirp}: " prefix is unusual. Better: "disas, slirp: ".
> But I'd instead split the patch into the slirp part, where you really
> replace stuff, and the disas part, where you merely drop an unused macro
> definition.Thanks, Accepting your first suggestion as just now realized that actually this macro is in use (disas/m68k.c lines 4732 and 4796). (Can't explain how i missed that in v0)
