Re: What's cooking in git.git (Jan 2013, #07; Thu, 17)
On 01/18/2013 01:14 AM, Junio C Hamano wrote: [...] * mh/imap-send-shrinkage (2013-01-15) 14 commits - imap-send.c: simplify logic in lf_to_crlf() - imap-send.c: fold struct store into struct imap_store - imap-send.c: remove unused field imap_store::uidvalidity - imap-send.c: use struct imap_store instead of struct store - imap-send.c: remove unused field imap_store::trashnc - imap-send.c: remove namespace fields from struct imap - imap-send.c: remove struct imap argument to parse_imap_list_l() - imap-send.c: inline parse_imap_list() in parse_list() - imap-send.c: remove some unused fields from struct store - imap-send.c: remove struct message - imap-send.c: remove struct store_conf - iamp-send.c: remove unused struct imap_store_conf - imap-send.c: remove struct msg_data - imap-send.c: remove msg_data::flags, which was always zero Remove a lot of unused code from git imap-send. With a further comment fixup in patch #6, this seems ready for 'next'. Expecting a reroll. I'm confused. It seems like you are referring to the comment improvement suggested by Jonathan Nieder [1] that you have already incorporated [2] into mh/imap-send-shrinkage. If you think there is something that needs rerolling, please explain. Thanks, Michael [1] http://permalink.gmane.org/gmane.comp.version-control.git/213672 [2] http://permalink.gmane.org/gmane.comp.version-control.git/213681 -- Michael Haggerty mhag...@alum.mit.edu http://softwareswirl.blogspot.com/ -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: What's cooking in git.git (Jan 2013, #07; Thu, 17)
Michael Haggerty mhag...@alum.mit.edu writes: On 01/18/2013 01:14 AM, Junio C Hamano wrote: [...] * mh/imap-send-shrinkage (2013-01-15) 14 commits - imap-send.c: simplify logic in lf_to_crlf() - imap-send.c: fold struct store into struct imap_store - imap-send.c: remove unused field imap_store::uidvalidity - imap-send.c: use struct imap_store instead of struct store - imap-send.c: remove unused field imap_store::trashnc - imap-send.c: remove namespace fields from struct imap - imap-send.c: remove struct imap argument to parse_imap_list_l() - imap-send.c: inline parse_imap_list() in parse_list() - imap-send.c: remove some unused fields from struct store - imap-send.c: remove struct message - imap-send.c: remove struct store_conf - iamp-send.c: remove unused struct imap_store_conf - imap-send.c: remove struct msg_data - imap-send.c: remove msg_data::flags, which was always zero Remove a lot of unused code from git imap-send. With a further comment fixup in patch #6, this seems ready for 'next'. Expecting a reroll. I'm confused. It seems like you are referring to the comment improvement suggested by Jonathan Nieder... It was an indication that I just forgot our previous exchange in which I said the remaining issues are so minor I can squash these in. Sorry about that. Will merge to 'next'. Thanks. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
What's cooking in git.git (Jan 2013, #07; Thu, 17)
Here are the topics that have been cooking. Commits prefixed with '-' are only in 'pu' (proposed updates) while commits prefixed with '+' are in 'next'. As usual, this cycle is expected to last for 8 to 10 weeks, with a preview -rc0 sometime in the middle of next month. You can find the changes described here in the integration branches of the repositories listed at http://git-blame.blogspot.com/p/git-public-repositories.html -- [New Topics] * fc/remote-hg-fixup-url (2013-01-15) 1 commit (merged to 'next' on 2013-01-15 at d2acb2d) + remote-hg: store converted URL Update to the Hg remote helper (in contrib/). Will merge to 'master'. * mh/remote-hg-mode-bits-fix (2013-01-15) 1 commit (merged to 'next' on 2013-01-15 at ad57d9f) + remote-hg: fix handling of file perms when pushing Update to the Hg remote helper (in contrib/). Will merge to 'master'. * jc/valgrind-memcmp-bsearch (2013-01-14) 1 commit - ignore memcmp() overreading in bsearch() callback Squelch false positive in valgrind tests Will discard. * mh/imap-send-shrinkage (2013-01-15) 14 commits - imap-send.c: simplify logic in lf_to_crlf() - imap-send.c: fold struct store into struct imap_store - imap-send.c: remove unused field imap_store::uidvalidity - imap-send.c: use struct imap_store instead of struct store - imap-send.c: remove unused field imap_store::trashnc - imap-send.c: remove namespace fields from struct imap - imap-send.c: remove struct imap argument to parse_imap_list_l() - imap-send.c: inline parse_imap_list() in parse_list() - imap-send.c: remove some unused fields from struct store - imap-send.c: remove struct message - imap-send.c: remove struct store_conf - iamp-send.c: remove unused struct imap_store_conf - imap-send.c: remove struct msg_data - imap-send.c: remove msg_data::flags, which was always zero Remove a lot of unused code from git imap-send. With a further comment fixup in patch #6, this seems ready for 'next'. Expecting a reroll. * nd/attr-debug-fix (2013-01-15) 1 commit (merged to 'next' on 2013-01-15 at 8460acf) + attr: make it build with DEBUG_ATTR again Fix debugging support that was broken in earlier change. Will merge to 'master'. * nd/fix-directory-attrs-off-by-one (2013-01-16) 2 commits (merged to 'next' on 2013-01-16 at bd63e61) + attr: avoid calling find_basename() twice per path (merged to 'next' on 2013-01-15 at e0a0129) + attr: fix off-by-one directory component length calculation Fix performance regression introduced by an earlier change to let attributes apply to directories. Will merge to 'master'. * nd/fix-perf-parameters-in-tests (2013-01-15) 1 commit (merged to 'next' on 2013-01-15 at fedbdb9) + test-lib.sh: unfilter GIT_PERF_* Allow GIT_PERF_* environment variables to be passed through the test framework. Will merge to 'master'. * pw/p4-branch-fixes (2013-01-15) 14 commits (merged to 'next' on 2013-01-15 at 1ee379e) + git p4: fix submit when no master branch + git p4 test: keep P4CLIENT changes inside subshells + git p4: fix sync --branch when no master branch + git p4: fail gracefully on sync with no master branch + git p4: rearrange self.initialParent use + git p4: allow short ref names to --branch + git p4 doc: fix branch detection example + git p4: clone --branch should checkout master + git p4: verify expected refs in clone --bare test + git p4: create p4/HEAD on initial clone + git p4: inline listExistingP4GitBranches + git p4: add comments to p4BranchesInGit + git p4: rearrange and simplify hasOrigin handling + git p4: test sync/clone --branch behavior Fix git p4 around branch handling. Will merge to 'master'. * ss/help-htmlpath-config-doc (2013-01-15) 1 commit (merged to 'next' on 2013-01-17 at 99bfae2) + config.txt: Document help.htmlpath config parameter Add missing doc. Will merge to 'master'. * cr/push-force-tag-update (2013-01-16) 1 commit - push: fix refs/tags/ hierarchy cannot be updated without --force Regression fix. Will merge to 'next' and then to 'master' soonish. * jk/suppress-clang-warning (2013-01-16) 1 commit - fix clang -Wunused-value warnings for error functions Will merge to 'next'. * rs/clarify-entry-cmp-sslice (2013-01-16) 1 commit - refs: use strncmp() instead of strlen() and memcmp() Will merge to 'next'. * ch/add-auto-submitted-in-sample-post-receive-email (2013-01-17) 1 commit - Add Auto-Submitted header to post-receive-email Will merge to 'next'. * jc/remove-treesame-parent-in-simplify-merges (2013-01-17) 1 commit - simplify-merges: drop merge from irrelevant side branch The --simplify-merges logic did not cull irrelevant parents from a merge that is otherwise not interesting with respect to the paths we are following. As this touches a fairly core part of the revision traversal infrastructure, it is appreciated to have an extra set of eyes for sanity check. Waiting for