Hi, David Michael Barr wrote: > On Thu, Jun 7, 2012 at 7:55 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: >> * db/vcs-svn (2012-06-01) 6 commits >> - vcs-svn: drop no-op reset methods >> - vcs-svn: fix signedness warnings >> - vcs-svn: prefer strstr over memmem >> - vcs-svn: prefer constcmp to prefixcmp >> - vcs-svn: simplify cleanup in apply_one_window() >> - vcs-svn: fix clang-analyzer error >> >> I do not know the doneness of this series that came out of the >> blue. Are people involved in vcs-svn happy with this series? > > At the end of this series, vcs-svn is static-analyzer clean when > merged into svn-dump-fast-export. Thanks for the cc. I generally like the code and generally dislike the descriptions which amount to "appease the static analyzer" and don't reflect the thought you actually put into the patches, which sets a bad precedent. I was planning to munge the descriptions, ask you to look it over, and then ask Junio to pull. The signedness patch still leaves me worried: for example, where len is of type (size_t) and delta_len of type (off_t *), - if (len > *delta_len || + if ((off_t) len > *delta_len || changes the meaning for the worse if len is very large on a system (think: 64-bit PC) where size_t and off_t have the same width. It's exploitable. :( But I like the spirit of the patches. The "drop no-op reset methods" patch is unrisky and could go in directly. Jonathan -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html