{"thread":{"id":"52018","subject":"What's cooking in git.git (Oct 2019, #03; Fri, 11)","startedAt":"2019-10-11T07:42:41Z","lastAt":"2019-10-16T01:21:44Z","messageCount":12,"participants":["Junio C Hamano","Elijah Newren","Johannes Schindelin","Eric Wong","Derrick Stolee"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"383881","messageId":"xmqq8sprhgzc.fsf@gitster-ct.c.googlers.com","threadId":"52018","inReplyTo":null,"subject":"What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-10-11T07:35:35Z","receivedAt":"2019-10-11T07:42:41Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking.  Commits prefixed with\n'-' are only in 'pu' (proposed updates) while commits prefixed with\n'+' are in 'next'.  The ones marked with '.' do not appear in any of\nthe integration branches, but I am still holding onto them.\n\nWe may want to merge down fixes accumulated on the 'master' front\nto 'maint', to prepare for v2.23.1 soonish.  I'll look into it over\nthe weekend.\n\nYou can find the changes described here in the integration branches\nof the repositories listed at\n\n    http://git-blame.blogspot.com/p/git-public-repositories.html\n\n--------------------------------------------------\n[Graduated to \"master\"]\n\n* ab/pcre-jit-fixes (2019-08-19) 18 commits\n  (merged to 'next' on 2019-10-04 at 2d55f2b470)\n + grep: under --debug, show whether PCRE JIT is enabled\n + grep: do not enter PCRE2_UTF mode on fixed matching\n + grep: stess test PCRE v2 on invalid UTF-8 data\n + grep: create a \"is_fixed\" member in \"grep_pat\"\n + grep: consistently use \"p->fixed\" in compile_regexp()\n + grep: stop using a custom JIT stack with PCRE v1\n + grep: stop \"using\" a custom JIT stack with PCRE v2\n + grep: remove overly paranoid BUG(...) code\n + grep: use PCRE v2 for optimized fixed-string search\n + grep: remove the kwset optimization\n + grep: drop support for \\0 in --fixed-strings <pattern>\n + grep: make the behavior for NUL-byte in patterns sane\n + grep tests: move binary pattern tests into their own file\n + grep tests: move \"grep binary\" alongside the rest\n + grep: inline the return value of a function call used only once\n + t4210: skip more command-line encoding tests on MinGW\n + grep: don't use PCRE2?_UTF8 with \"log --encoding=<non-utf8>\"\n + log tests: test regex backends in \"--encode=<enc>\" tests\n (this branch is used by cb/pcre1-cleanup.)\n\n A few simplification and bugfixes to PCRE interface.\n\n\n* ah/cleanups (2019-10-03) 4 commits\n  (merged to 'next' on 2019-10-04 at 1345f09afb)\n + git_mkstemps_mode(): replace magic numbers with computed value\n + wrapper: use a loop instead of repetitive statements\n + diffcore-break: use a goto instead of a redundant if statement\n + commit-graph: remove a duplicate assignment\n\n Miscellaneous code clean-ups.\n\n\n* am/visual-studio-config-fix (2019-09-28) 1 commit\n  (merged to 'next' on 2019-10-04 at 135d93143b)\n + contrib/buildsystems: fix Visual Studio Debug configuration\n\n Dev support.\n\n\n* as/shallow-slab-use-fix (2019-10-02) 1 commit\n  (merged to 'next' on 2019-10-04 at f3a22d2b18)\n + shallow.c: don't free unallocated slabs\n\n Correct code that tried to reference all entries in a sparse array\n of pointers by mistake.\n\n\n* bc/object-id-part17 (2019-08-19) 26 commits\n  (merged to 'next' on 2019-10-04 at b0460b0db2)\n + midx: switch to using the_hash_algo\n + builtin/show-index: replace sha1_to_hex\n + rerere: replace sha1_to_hex\n + builtin/receive-pack: replace sha1_to_hex\n + builtin/index-pack: replace sha1_to_hex\n + packfile: replace sha1_to_hex\n + wt-status: convert struct wt_status to object_id\n + cache: remove null_sha1\n + builtin/worktree: switch null_sha1 to null_oid\n + builtin/repack: write object IDs of the proper length\n + pack-write: use hash_to_hex when writing checksums\n + sequencer: convert to use the_hash_algo\n + bisect: switch to using the_hash_algo\n + sha1-lookup: switch hard-coded constants to the_hash_algo\n + config: use the_hash_algo in abbrev comparison\n + combine-diff: replace GIT_SHA1_HEXSZ with the_hash_algo\n + bundle: switch to use the_hash_algo\n + connected: switch GIT_SHA1_HEXSZ to the_hash_algo\n + show-index: switch hard-coded constants to the_hash_algo\n + blame: remove needless comparison with GIT_SHA1_HEXSZ\n + builtin/rev-parse: switch to use the_hash_algo\n + builtin/blame: switch uses of GIT_SHA1_HEXSZ to the_hash_algo\n + builtin/receive-pack: switch to use the_hash_algo\n + fetch-pack: use parse_oid_hex\n + patch-id: convert to use the_hash_algo\n + builtin/replace: make hash size independent\n\n Preparation for SHA-256 upgrade continues.\n\n\n* cb/pcre1-cleanup (2019-08-26) 2 commits\n  (merged to 'next' on 2019-10-04 at a2dd896ee8)\n + grep: refactor and simplify PCRE1 support\n + grep: make sure NO_LIBPCRE1_JIT disable JIT in PCRE1\n (this branch uses ab/pcre-jit-fixes.)\n\n PCRE fixes.\n\n\n* dl/rev-list-doc-cleanup (2019-10-06) 1 commit\n  (merged to 'next' on 2019-10-07 at 712594feb1)\n + git-rev-list.txt: prune options in synopsis\n\n Doc update.\n\n\n* en/clean-nested-with-ignored (2019-10-02) 13 commits\n  (merged to 'next' on 2019-10-03 at 969ec06cc7)\n + dir: special case check for the possibility that pathspec is NULL\n  (merged to 'next' on 2019-09-30 at 778cc31eac)\n + clean: fix theoretical path corruption\n + clean: rewrap overly long line\n + clean: avoid removing untracked files in a nested git repository\n + clean: disambiguate the definition of -d\n + git-clean.txt: do not claim we will delete files with -n/--dry-run\n + dir: add commentary explaining match_pathspec_item's return value\n + dir: if our pathspec might match files under a dir, recurse into it\n + dir: make the DO_MATCH_SUBMODULE code reusable for a non-submodule case\n + dir: also check directories for matching pathspecs\n + dir: fix off-by-one error in match_pathspec_item\n + dir: fix typo in comment\n + t7300: add testcases showing failure to clean specified pathspecs\n\n \"git clean\" fixes.\n\n\n* js/diff-rename-force-stable-sort (2019-10-02) 2 commits\n  (merged to 'next' on 2019-10-04 at e02d882510)\n + diffcore_rename(): use a stable sort\n + Move git_sort(), a stable sort, into into libgit.a\n\n The rename detection logic sorts a list of rename source candidates\n by similarity to pick the best candidate, which means that a tie\n between sources with the same similarity is broken by the original\n location in the original candidate list (which is sorted by path).\n Force the sorting by similarity done with a stable sort, which is\n not promised by system supplied qsort(3), to ensure consistent\n results across platforms.\n\n\n* js/mingw-spawn-with-spaces-in-path (2019-10-02) 1 commit\n  (merged to 'next' on 2019-10-04 at 0fdb87dd53)\n + t0061: fix test for argv[0] with spaces (MINGW only)\n\n Test fix.\n\n\n* js/range-diff-noprefix (2019-10-03) 1 commit\n  (merged to 'next' on 2019-10-04 at 56cf767bdb)\n + range-diff: internally force `diff.noprefix=true`\n\n \"git range-diff\" segfaulted when diff.noprefix configuration was\n used, as it blindly expected the patch it internally generates to\n have the standard a/ and b/ prefixes.  The command now forces the\n internal patch to be built without any prefix, not to be affected\n by any end-user configuration.\n\n\n* js/stash-apply-in-secondary-worktree (2019-10-06) 1 commit\n  (merged to 'next' on 2019-10-07 at b3474c9c3d)\n + stash apply: report status correctly even in a worktree's subdirectory\n\n \"git stash apply\" in a subdirectory of a secondary worktree failed\n to access the worktree correctly, which has been corrected.\n\n\n* kt/add-i-progress (2019-10-04) 1 commit\n  (merged to 'next' on 2019-10-07 at 00cf8fe076)\n + add -i: show progress counter in the prompt\n\n \"git add -i\" has been taught to show the total number of hunks and\n the hunks that has been processed so far when showing prompts.\n\n\n* pm/p4-auto-delete-named-temporary (2019-10-06) 1 commit\n  (merged to 'next' on 2019-10-07 at 4f45be70f5)\n + git-p4: auto-delete named temporary file\n\n\n* pw/rebase-i-show-HEAD-to-reword (2019-08-19) 3 commits\n  (merged to 'next' on 2019-10-04 at ab3d7eeb72)\n + sequencer: simplify root commit creation\n + rebase -i: check for updated todo after squash and reword\n + rebase -i: always update HEAD before rewording\n (this branch is used by ra/rebase-i-more-options.)\n\n \"git rebase -i\" showed a wrong HEAD while \"reword\" open the editor.\n\n\n* py/git-gui-has-maintainer (2019-10-06) 1 commit\n  (merged to 'next' on 2019-10-07 at 0945190c4c)\n + Documentation: update the location of the git-gui repo\n\n Doc update.\n\n\n* rs/convert-fix-utf-without-dash (2019-10-06) 1 commit\n  (merged to 'next' on 2019-10-07 at 9d0e27b5c3)\n + convert: fix handling of dashless UTF prefix in validate_encoding()\n\n The code to skip \"UTF\" and \"UTF-\" prefix, when computing an advice\n message, did not work correctly when the prefix was \"UTF\", which\n has been fixed.\n\n\n* rs/dedup-includes (2019-10-04) 1 commit\n  (merged to 'next' on 2019-10-07 at 5a4fc44657)\n + treewide: remove duplicate #include directives\n\n Code cleanup.\n\n\n* rs/test-remove-useless-debugging-cat (2019-10-07) 1 commit\n  (merged to 'next' on 2019-10-07 at 6d8cb22a4f)\n + tests: remove \"cat foo\" before \"test_i18ngrep bar foo\"\n\n Code cleanup.\n\n\n* sg/name-rev-cutoff-underflow-fix (2019-09-28) 1 commit\n  (merged to 'next' on 2019-10-04 at 25e4436b3d)\n + name-rev: avoid cutoff timestamp underflow\n\n Integer arithmetic fix.\n\n\n* tk/git-svn-trim-author-name (2019-09-28) 1 commit\n  (merged to 'next' on 2019-10-04 at c1c619f7c4)\n + git-svn: trim leading and trailing whitespaces in author name\n\n The author names taken from SVN repositories may have extra leading\n or trailing whitespaces, which are now munged away.\n\n--------------------------------------------------\n[New Topics]\n\n* ds/sparse-cone (2019-10-08) 17 commits\n - sparse-checkout: cone mode should not interact with .gitignore\n - sparse-checkout: write using lockfile\n - sparse-checkout: update working directory in-process\n - sparse-checkout: sanitize for nested folders\n - read-tree: show progress by default\n - unpack-trees: add progress to clear_ce_flags()\n - unpack-trees: hash less in cone mode\n - sparse-checkout: init and set in cone mode\n - sparse-checkout: use hashmaps for cone patterns\n - sparse-checkout: add 'cone' mode\n - trace2: add region in clear_ce_flags\n - sparse-checkout: create 'disable' subcommand\n - sparse-checkout: add '--stdin' option to set subcommand\n - sparse-checkout: 'set' subcommand\n - clone: add --sparse mode\n - sparse-checkout: create 'init' subcommand\n - sparse-checkout: create builtin with 'list' subcommand\n\n Management of sparsely checked-out working tree has gained a\n dedicated \"sparse-checkout\" command.\n\n Seems not to play well with the hashmap updates.\n\n\n* dl/format-patch-doc-test-cleanup (2019-10-09) 1 commit\n  (merged to 'next' on 2019-10-11 at 992da06f37)\n + t4014: treat rev-list output as the expected value\n\n test cleanup.\n\n Will merge to 'master'.\n\n\n* dl/t0000-skip-test-test (2019-10-09) 1 commit\n  (merged to 'next' on 2019-10-11 at b210c1d1fa)\n + t0000: cover GIT_SKIP_TESTS blindspots\n\n test update.\n\n Will merge to 'master'.\n\n\n* gs/sq-quote-buf-pretty (2019-10-08) 1 commit\n  (merged to 'next' on 2019-10-11 at 2f7c00612b)\n + sq_quote_buf_pretty: don't drop empty arguments\n\n Pretty-printed command line formatter (used in e.g. reporting the\n command being run by the tracing API) had a bug that lost an\n argument that is an empty string, which has been corrected.\n\n Will merge to 'master'.\n\n\n* jk/coc (2019-10-10) 2 commits\n  (merged to 'next' on 2019-10-11 at 9765fd29c3)\n + CODE_OF_CONDUCT: mention individual project-leader emails\n + add a Code of Conduct document\n\n Code-of-conduct document.\n\n Will merge to 'master'.\n\n\n* js/xdiffi-comment-updates (2019-10-09) 1 commit\n  (merged to 'next' on 2019-10-11 at 157226cf3a)\n + xdiffi: fix typos and touch up comments\n\n Comment update.\n\n Will merge to 'master'.\n\n\n* jt/push-avoid-lazy-fetch (2019-10-09) 1 commit\n  (merged to 'next' on 2019-10-11 at b5efe1634a)\n + send-pack: never fetch when checking exclusions\n\n Performance hack.\n\n Will merge to 'master'.\n\n\n* tg/range-diff-output-update (2019-10-09) 1 commit\n  (merged to 'next' on 2019-10-11 at fd8c5d7ec5)\n + range-diff: don't segfault with mode-only changes\n\n \"git range-diff\" failed to handle mode-only change, which has been\n corrected.\n\n Will merge to 'master'.\n\n\n* wb/fsmonitor-bitmap-fix (2019-10-10) 1 commit\n - fsmonitor: don't fill bitmap with entries to be removed\n\n A segfault fix.\n\n Perhaps an update to the test script may be worth doing?\n cf. <20191010110732.GJ29845@szeder.dev>\n\n\n* dl/compat-cleanup (2019-10-11) 1 commit\n - pthread.h: manually align parameter lists\n\n Code formatting micronit fix.\n\n Will merge to 'next'.\n\n\n* jj/stash-reset-only-toplevel (2019-10-11) 1 commit\n - stash: avoid recursive hard reset on submodules\n\n\n* js/doc-stash-save (2019-10-11) 1 commit\n - doc(stash): clarify the description of `save`\n\n Doc clarification.\n\n Will merge to 'next'.\n\n\n* ta/t1308-typofix (2019-10-11) 1 commit\n - t1308-config-set: fix a test that has a typo\n\n Test fix.\n\n Will merge to 'next'.\n\n--------------------------------------------------\n[Stalled]\n\n* cb/pcre2-chartables-leakfix (2019-08-06) 3 commits\n - grep: avoid leak of chartables in PCRE2\n - grep: make PCRE2 aware of custom allocator\n - grep: make PCRE1 aware of custom allocator\n\n WIP (this is v3 which is rather old, where the last message in the\n discussion talks about v6 that has never been sent to the list).\n cf. <CAPUEspjJNSrJQT7xV2fsdp2t5odW5fzzPdDxuar_5x_JPUtf6Q@mail.gmail.com>\n\n\n* es/walken-tutorial (2019-10-11) 2 commits\n - SQUASH???\n - documentation: add tutorial for object walking\n\n A tutorial on object enumeration.\n\n\n\n* dl/format-patch-cover-letter-subject (2019-09-05) 1 commit\n - format-patch: learn --infer-cover-subject option\n\n \"git format-patch --cover-letter\" learned to optionally use the\n first paragraph (typically a single-liner) of branch.*.description\n as the subject of the cover letter.\n\n Reroll with a redesign with less emphasis on \"subject\" coming?\n\n\n* jn/unknown-index-extensions (2018-11-21) 2 commits\n - index: offer advice for unknown index extensions\n - index: do not warn about unrecognized extensions\n\n A bit too alarming warning given when unknown index extensions\n exist is getting revamped.\n\n Expecting a reroll.\n\n\n* jc/format-patch-delay-message-id (2019-04-05) 1 commit\n - format-patch: move message-id and related headers to the end\n\n The location \"git format-patch --thread\" adds the Message-Id:\n header in the series of header fields has been moved down, which\n may help working around a suspected bug in GMail MSA, reported at\n <CAHk-=whP1stFZNAaJiMi5eZ9rj0MRt20Y_yHVczZPH+O01d+sA@mail.gmail.com>\n\n Waiting for feedback to see if it truly helps.\n Needs tests.\n\n\n* jt/fetch-cdn-offload (2019-03-12) 9 commits\n - SQUASH???\n - upload-pack: send part of packfile response as uri\n - fetch-pack: support more than one pack lockfile\n - upload-pack: refactor reading of pack-objects out\n - Documentation: add Packfile URIs design doc\n - Documentation: order protocol v2 sections\n - http-fetch: support fetching packfiles by URL\n - http: improve documentation of http_pack_request\n - http: use --stdin when getting dumb HTTP pack\n\n WIP for allowing a response to \"git fetch\" to instruct the bulk of\n the pack contents to be instead taken from elsewhere (aka CDN).\n\n\n* js/protocol-advertise-multi (2018-12-28) 1 commit\n - protocol: advertise multiple supported versions\n\n The transport layer has been updated so that the protocol version\n used can be negotiated between the parties, by the initiator\n listing the protocol versions it is willing to talk, and the other\n side choosing from one of them.\n\n Expecting a reroll.\n cf. <CANq=j3u-zdb_FvNJGPCmygNMScseav63GhVvBX3NcVS4f7TejA@mail.gmail.com>\n\n\n* mk/use-size-t-in-zlib (2018-10-15) 1 commit\n - zlib.c: use size_t for size\n\n The wrapper to call into zlib followed our long tradition to use\n \"unsigned long\" for sizes of regions in memory, which have been\n updated to use \"size_t\".\n\n--------------------------------------------------\n[Cooking]\n\n* dl/octopus-graph-bug (2019-10-04) 5 commits\n  (merged to 'next' on 2019-10-07 at c6bc2fe4a0)\n + t4214: demonstrate octopus graph coloring failure\n + t4214: explicitly list tags in log\n + t4214: generate expect in their own test cases\n + t4214: use test_merge\n + test-lib: let test_merge() perform octopus merges\n\n \"git log --graph\" for an octopus merge is sometimes colored\n incorrectly, which is demonstrated and documented but not yet\n fixed.\n\n Will merge to 'master'.\n\n\n* js/azure-pipelines-msvc (2019-10-06) 13 commits\n  (merged to 'next' on 2019-10-07 at d5a3604dc6)\n + ci: also build and test with MS Visual Studio on Azure Pipelines\n + ci: really use shallow clones on Azure Pipelines\n + tests: let --immediate and --write-junit-xml play well together\n + test-tool run-command: learn to run (parts of) the testsuite\n + vcxproj: include more generated files\n + vcxproj: only copy `git-remote-http.exe` once it was built\n + msvc: work around a bug in GetEnvironmentVariable()\n + msvc: handle DEVELOPER=1\n + msvc: ignore some libraries when linking\n + compat/win32/path-utils.h: add #include guards\n + winansi: use FLEX_ARRAY to avoid compiler warning\n + msvc: avoid using minus operator on unsigned types\n + push: do not pretend to return `int` from `die_push_simple()`\n\n CI updates.\n\n Will merge to 'master'.\n\n\n* js/trace2-cap-max-output-files (2019-10-05) 4 commits\n  (merged to 'next' on 2019-10-09 at e8a126d880)\n + trace2: write discard message to sentinel files\n + trace2: discard new traces if target directory has too many files\n + docs: clarify trace2 version invariants\n + docs: mention trace2 target-dir mode in git-config\n\n The trace2 output, when sending them to files in a designated\n directory, can populate the directory with too many files; a\n mechanism is introduced to set the maximum number of files and\n discard further logs when the maximum is reached.\n\n Will merge to 'master'.\n\n\n* bc/hash-independent-tests-part-6 (2019-10-06) 15 commits\n - t4048: abstract away SHA-1-specific constants\n - t4045: make hash-size independent\n - t4044: update test to work with SHA-256\n - t4039: abstract away SHA-1-specific constants\n - t4038: abstract away SHA-1 specific constants\n - t4034: abstract away SHA-1-specific constants\n - t4027: make hash-size independent\n - t4015: abstract away SHA-1-specific constants\n - t4011: abstract away SHA-1-specific constants\n - t4010: abstract away SHA-1-specific constants\n - t3429: remove SHA1 annotation\n - t1305: avoid comparing extensions\n - rev-parse: add an --object-format option\n - t/oid-info: add empty tree and empty blob values\n - t/oid-info: allow looking up hash algorithm name\n\n Test updates to prepare for SHA-2 transition continues.\n\n May be rerolled.\n cf. <20191008194012.4s35pbwbh25rblhb@camp.crustytoothpaste.net>\n\n\n* bw/format-patch-o-create-leading-dirs (2019-10-06) 1 commit\n - format-patch: create leading components of output directory\n\n \"git format-patch -o <outdir>\" did an equivalent of \"mkdir <outdir>\"\n not \"mkdir -p <outdir>\", which is being corrected.\n\n Use of adjust shared perm on the leading directory may have\n security implictions.\n\n\n* ag/sequencer-todo-updates (2019-10-08) 5 commits\n - sequencer: directly call pick_commits() from complete_action()\n - rebase: fill `squash_onto' in get_replay_opts()\n - sequencer: move the code writing total_nr on the disk to a new function\n - sequencer: update `done_nr' when skipping commands in a todo list\n - sequencer: update `total_nr' when adding an item to a todo list\n\n Reduce unnecessary reading of state variables back from the disk\n during sequener operation.\n\n Will merge to 'next'.\n\n\n* js/fetch-jobs (2019-10-06) 1 commit\n  (merged to 'next' on 2019-10-07 at e798bac35d)\n + fetch: let --jobs=<n> parallelize --multiple, too\n\n \"git fetch --jobs=<n>\" allowed <n> parallel jobs when fetching\n submodules, but this did not apply to \"git fetch --multiple\" that\n fetches from multiple remote repositories.  It now does.\n\n Will merge to 'master'.\n\n\n* gs/commit-graph-trace-with-cmd (2019-10-02) 1 commit\n  (merged to 'next' on 2019-10-07 at 369df0e5cd)\n + commit-graph: emit trace2 cmd_mode for each sub-command\n\n Dev support.\n\n Will merge to 'master'.\n\n\n* js/trace2-fetch-push (2019-10-08) 3 commits\n  (merged to 'next' on 2019-10-11 at 3a2bb143f8)\n + transport: push codepath can take arbitrary repository\n  (merged to 'next' on 2019-10-04 at 1d63701064)\n + push: add trace2 instrumentation\n + fetch: add trace2 instrumentation\n\n Dev support.\n\n Will merge to 'master'.\n\n\n* mt/threaded-grep-in-object-store (2019-10-02) 11 commits\n - grep: move driver pre-load out of critical section\n - grep: re-enable threads in non-worktree case\n - grep: protect packed_git [re-]initialization\n - grep: allow submodule functions to run in parallel\n - submodule-config: add skip_if_read option to repo_read_gitmodules()\n - grep: replace grep_read_mutex by internal obj read lock\n - object-store: allow threaded access to object reading\n - replace-object: make replace operations thread-safe\n - grep: fix racy calls in grep_objects()\n - grep: fix race conditions at grep_submodule()\n - grep: fix race conditions on userdiff calls\n\n Traditionally, we avoided threaded grep while searching in objects\n (as opposed to files in the working tree) as accesses to the object\n layer is not thread-safe.  This limitation is getting lifted.\n\n\n* am/t0028-utf16-tests (2019-09-28) 2 commits\n  (merged to 'next' on 2019-10-09 at 453900a4e8)\n + t0028: add more tests\n + t0028: fix test for UTF-16-LE-BOM\n\n Test fixes.\n\n Will merge to 'master'.\n\n\n* en/fast-imexport-nested-tags (2019-10-04) 8 commits\n  (merged to 'next' on 2019-10-07 at 3e75779e10)\n + fast-export: handle nested tags\n + t9350: add tests for tags of things other than a commit\n + fast-export: allow user to request tags be marked with --mark-tags\n + fast-export: add support for --import-marks-if-exists\n + fast-import: add support for new 'alias' command\n + fast-import: allow tags to be identified by mark labels\n + fast-import: fix handling of deleted tags\n + fast-export: fix exporting a tag and nothing else\n\n Updates to fast-import/export.\n\n Will merge to 'master'.\n\n\n* jk/packfile-reuse-cleanup (2019-09-13) 10 commits\n  (merged to 'next' on 2019-09-30 at dc60b31833)\n + pack-objects: improve partial packfile reuse\n + builtin/pack-objects: introduce obj_is_packed()\n + pack-objects: introduce pack.allowPackReuse\n + csum-file: introduce hashfile_total()\n + pack-bitmap: introduce bitmap_walk_contains()\n + pack-bitmap: don't rely on bitmap_git->reuse_objects\n + ewah/bitmap: always allocate 2 more words\n + ewah/bitmap: introduce bitmap_word_alloc()\n + packfile: expose get_delta_base()\n + builtin/pack-objects: report reused packfile objects\n\n The way \"git pack-objects\" reuses objects stored in existing pack\n to generate its result has been improved.\n\n On hold until we give it a more thorough review.\n cf. <20191002155721.GD6116@sigill.intra.peff.net>\n\n\n* ew/hashmap (2019-10-07) 20 commits\n  (merged to 'next' on 2019-10-09 at 9b75b4ce73)\n + hashmap_entry: remove first member requirement from docs\n + hashmap: remove type arg from hashmap_{get,put,remove}_entry\n + OFFSETOF_VAR macro to simplify hashmap iterators\n + hashmap: introduce hashmap_free_entries\n + hashmap: hashmap_{put,remove} return hashmap_entry *\n + hashmap: use *_entry APIs for iteration\n + hashmap_cmp_fn takes hashmap_entry params\n + hashmap_get{,_from_hash} return \"struct hashmap_entry *\"\n + hashmap: use *_entry APIs to wrap container_of\n + hashmap_get_next returns \"struct hashmap_entry *\"\n + introduce container_of macro\n + hashmap_put takes \"struct hashmap_entry *\"\n + hashmap_remove takes \"const struct hashmap_entry *\"\n + hashmap_get takes \"const struct hashmap_entry *\"\n + hashmap_add takes \"struct hashmap_entry *\"\n + hashmap_get_next takes \"const struct hashmap_entry *\"\n + hashmap_entry_init takes \"struct hashmap_entry *\"\n + packfile: use hashmap_entry in delta_base_cache_entry\n + coccicheck: detect hashmap_entry.hash assignment\n + diff: use hashmap_entry_init on moved_entry.ent\n\n Code clean-up of the hashmap API, both users and implementation.\n\n Will merge to 'master'.\n\n\n* js/builtin-add-i (2019-08-30) 11 commits\n . built-in add -i: implement the `help` command\n . built-in add -i: use color in the main loop\n . built-in add -i: support `?` (prompt help)\n . built-in add -i: show unique prefixes of the commands\n . Add a function to determine unique prefixes for a list of strings\n . built-in add -i: implement the main loop\n . built-in add -i: color the header in the `status` command\n . built-in add -i: refresh the index before running `status`\n . built-in add -i: implement the `status` command\n . diff: export diffstat interface\n . Start to implement a built-in version of `git add --interactive`\n\n The beginning of rewriting \"git add -i\" in C.\n\n On hold, waiting for tg/stash-refresh-index to work well with this.\n\n\n* en/merge-recursive-cleanup (2019-10-08) 26 commits\n  (merged to 'next' on 2019-10-09 at aab23cae2e)\n + merge-recursive: fix the fix to the diff3 common ancestor label\n  (merged to 'next' on 2019-10-03 at 7b5a32726b)\n + merge-recursive: fix the diff3 common ancestor label for virtual commits\n  (merged to 'next' on 2019-09-18 at f52cb08a14)\n + merge-recursive: alphabetize include list\n + merge-recursive: add sanity checks for relevant merge_options\n + merge-recursive: rename MERGE_RECURSIVE_* to MERGE_VARIANT_*\n + merge-recursive: split internal fields into a separate struct\n + merge-recursive: avoid losing output and leaking memory holding that output\n + merge-recursive: comment and reorder the merge_options fields\n + merge-recursive: consolidate unnecessary fields in merge_options\n + merge-recursive: move some definitions around to clean up the header\n + merge-recursive: rename merge_options argument to opt in header\n + merge-recursive: rename 'mrtree' to 'result_tree', for clarity\n + merge-recursive: use common name for ancestors/common/base_list\n + merge-recursive: fix some overly long lines\n + cache-tree: share code between functions writing an index as a tree\n + merge-recursive: don't force external callers to do our logging\n + merge-recursive: remove useless parameter in merge_trees()\n + merge-recursive: exit early if index != head\n + Ensure index matches head before invoking merge machinery, round N\n + merge-recursive: remove another implicit dependency on the_repository\n + merge-recursive: future-proof update_file_flags() against memory leaks\n + merge-recursive: introduce an enum for detect_directory_renames values\n + merge-recursive: provide a better label for diff3 common ancestor\n + merge-recursive: enforce opt->ancestor != NULL when calling merge_trees()\n + checkout: provide better conflict hunk description with detached HEAD\n + merge-recursive: be consistent with assert\n\n The merge-recursive machiery is one of the most complex parts of\n the system that accumulated cruft over time.  This large series\n cleans up the implementation quite a bit.\n\n Will merge to 'master'.\n\n\n* vn/reset-deleted-ita (2019-07-26) 1 commit\n - reset: unstage empty deleted ita files\n\n \"git reset HEAD [<pathspec>]\" did not reset an empty file that was\n added with the intent-to-add bit.\n\n Expecting a reroll.\n\n\n* ra/rebase-i-more-options (2019-09-09) 6 commits\n - rebase: add --reset-author-date\n - rebase -i: support --ignore-date\n - sequencer: rename amend_author to author_to_rename\n - rebase -i: support --committer-date-is-author-date\n - sequencer: allow callers of read_author_script() to ignore fields\n - rebase -i: add --ignore-whitespace flag\n\n \"git rebase -i\" learned a few options that are known by \"git\n rebase\" proper.\n\n Needs to consider existing GIT_COMMITTER_DATE environment, etc.\n cf. <5adde732-173b-d24d-d23f-bb4d043076d7@gmail.com>\n\n\n* jc/format-patch-noclobber (2019-02-22) 1 commit\n - format-patch: --no-clobber refrains from overwriting output files\n\n \"git format-patch\" used to overwrite an existing patch/cover-letter\n file.  A new \"--no-clobber\" option stops it.\n\n Will discard.\n\n--------------------------------------------------\n[Discarded]\n\n* dl/remote-save-to-push (2018-12-11) 1 commit\n . remote: add --save-to-push option to git remote set-url\n\n \"git remote set-url\" learned a new option that moves existing value\n of the URL field to pushURL field of the remote before replacing\n the URL field with a new value.\n\n Anybody who wants to champion this topic?\n I am personally not yet quite convinced if this is worth pursuing.\n\n\n* js/trace2-json-schema (2019-07-25) 3 commits\n . ci: run trace2 schema validation in the CI suite\n . trace2: add a schema validator for trace2 events\n . trace2: add a JSON schema for trace2 events\n\n The JSON output produced by \"trace2\" subsystem now has JSON schema\n defined on it, to allow us validate the output and catch deviation.\n\n Abandoned, at least for now.\n cf. <20190910182305.GA11186@google.com>\n\n\n* pd/fetch-jobs (2019-08-13) 5 commits\n . fetch: make --jobs control submodules and remotes\n . fetch: add the --submodule-fetch-jobs option\n . fetch: add the fetch.jobs config key\n . fetch: add the \"--fetch-jobs\" option\n . fetch: rename max_children to max_children_for_submodules\n\n \"git fetch --jobs\" is getting taught to also run fetch jobs in\n parallel when fetching from multiple remote repositories.\n\n cf. <nycvar.QRO.7.76.6.1909111359150.5377@tvgsbejvaqbjf.bet>\n\n\n* js/honor-cflags-in-hdr-check (2019-08-26) 1 commit\n  (merged to 'next' on 2019-09-09 at fcd9ee9f1b)\n + hdr-check: make it work on Windows\n\n Build fix to make sure hdr-check is run with the same preprocessor\n macros predefined by the $(MAKE) procedure.\n\n Superseded by dl/honor-cflags-in-hdr-check series.\n"},{"id":"383894","messageId":"CABPp-BE4f5f3HyZu9wOyq599JN-n0EMF08di+2V51uxDMEwuGQ@mail.gmail.com","threadId":"52018","inReplyTo":"xmqq8sprhgzc.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2019-10-11T15:44:50Z","receivedAt":"2019-10-11T15:45:05Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Fri, Oct 11, 2019 at 12:35 AM Junio C Hamano <gitster@pobox.com> wrote:\n> [Cooking]\n\n[...]\n\n> * en/fast-imexport-nested-tags (2019-10-04) 8 commits\n>   (merged to 'next' on 2019-10-07 at 3e75779e10)\n>  + fast-export: handle nested tags\n>  + t9350: add tests for tags of things other than a commit\n>  + fast-export: allow user to request tags be marked with --mark-tags\n>  + fast-export: add support for --import-marks-if-exists\n>  + fast-import: add support for new 'alias' command\n>  + fast-import: allow tags to be identified by mark labels\n>  + fast-import: fix handling of deleted tags\n>  + fast-export: fix exporting a tag and nothing else\n>\n>  Updates to fast-import/export.\n>\n>  Will merge to 'master'.\n\nAny chance this will merge down before 2.24.0?  I'd really like to see\nand use it within filter-repo.\n"},{"id":"383904","messageId":"xmqq4l0fgroj.fsf@gitster-ct.c.googlers.com","threadId":"52018","inReplyTo":"CABPp-BE4f5f3HyZu9wOyq599JN-n0EMF08di+2V51uxDMEwuGQ@mail.gmail.com","subject":"Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-10-11T16:42:04Z","receivedAt":"2019-10-11T16:42:12Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren <newren@gmail.com> writes:\n\n>> * en/fast-imexport-nested-tags (2019-10-04) 8 commits\n>>   (merged to 'next' on 2019-10-07 at 3e75779e10)\n>>  + fast-export: handle nested tags\n>>  ...\n>>  + fast-export: fix exporting a tag and nothing else\n>>\n>>  Updates to fast-import/export.\n>>\n>>  Will merge to 'master'.\n>\n> Any chance this will merge down before 2.24.0?  I'd really like to see\n> and use it within filter-repo.\n\nA few general guidelines I use are that a typical topic spends 1\nweek in 'next' (a trivial one-liner may be there for much shorter\ntime, an involved multi-patch topic that touches the core part of\nthe system may have to spend more), that an involved topic that is\nnot in 'master' by rc0 would not appear in the next release, and\nthat any topic that is not in 'master' by rc1 needs compelling\nreason to be in the next release.  So it is cutting a bit too close\nfor this topic, it seems, but we'll see.\n\n"},{"id":"383930","messageId":"CABPp-BF346QbjCyWP7HSP9Lh7mDRgtwtPC8b4mqNv4znUHoGFw@mail.gmail.com","threadId":"52018","inReplyTo":"xmqq4l0fgroj.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2019-10-11T20:39:32Z","receivedAt":"2019-10-11T20:39:48Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Fri, Oct 11, 2019 at 9:42 AM Junio C Hamano <gitster@pobox.com> wrote:\n>\n> Elijah Newren <newren@gmail.com> writes:\n>\n> >> * en/fast-imexport-nested-tags (2019-10-04) 8 commits\n> >>   (merged to 'next' on 2019-10-07 at 3e75779e10)\n> >>  + fast-export: handle nested tags\n> >>  ...\n> >>  + fast-export: fix exporting a tag and nothing else\n> >>\n> >>  Updates to fast-import/export.\n> >>\n> >>  Will merge to 'master'.\n> >\n> > Any chance this will merge down before 2.24.0?  I'd really like to see\n> > and use it within filter-repo.\n>\n> A few general guidelines I use are that a typical topic spends 1\n> week in 'next' (a trivial one-liner may be there for much shorter\n> time, an involved multi-patch topic that touches the core part of\n> the system may have to spend more), that an involved topic that is\n> not in 'master' by rc0 would not appear in the next release, and\n> that any topic that is not in 'master' by rc1 needs compelling\n> reason to be in the next release.  So it is cutting a bit too close\n> for this topic, it seems, but we'll see.\n\n:-(\n\nDid I shoot myself in the foot by being quick to jump on Rene's couple\nof cosmetic touch-up suggestions he posted over a week after the\nseries was originally posted?  In other words, if I hadn't stopped you\nfrom merging the series down to next to incorporate those\nclean-ups[1], would the story have been different?\n\n[1] https://public-inbox.org/git/CABPp-BHvzyLf=Wwhv45qKdKjiTVwHtsMdfA0hd5Ejf5fmhHyPg@mail.gmail.com/\n"},{"id":"383950","messageId":"xmqqwodag7rd.fsf@gitster-ct.c.googlers.com","threadId":"52018","inReplyTo":"CABPp-BF346QbjCyWP7HSP9Lh7mDRgtwtPC8b4mqNv4znUHoGFw@mail.gmail.com","subject":"Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-10-11T23:52:22Z","receivedAt":"2019-10-11T23:52:27Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren <newren@gmail.com> writes:\n\n> Did I shoot myself in the foot by being quick to jump on Rene's couple\n> of cosmetic touch-up suggestions he posted over a week after the\n> series was originally posted?\n\nIf the suggested updates were simple enough to do and would improve\nthe result sufficiently (which is probably true in this case), it is\npossible that the topic would have been marked \"Expecting a reroll\"\nand stayed out of 'next' longer.  Or I may have missed the suggested\nupdates before merging the known-incomplete series to 'next', and\nthe topic may have got marked \"Expecting a follow-up\" and stayed in\nthat state until the update happened.  Or I may have simply missed\nthe suggestion and allowed you to successfully game the system.  So\nyou have 2/3 chance of delaying the topic's graduation by doing so\n;-).\n\nBut over time in the longer run, those contributors who prioritize\nquality over haste are rewarded with more trust (which results in\nshorter gestation period for their topics in both 'pu' and 'next'),\nwhile those contributors with tendency to sneakily gaming the system\neventually get caught and their contribution has to be looked at with\nfiner toothed comb than usual, making more work on everybody and\ndelaying the whole thing, including but not limited to their topics.\n\nAt least, that is how the system hopefully works.\n\nBy the way, I think I made a mistake in my calendar math.\n\nThis topic was merged to 'next' on the 7th and it is not especially\ntricky; unless I (or somebody else) find glaring issues in it during\nthe final sanity check before merging it to 'master' during the next\nbatch, it would take the normal course to 'master' before the 18th,\non which the rc0 is planned.\n"},{"id":"383956","messageId":"xmqqblumg52o.fsf@gitster-ct.c.googlers.com","threadId":"52018","inReplyTo":"xmqqwodag7rd.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-10-12T00:50:23Z","receivedAt":"2019-10-12T00:50:31Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> By the way, I think I made a mistake in my calendar math.\n>\n> This topic was merged to 'next' on the 7th and it is not especially\n> tricky; unless I (or somebody else) find glaring issues in it during\n> the final sanity check before merging it to 'master' during the next\n> batch, it would take the normal course to 'master' before the 18th,\n> on which the rc0 is planned.\n\nUnless I lose power or net due to typhoon, I plan to examine topics\nthat were merged to 'next' to see which ones should be part of the\nnext batch over the weekend.  This will be among those topics.\n\nThanks.\n\n"},{"id":"383977","messageId":"nycvar.QRO.7.76.6.1910122327250.3272@tvgsbejvaqbjf.bet","threadId":"52018","inReplyTo":"xmqq8sprhgzc.fsf@gitster-ct.c.googlers.com","subject":"ds/sparse-cone, was Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2019-10-12T21:31:23Z","receivedAt":"2019-10-12T21:31:45Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Fri, 11 Oct 2019, Junio C Hamano wrote:\n\n> * ds/sparse-cone (2019-10-08) 17 commits\n>  - sparse-checkout: cone mode should not interact with .gitignore\n>  - sparse-checkout: write using lockfile\n>  - sparse-checkout: update working directory in-process\n>  - sparse-checkout: sanitize for nested folders\n>  - read-tree: show progress by default\n>  - unpack-trees: add progress to clear_ce_flags()\n>  - unpack-trees: hash less in cone mode\n>  - sparse-checkout: init and set in cone mode\n>  - sparse-checkout: use hashmaps for cone patterns\n>  - sparse-checkout: add 'cone' mode\n>  - trace2: add region in clear_ce_flags\n>  - sparse-checkout: create 'disable' subcommand\n>  - sparse-checkout: add '--stdin' option to set subcommand\n>  - sparse-checkout: 'set' subcommand\n>  - clone: add --sparse mode\n>  - sparse-checkout: create 'init' subcommand\n>  - sparse-checkout: create builtin with 'list' subcommand\n>\n>  Management of sparsely checked-out working tree has gained a\n>  dedicated \"sparse-checkout\" command.\n>\n>  Seems not to play well with the hashmap updates.\n\nHrm. I had sent out links to the three fixups needed to make the build\ngreen:\n\nhttps://public-inbox.org/git/nycvar.QRO.7.76.6.1910081055210.46@tvgsbejvaqbjf.bet/\n\nIn particular, the patches to squash were:\n\nhttps://github.com/git-for-windows/git/commit/f74259754971b427a14e6290681e18950824b99d\nhttps://github.com/git-for-windows/git/commit/124c8bc08e974e76ca7d956dc07eb288e71d639e\nhttps://github.com/git-for-windows/git/commit/45948433d1b48ff513fbd37f134c0f1491c78192\n\nFor your convenience:\n\n-- snip --\ndiff --git a/builtin/sparse-checkout.c b/builtin/sparse-checkout.c\nindex 160afb2fd7f..fb21d6f8780 100644\n--- a/builtin/sparse-checkout.c\n+++ b/builtin/sparse-checkout.c\n@@ -229,9 +229,9 @@ static void insert_recursive_pattern(struct pattern_list *pl, struct strbuf *pat\n\n \te->patternlen = path->len;\n \te->pattern = strbuf_detach(path, NULL);\n-\thashmap_entry_init(e, memhash(e->pattern, e->patternlen));\n+\thashmap_entry_init(&e->ent, memhash(e->pattern, e->patternlen));\n\n-\thashmap_add(&pl->recursive_hashmap, e);\n+\thashmap_add(&pl->recursive_hashmap, &e->ent);\n\n \twhile (e->patternlen) {\n \t\tchar *slash = strrchr(e->pattern, '/');\n@@ -245,24 +245,26 @@ static void insert_recursive_pattern(struct pattern_list *pl, struct strbuf *pat\n \t\te = xmalloc(sizeof(struct pattern_entry));\n \t\te->patternlen = newlen;\n \t\te->pattern = xstrndup(oldpattern, newlen);\n-\t\thashmap_entry_init(e, memhash(e->pattern, e->patternlen));\n+\t\thashmap_entry_init(&e->ent, memhash(e->pattern, e->patternlen));\n\n-\t\tif (!hashmap_get(&pl->parent_hashmap, e, NULL))\n-\t\t\thashmap_add(&pl->parent_hashmap, e);\n+\t\tif (!hashmap_get_entry(&pl->parent_hashmap, e, ent, NULL))\n+\t\t\thashmap_add(&pl->parent_hashmap, &e->ent);\n \t}\n }\n\n static void write_cone_to_file(FILE *fp, struct pattern_list *pl)\n {\n \tint i;\n-\tstruct pattern_entry *entry;\n+\tstruct hashmap_entry *e;\n \tstruct hashmap_iter iter;\n \tstruct string_list sl = STRING_LIST_INIT_DUP;\n \tstruct strbuf parent_pattern = STRBUF_INIT;\n\n \thashmap_iter_init(&pl->parent_hashmap, &iter);\n-\twhile ((entry = hashmap_iter_next(&iter))) {\n-\t\tif (hashmap_get(&pl->recursive_hashmap, entry, NULL))\n+\twhile ((e = hashmap_iter_next(&iter))) {\n+\t\tstruct pattern_entry *entry =\n+\t\t\tcontainer_of(e, struct pattern_entry, ent);\n+\t\tif (hashmap_get_entry(&pl->recursive_hashmap, entry, ent, NULL))\n \t\t\tcontinue;\n\n \t\tif (!hashmap_contains_parent(&pl->recursive_hashmap,\n@@ -286,7 +288,9 @@ static void write_cone_to_file(FILE *fp, struct pattern_list *pl)\n \tstring_list_clear(&sl, 0);\n\n \thashmap_iter_init(&pl->recursive_hashmap, &iter);\n-\twhile ((entry = hashmap_iter_next(&iter))) {\n+\twhile ((e = hashmap_iter_next(&iter))) {\n+\t\tstruct pattern_entry *entry =\n+\t\t\tcontainer_of(e, struct pattern_entry, ent);\n \t\tif (!hashmap_contains_parent(&pl->recursive_hashmap,\n \t\t\t\t\t     entry->pattern,\n \t\t\t\t\t     &parent_pattern))\ndiff --git a/dir.c b/dir.c\nindex 0135f9e2180..9efcdc9aacd 100644\n--- a/dir.c\n+++ b/dir.c\n@@ -612,10 +612,13 @@ void parse_path_pattern(const char **pattern,\n }\n\n int pl_hashmap_cmp(const void *unused_cmp_data,\n-\t\t   const void *a, const void *b, const void *key)\n+\t\t   const struct hashmap_entry *a, const struct hashmap_entry *b,\n+\t\t   const void *key)\n {\n-\tconst struct pattern_entry *ee1 = (const struct pattern_entry *)a;\n-\tconst struct pattern_entry *ee2 = (const struct pattern_entry *)b;\n+\tconst struct pattern_entry *ee1 =\n+\t\tcontainer_of(a, struct pattern_entry, ent);\n+\tconst struct pattern_entry *ee2 =\n+\t\tcontainer_of(b, struct pattern_entry, ent);\n\n \tsize_t min_len = ee1->patternlen <= ee2->patternlen\n \t\t\t ? ee1->patternlen\n@@ -660,10 +663,11 @@ static void add_pattern_to_hashsets(struct pattern_list *pl, struct path_pattern\n \t\ttranslated = xmalloc(sizeof(struct pattern_entry));\n \t\ttranslated->pattern = truncated;\n \t\ttranslated->patternlen = given->patternlen - 2;\n-\t\thashmap_entry_init(translated,\n+\t\thashmap_entry_init(&translated->ent,\n \t\t\t\t   memhash(translated->pattern, translated->patternlen));\n\n-\t\tif (!hashmap_get(&pl->recursive_hashmap, translated, NULL)) {\n+\t\tif (!hashmap_get_entry(&pl->recursive_hashmap,\n+\t\t\t\t       translated, ent, NULL)) {\n \t\t\t/* We did not see the \"parent\" included */\n \t\t\twarning(_(\"unrecognized negative pattern: '%s'\"),\n \t\t\t\tgiven->pattern);\n@@ -672,8 +676,8 @@ static void add_pattern_to_hashsets(struct pattern_list *pl, struct path_pattern\n \t\t\tgoto clear_hashmaps;\n \t\t}\n\n-\t\thashmap_add(&pl->parent_hashmap, translated);\n-\t\thashmap_remove(&pl->recursive_hashmap, translated, &data);\n+\t\thashmap_add(&pl->parent_hashmap, &translated->ent);\n+\t\thashmap_remove(&pl->recursive_hashmap, &translated->ent, &data);\n \t\tfree(data);\n \t\treturn;\n \t}\n@@ -688,16 +692,16 @@ static void add_pattern_to_hashsets(struct pattern_list *pl, struct path_pattern\n\n \ttranslated->pattern = xstrdup(given->pattern);\n \ttranslated->patternlen = given->patternlen;\n-\thashmap_entry_init(translated,\n+\thashmap_entry_init(&translated->ent,\n \t\t\t   memhash(translated->pattern, translated->patternlen));\n\n-\thashmap_add(&pl->recursive_hashmap, translated);\n+\thashmap_add(&pl->recursive_hashmap, &translated->ent);\n\n-\tif (hashmap_get(&pl->parent_hashmap, translated, NULL)) {\n+\tif (hashmap_get_entry(&pl->parent_hashmap, translated, ent, NULL)) {\n \t\t/* we already included this at the parent level */\n \t\twarning(_(\"your sparse-checkout file may have issues: pattern '%s' is repeated\"),\n \t\t\tgiven->pattern);\n-\t\thashmap_remove(&pl->parent_hashmap, translated, &data);\n+\t\thashmap_remove(&pl->parent_hashmap, &translated->ent, &data);\n \t\tfree(data);\n \t\tfree(translated);\n \t}\n@@ -706,8 +710,8 @@ static void add_pattern_to_hashsets(struct pattern_list *pl, struct path_pattern\n\n clear_hashmaps:\n \twarning(_(\"disabling cone pattern matching\"));\n-\thashmap_free(&pl->parent_hashmap, 1);\n-\thashmap_free(&pl->recursive_hashmap, 1);\n+\thashmap_free(&pl->parent_hashmap);\n+\thashmap_free(&pl->recursive_hashmap);\n \tpl->use_cone_patterns = 0;\n }\n\n@@ -719,8 +723,8 @@ static int hashmap_contains_path(struct hashmap *map,\n \t/* Check straight mapping */\n \tp.pattern = pattern->buf;\n \tp.patternlen = pattern->len;\n-\thashmap_entry_init(&p, memhash(p.pattern, p.patternlen));\n-\treturn !!hashmap_get(map, &p, NULL);\n+\thashmap_entry_init(&p.ent, memhash(p.pattern, p.patternlen));\n+\treturn !!hashmap_get_entry(map, &p, ent, NULL);\n }\n\n int hashmap_contains_parent(struct hashmap *map,\ndiff --git a/dir.h b/dir.h\nindex 09b7c4c44be..81d9f1b0a4e 100644\n--- a/dir.h\n+++ b/dir.h\n@@ -301,7 +301,8 @@ int is_excluded(struct dir_struct *dir,\n \t\tconst char *name, int *dtype);\n\n int pl_hashmap_cmp(const void *unused_cmp_data,\n-\t\t   const void *a, const void *b, const void *key);\n+\t\t   const struct hashmap_entry *a, const struct hashmap_entry *b,\n+\t\t   const void *key);\n int hashmap_contains_parent(struct hashmap *map,\n \t\t\t    const char *path,\n \t\t\t    struct strbuf *buffer);\n-- snap --\n\nWhy not pick it up and squash it into the merge commit?\n\nCiao,\nDscho\n"},{"id":"384054","messageId":"20191015015052.GA19636@dcvr","threadId":"52018","inReplyTo":"nycvar.QRO.7.76.6.1910122327250.3272@tvgsbejvaqbjf.bet","subject":"Re: ds/sparse-cone, was Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Eric Wong","fromEmail":"e@80x24.org","sentAt":"2019-10-15T01:50:52Z","receivedAt":"2019-10-15T01:50:55Z","isPatch":false,"sender":{"key":"e@80x24.org","avatar":null},"body":"Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:\n> On Fri, 11 Oct 2019, Junio C Hamano wrote:\n> > * ds/sparse-cone (2019-10-08) 17 commits\n> >  - sparse-checkout: cone mode should not interact with .gitignore\n> >  - sparse-checkout: write using lockfile\n> >  - sparse-checkout: update working directory in-process\n> >  - sparse-checkout: sanitize for nested folders\n> >  - read-tree: show progress by default\n> >  - unpack-trees: add progress to clear_ce_flags()\n> >  - unpack-trees: hash less in cone mode\n> >  - sparse-checkout: init and set in cone mode\n> >  - sparse-checkout: use hashmaps for cone patterns\n> >  - sparse-checkout: add 'cone' mode\n> >  - trace2: add region in clear_ce_flags\n> >  - sparse-checkout: create 'disable' subcommand\n> >  - sparse-checkout: add '--stdin' option to set subcommand\n> >  - sparse-checkout: 'set' subcommand\n> >  - clone: add --sparse mode\n> >  - sparse-checkout: create 'init' subcommand\n> >  - sparse-checkout: create builtin with 'list' subcommand\n> >\n> >  Management of sparsely checked-out working tree has gained a\n> >  dedicated \"sparse-checkout\" command.\n> >\n> >  Seems not to play well with the hashmap updates.\n> \n> Hrm. I had sent out links to the three fixups needed to make the build\n> green:\n> \n> https://public-inbox.org/git/nycvar.QRO.7.76.6.1910081055210.46@tvgsbejvaqbjf.bet/\n> \n> In particular, the patches to squash were:\n> \n> https://github.com/git-for-windows/git/commit/f74259754971b427a14e6290681e18950824b99d\n> https://github.com/git-for-windows/git/commit/124c8bc08e974e76ca7d956dc07eb288e71d639e\n> https://github.com/git-for-windows/git/commit/45948433d1b48ff513fbd37f134c0f1491c78192\n\n> diff --git a/dir.c b/dir.c\n> index 0135f9e2180..9efcdc9aacd 100644\n> --- a/dir.c\n> +++ b/dir.c\n\n<snip>\n> @@ -706,8 +710,8 @@ static void add_pattern_to_hashsets(struct pattern_list *pl, struct path_pattern\n> \n>  clear_hashmaps:\n>  \twarning(_(\"disabling cone pattern matching\"));\n> -\thashmap_free(&pl->parent_hashmap, 1);\n> -\thashmap_free(&pl->recursive_hashmap, 1);\n> +\thashmap_free(&pl->parent_hashmap);\n> +\thashmap_free(&pl->recursive_hashmap);\n\nI just took a brief look, but that appears to leak memory.\n\n\"hashmap_free(var, 1)\" should be replaced with\n\"hashmap_free_entries(var, struct foo, member)\"\n\nOnly \"hashmap_free(var, 0)\" can become \"hashmap_free(var)\"\n"},{"id":"384061","messageId":"xmqqzhi2bsp8.fsf@gitster-ct.c.googlers.com","threadId":"52018","inReplyTo":"20191015015052.GA19636@dcvr","subject":"Re: ds/sparse-cone, was Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-10-15T03:20:19Z","receivedAt":"2019-10-15T03:20:28Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Eric Wong <e@80x24.org> writes:\n\n> I just took a brief look, but that appears to leak memory.\n>\n> \"hashmap_free(var, 1)\" should be replaced with\n> \"hashmap_free_entries(var, struct foo, member)\"\n>\n> Only \"hashmap_free(var, 0)\" can become \"hashmap_free(var)\"\n\nI deliberately avoided merge-time band-aid fixups on this topic and\new/hashmap exactly because I was sure that I'd introduce a similar\nbugs by doing so myself.  Using evil merges can be a great way to\nhelp multiple topics polished independently at the same time, but\nwhen overused, can hide this kind of gotchas quite easily.\n\nA reroll on top of ew/hashmap would be desirable, now that topic is\nready for 'master'.\n\nThanks.  \n"},{"id":"384068","messageId":"20191015071126.GA908@dcvr","threadId":"52018","inReplyTo":"xmqqzhi2bsp8.fsf@gitster-ct.c.googlers.com","subject":"Re: ds/sparse-cone, was Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Eric Wong","fromEmail":"e@80x24.org","sentAt":"2019-10-15T07:11:26Z","receivedAt":"2019-10-15T07:11:28Z","isPatch":false,"sender":{"key":"e@80x24.org","avatar":null},"body":"Junio C Hamano <gitster@pobox.com> wrote:\n> Eric Wong <e@80x24.org> writes:\n> \n> > I just took a brief look, but that appears to leak memory.\n> >\n> > \"hashmap_free(var, 1)\" should be replaced with\n> > \"hashmap_free_entries(var, struct foo, member)\"\n> >\n> > Only \"hashmap_free(var, 0)\" can become \"hashmap_free(var)\"\n> \n> I deliberately avoided merge-time band-aid fixups on this topic and\n> ew/hashmap exactly because I was sure that I'd introduce a similar\n> bugs by doing so myself.  Using evil merges can be a great way to\n> help multiple topics polished independently at the same time, but\n> when overused, can hide this kind of gotchas quite easily.\n> \n> A reroll on top of ew/hashmap would be desirable, now that topic is\n> ready for 'master'.\n\nJust to be clear, that reroll should come from Stolee (+Cc-ed), right?\nI'll be around to help answer questions, but also pretty busy\nwith other stuff and I think Stolee knows this API pretty well :>\n"},{"id":"384083","messageId":"fdea4b03-3810-1317-7fe3-5100482f7b34@gmail.com","threadId":"52018","inReplyTo":"20191015071126.GA908@dcvr","subject":"Re: ds/sparse-cone, was Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Derrick Stolee","fromEmail":"stolee@gmail.com","sentAt":"2019-10-15T12:54:58Z","receivedAt":"2019-10-15T12:55:03Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 10/15/2019 3:11 AM, Eric Wong wrote:\n> Junio C Hamano <gitster@pobox.com> wrote:\n>> Eric Wong <e@80x24.org> writes:\n>>\n>>> I just took a brief look, but that appears to leak memory.\n>>>\n>>> \"hashmap_free(var, 1)\" should be replaced with\n>>> \"hashmap_free_entries(var, struct foo, member)\"\n>>>\n>>> Only \"hashmap_free(var, 0)\" can become \"hashmap_free(var)\"\n>>\n>> I deliberately avoided merge-time band-aid fixups on this topic and\n>> ew/hashmap exactly because I was sure that I'd introduce a similar\n>> bugs by doing so myself.  Using evil merges can be a great way to\n>> help multiple topics polished independently at the same time, but\n>> when overused, can hide this kind of gotchas quite easily.\n>>\n>> A reroll on top of ew/hashmap would be desirable, now that topic is\n>> ready for 'master'.\n> \n> Just to be clear, that reroll should come from Stolee (+Cc-ed), right?\n> I'll be around to help answer questions, but also pretty busy\n> with other stuff and I think Stolee knows this API pretty well :>\n\nI'm working on the re-roll, yes. I was waiting for ew/hashmap to merge\nwith history that included ds/include-exclude. Now the current 'master'\nhas both, so I can rebase and check everything carefully. v4 should\nhave every commit compile with the new hashmap API.\n\nThanks for pointing out the bugs with the suggested fixups. I'll\nbe careful as I adapt the new API.\n\n-Stolee\n\n"},{"id":"384148","messageId":"xmqqd0exbi3f.fsf@gitster-ct.c.googlers.com","threadId":"52018","inReplyTo":"fdea4b03-3810-1317-7fe3-5100482f7b34@gmail.com","subject":"Re: ds/sparse-cone, was Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-10-16T01:21:40Z","receivedAt":"2019-10-16T01:21:44Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Derrick Stolee <stolee@gmail.com> writes:\n\n> On 10/15/2019 3:11 AM, Eric Wong wrote:\n>> Junio C Hamano <gitster@pobox.com> wrote:\n>>> Eric Wong <e@80x24.org> writes:\n>>>\n>>>> I just took a brief look, but that appears to leak memory.\n>>>>\n>>>> \"hashmap_free(var, 1)\" should be replaced with\n>>>> \"hashmap_free_entries(var, struct foo, member)\"\n>>>>\n>>>> Only \"hashmap_free(var, 0)\" can become \"hashmap_free(var)\"\n>>>\n>>> I deliberately avoided merge-time band-aid fixups on this topic and\n>>> ew/hashmap exactly because I was sure that I'd introduce a similar\n>>> bugs by doing so myself.  Using evil merges can be a great way to\n>>> help multiple topics polished independently at the same time, but\n>>> when overused, can hide this kind of gotchas quite easily.\n>>>\n>>> A reroll on top of ew/hashmap would be desirable, now that topic is\n>>> ready for 'master'.\n>> \n>> Just to be clear, that reroll should come from Stolee (+Cc-ed), right?\n>> I'll be around to help answer questions, but also pretty busy\n>> with other stuff and I think Stolee knows this API pretty well :>\n>\n> I'm working on the re-roll, yes. I was waiting for ew/hashmap to merge\n> with history that included ds/include-exclude. Now the current 'master'\n> has both, so I can rebase and check everything carefully. v4 should\n> have every commit compile with the new hashmap API.\n\nThanks, both.\n"}]}