Wireshark mailing list archives
Re: Several small commits vs. one big
From: Thomas Wiens <th.wiens () gmx de>
Date: Wed, 10 Jul 2019 17:49:04 +0200
On 10 Jul 2019, at 08:37, Tomasz Moń wrote:
My general experience with splitting big changes into small commits *after* they are done is rather bad. That is, the artificially splitted changes I have seen were failing to work (sometimes even fail to compile individually). As I find such artificial splitting to be harder than the actual development, I always tend to commit as small pieces of code I see fit for a single commit.
I agree with you. But it's a reverse engineered protocol, and I only got the complete picture of how it works by implementing all new functions, and then step by step renaming the parameter. My question was also related to the ones who do the code review. Also many small commits will result in many compiler test-runs. I don't know how many ressources they take. I've started to split my big commit into smaller ones, I think this will be possible without having a non-working version in between. -- Thomas ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Several small commits vs. one big Thomas Wiens (Jul 09)
- Re: Several small commits vs. one big Tomasz Moń (Jul 09)
- Re: Several small commits vs. one big Thomas Wiens (Jul 10)
- Re: Several small commits vs. one big Tomasz Moń (Jul 09)
