On 10/10/2017 03:10 PM, Cornelia Huck wrote: > On Wed, 4 Oct 2017 17:41:44 +0200 > Halil Pasic <pa...@linux.vnet.ibm.com> wrote: > >> Some of ioinst the handlers look very much the same: they basically >> delegate the work to the appropriate css function (doing some always the >> same stuff before and after the call to the appropriate css function). >> Let us create a template and get rid of some code. >> >> Signed-off-by: Halil Pasic <pa...@linux.vnet.ibm.com> >> Suggested-by: Marc Hartmayer <mhart...@linux.vnet.ibm.com> >> --- >> target/s390x/ioinst.c | 59 >> ++++++++++++--------------------------------------- >> 1 file changed, 14 insertions(+), 45 deletions(-) > > Staring at this patch, I'm not sure I like it, although I can't quite > put a finger on *what* I don't like... maybe the whole 'template' > approach. >
Well that's why I was careful to make it a separate patch. I'm also a bit ambivalent, and did not want to include it in this series along the lines can be done later if somebody wants, but Marc convinced me -- kind of. We can just forget about it. Halil