On 6/18/21 1:49 AM, BALATON Zoltan wrote: > On Wed, 16 Jun 2021, Philippe Mathieu-Daudé wrote: >> On 6/16/21 10:01 PM, BALATON Zoltan wrote:
>>> Having a send_recv in one func >>> allowed to avoid if-else in some places like these but if you think it's >>> better without this function at all I can work with that too. I'll have >>> to check if these changes could break anything. At first sight I'm not >>> sure errors are handled as before if recv fails but it was years ago I >>> did the sm501 and ati parts and I forgot how they work so I need to >>> check again. I'll wait for the final version of the series then and test >>> that. >> >> Thanks, that would be great! > > I've tried AmigaOS and MorphOS on sam460ex and mac99 with sm501 and > ati-vga and these still work so I think the patches are OK but I did not > test other changes to other machines so I did not give a tested-by for > the series just some reviewed-by to patches I've verified. (Found a > regression with AROS but that's not related to these changes.) Thank you Zoltan for your testing! Phil.
