Re: [PATCH] !fixup Use macros for casting Channel types

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2016-10-11 at 10:00 -0400, Frediano Ziglio wrote:
> > 
> > 
> Beside that... how to better handle all these fixup patches?
> I tend to prefer fixup patches for patch to big patches so
> it's easier to review and people don't have to do weird diff
> commands to understand what's changed from a version to another
> but possibly this causes problems to import back and merge.

I agree. It's nicer and allows you to see exactly what is being fixed.


> 
> Also, should all these fixup be acked or as minor issues
> could be merged if there are no feedback?
> 

I think it would be good to ack the fixup patches unless they were
*very* minor.

Jonathon
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]