{"push_id":1040715869,"size":4,"distinct_size":0,"ref":"refs/heads/jk/skip-prefix-icase","head":"547954968f49b844978f98701be616403c8e70dd","before":"0af6d4f2b49c3ff9fe797c0d89206b0fa4cc9cdd","commits":[{"sha":"90f7b16b3adc78d4bbabbd426fb69aa78c714f71","author":{"email":"a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","name":"Junio C Hamano"},"message":"Git 2.8\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/90f7b16b3adc78d4bbabbd426fb69aa78c714f71"},{"sha":"da8bf8ddc014c5eefdc365ed1c28f45cea074af8","author":{"email":"696e3fbcf235d40b6ea1ebd61c87cbea79d444e7@peff.net","name":"Jeff King"},"message":"move string functions out of git-compat-util\n\nWe have several inline string helpers (skip_prefix, etc)\ndefined in git-compat-util.h. This is convenient, as it\nmakes them available everywhere, but this is probably not\nthe best place for them. Besides cluttering a file which is\notherwise mostly about compatibility issues, defining inline\nfunctions in git-compat-util.h restricts what they can do.\n\nFor instance, if we wanted to use tolower() in one of the\nfunctions, we run into an ordering problem with our\nredefinitions of tolower(), which are done later in\ngit-compat-util.h.\n\nThis patch moves them to their own header file, strutil.h.\nHowever, it also includes them in strbuf.h (which needs them\nfor its inline strbuf_strip_suffix). This also puts them\nindirectly into cache.h, which means they are effectively\navailable everywhere (just as they were before).","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/da8bf8ddc014c5eefdc365ed1c28f45cea074af8"},{"sha":"d059f13a6cfcf455f4fd4e83bbeed3368af81611","author":{"email":"696e3fbcf235d40b6ea1ebd61c87cbea79d444e7@peff.net","name":"Jeff King"},"message":"log: refactor add_header to drop some magic numbers\n\nWe want to chomp newlines off the end of the \"value\" string.\nBut because it's const, we must keep its length. This leads\nto us having to tweak that length later, to account for\nmoving the pointer forward.\n\nSince we are about to create a copy of it anyway, let's just\nwait and chomp at the end.","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/d059f13a6cfcf455f4fd4e83bbeed3368af81611"},{"sha":"547954968f49b844978f98701be616403c8e70dd","author":{"email":"696e3fbcf235d40b6ea1ebd61c87cbea79d444e7@peff.net","name":"Jeff King"},"message":"strutil: add skip_prefix_icase\n\nSome sites that otherwise would use skip_prefix cannot do\nso, because it has no way to do case-insensitive\ncomparisons. Such sites usually get around this by using\nstrncasecmp, at the cost of having to use magic numbers.\nWe can help them by providing a case-insensitive version of\nskip_prefix.\n\nUnfortunately, we don't share any code with the original\nskip_prefix. Since this is performance-sensitive code, we\nwould not want to introduce an extra \"do we are about case?\"\nconditional into the middle of the loop. We could instead\nuse macros or another technique to generate the\nalmost-identical implementations, but the function simply\nisn't long enough to merit that confusing boilerplate.\n\nTo show off the new function, we convert a simple case in\nlog's add_header function.","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/547954968f49b844978f98701be616403c8e70dd"}]}