1. 02 4月, 2013 1 次提交
    • J
      Merge branch 'da/downcase-u-in-usage' into maint · 41e603af
      Junio C Hamano 提交于
      * da/downcase-u-in-usage:
        contrib/mw-to-git/t/install-wiki.sh: use a lowercase "usage:" string
        contrib/examples/git-remote.perl: use a lowercase "usage:" string
        tests: use a lowercase "usage:" string
        git-svn: use a lowercase "usage:" string
        Documentation/user-manual.txt: use a lowercase "usage:" string
        templates/hooks--update.sample: use a lowercase "usage:" string
        contrib/hooks/setgitperms.perl: use a lowercase "usage:" string
        contrib/examples: use a lowercase "usage:" string
        contrib/fast-import/import-zips.py: use spaces instead of tabs
        contrib/fast-import/import-zips.py: fix broken error message
        contrib/fast-import: use a lowercase "usage:" string
        contrib/credential: use a lowercase "usage:" string
        git-cvsimport: use a lowercase "usage:" string
        git-cvsimport: use a lowercase "usage:" string
        git-cvsexportcommit: use a lowercase "usage:" string
        git-archimport: use a lowercase "usage:" string
        git-merge-one-file: use a lowercase "usage:" string
        git-relink: use a lowercase "usage:" string
        git-svn: use a lowercase "usage:" string
        git-sh-setup: use a lowercase "usage:" string
      41e603af
  2. 30 3月, 2013 2 次提交
  3. 29 3月, 2013 1 次提交
  4. 28 3月, 2013 6 次提交
  5. 27 3月, 2013 15 次提交
  6. 26 3月, 2013 12 次提交
  7. 22 3月, 2013 3 次提交
    • M
      diff.c: diff.renamelimit => diff.renameLimit in message · c9fc4415
      Max Nanasy 提交于
      In the warning message printed when rename or unmodified copy
      detection was skipped due to too many files, change "diff.renamelimit"
      to "diff.renameLimit", in order to make it consistent with git
      documentation, which consistently uses "diff.renameLimit".
      Signed-off-by: NMax Nanasy <max.nanasy@gmail.com>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      c9fc4415
    • J
      wt-status: fix possible use of uninitialized variable · b8527d5f
      Jeff King 提交于
      In wt_status_print_change_data, we accept a change_type flag
      that is meant to be either WT_STATUS_UPDATED or
      WT_STATUS_CHANGED.  We then switch() on this value to set
      the local variable "status" for each case, but do not
      provide a fallback "default" label to the switch statement.
      
      As a result, the compiler realizes that "status" might be
      unset, and complains with a warning. To silence this
      warning, we use the "int status = status" trick.  This is
      correct with the current code, as all callers provide one of
      the two expected change_type flags. However, it's also a
      maintenance trap, as there is nothing to prevent future
      callers from passing another flag, nor to document this
      assumption.
      
      Instead of using the "x = x" hack, let's handle the default
      case in the switch() statement with a die("BUG"). That tells
      the compiler and any readers of the code exactly what the
      function's input assumptions are.
      
      We could also convert the flag to an enum, which would
      provide a compile-time check on the function input. However,
      since these flags are part of a larger enum, that would make
      the code unnecessarily complex (we would have to make a new
      enum with just the two flags, and then convert it to the old
      enum for passing to sub-functions).
      Signed-off-by: NJeff King <peff@peff.net>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      b8527d5f
    • J
      fast-import: clarify "inline" logic in file_change_m · 3aa99df8
      Jeff King 提交于
      When we read a fast-import line like:
      
        M 100644 :1 foo.c
      
      we point the local object_entry variable "oe" to the object
      named by the mark ":1". When the input uses the "inline"
      construct, however, we do not have such an object_entry.
      
      The current code is careful not to access "oe" in the inline
      case, but we can make the assumption even more obvious (and
      catch violations of it) by setting oe to NULL and adding a
      comment. As a bonus, this also squelches an over-zealous gcc
      -Wuninitialized warning, which means we can drop the "oe =
      oe" initialization hack.
      Signed-off-by: NJeff King <peff@peff.net>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      3aa99df8