Re: [PATCH] fdisk: split fdisk.c

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

 



On 05/28/2009 12:26 PM, Karel Zak wrote:
  mess. Do you really need to split whole fdisk.c into three files by
  *one huge patch*? I don't think so.
You're right, it doesn't have to be one HUGE patch. Although i don't think it will change a lot about the readability and code traceability, it makes sense to split this into individual file separations. However, that way there's going to be a few otherwise unnecessary changes to keep the code compilable.

I'm already splitting the patches. I'd probably do like 1 patch per file rather than 1 file per function, because there's really not THAT much changes in the functions themselves, most of them are simply moved in bulk. Maybe keep the functions that had to be slightly adjusted separate.
--
To unsubscribe from this list: send the line "unsubscribe util-linux-ng" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux