Make sure files end with exactly one new line and police this in presubmit.
authoryangguo@chromium.org <yangguo@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Fri, 22 Nov 2013 13:50:39 +0000 (13:50 +0000)
committeryangguo@chromium.org <yangguo@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Fri, 22 Nov 2013 13:50:39 +0000 (13:50 +0000)
commitaa3518a0f37245ebe8f062dce97ee492e2a41652
treefc22a85f61fc9eef79deaf6cd1970c8e6655833d
parentbf473a4a94eeff9dab768a9e5a0d8713427dde32
Make sure files end with exactly one new line and police this in presubmit.

The changes are (excluding presubmit.py) mechanical. I added the following
lines after the check and iterated the presubmit script until all errors
went away:

f = open(name, "w");
if contents.endswith('\n\n'):
  f.write(contents[0:-1])
else:
  f.write(contents + '\n')

R=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/82803005

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@18017 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
390 files changed:
build/all.gyp
src/allocation-tracker.h
src/arraybuffer.js
src/checks.cc
src/harmony-array.js
src/harmony-string.js
src/heap-snapshot-generator-inl.h
src/heap-snapshot-generator.h
src/ia32/simulator-ia32.cc
src/x64/simulator-x64.cc
test/cctest/test-unbound-queue.cc
test/cctest/test-unique.cc
test/intl/collator/sr-sort.js
test/intl/overrides/caching.js
test/message/isvar.js
test/message/paren_in_arg_string.js
test/message/replacement-marker-as-argument.js
test/message/single-function-literal.js
test/mjsunit/arguments-load-across-eval.js
test/mjsunit/arguments-read-and-assignment.js
test/mjsunit/arguments.js
test/mjsunit/array-iteration.js
test/mjsunit/array-non-smi-length.js
test/mjsunit/array-tostring.js
test/mjsunit/bugs/bug-1344252.js
test/mjsunit/bugs/bug-proto.js
test/mjsunit/closures.js
test/mjsunit/compare-character.js
test/mjsunit/compare-nan.js
test/mjsunit/compiler/control-flow-1.js
test/mjsunit/compiler/inline-throw.js
test/mjsunit/compiler/lazy-const-lookup.js
test/mjsunit/compiler/optimized-closures.js
test/mjsunit/compiler/regress-1394.js
test/mjsunit/compiler/regress-3260426.js
test/mjsunit/compiler/regress-closures-with-eval.js
test/mjsunit/compiler/regress-inline-callfunctionstub.js
test/mjsunit/compiler/regress-toint32.js
test/mjsunit/compiler/rotate.js
test/mjsunit/const-declaration.js
test/mjsunit/cyclic-array-to-string.js
test/mjsunit/debug-breakpoints.js
test/mjsunit/debug-constructor.js
test/mjsunit/debug-evaluate-const.js
test/mjsunit/debug-function-scopes.js
test/mjsunit/debug-ignore-breakpoints.js
test/mjsunit/debug-liveedit-3.js
test/mjsunit/debug-liveedit-check-stack.js
test/mjsunit/debug-liveedit-compile-error.js
test/mjsunit/debug-liveedit-diff.js
test/mjsunit/debug-liveedit-utils.js
test/mjsunit/debug-mirror-cache.js
test/mjsunit/debug-setbreakpoint.js
test/mjsunit/debug-stepin-positions.js
test/mjsunit/debug-stepout-scope-part1.js
test/mjsunit/elide-double-hole-check-9.js
test/mjsunit/enumeration-order.js
test/mjsunit/error-accessors.js
test/mjsunit/eval-stack-trace.js
test/mjsunit/extra-arguments.js
test/mjsunit/fast-array-length.js
test/mjsunit/fast-literal.js
test/mjsunit/fun-name.js
test/mjsunit/function.js
test/mjsunit/get-prototype-of.js
test/mjsunit/getter-in-value-prototype.js
test/mjsunit/global-load-from-eval-in-with.js
test/mjsunit/global-load-from-nested-eval.js
test/mjsunit/harmony/array-find.js
test/mjsunit/harmony/array-findindex.js
test/mjsunit/harmony/collections.js
test/mjsunit/harmony/string-repeat.js
test/mjsunit/large-object-literal.js
test/mjsunit/local-load-from-eval.js
test/mjsunit/math-pow.js
test/mjsunit/math-round.js
test/mjsunit/math-sqrt.js
test/mjsunit/mirror-boolean.js
test/mjsunit/mirror-undefined.js
test/mjsunit/mjsunit.js
test/mjsunit/mul-exhaustive-part6.js
test/mjsunit/nans.js
test/mjsunit/never-optimize.js
test/mjsunit/new.js
test/mjsunit/newline-in-string.js
test/mjsunit/number-tostring-func.js
test/mjsunit/number-tostring-small.js
test/mjsunit/number-tostring.js
test/mjsunit/object-literal-conversions.js
test/mjsunit/object-literal-gc.js
test/mjsunit/packed-elements.js
test/mjsunit/property-object-key.js
test/mjsunit/prototype.js
test/mjsunit/regexp-indexof.js
test/mjsunit/regexp-results-cache.js
test/mjsunit/regress/regress-1017.js
test/mjsunit/regress/regress-1039610.js
test/mjsunit/regress/regress-105.js
test/mjsunit/regress/regress-1066899.js
test/mjsunit/regress/regress-1092.js
test/mjsunit/regress/regress-1099.js
test/mjsunit/regress/regress-1112.js
test/mjsunit/regress/regress-1114040.js
test/mjsunit/regress/regress-1117.js
test/mjsunit/regress/regress-1178598.js
test/mjsunit/regress/regress-1181.js
test/mjsunit/regress/regress-1246.js
test/mjsunit/regress/regress-1254366.js
test/mjsunit/regress/regress-131994.js
test/mjsunit/regress/regress-137.js
test/mjsunit/regress/regress-1546.js
test/mjsunit/regress/regress-1591.js
test/mjsunit/regress/regress-1647.js
test/mjsunit/regress/regress-166379.js
test/mjsunit/regress/regress-1748.js
test/mjsunit/regress/regress-1757.js
test/mjsunit/regress/regress-1853.js
test/mjsunit/regress/regress-186.js
test/mjsunit/regress/regress-1919169.js
test/mjsunit/regress/regress-193.js
test/mjsunit/regress/regress-20070207.js
test/mjsunit/regress/regress-2027.js
test/mjsunit/regress/regress-2119.js
test/mjsunit/regress/regress-2170.js
test/mjsunit/regress/regress-2172.js
test/mjsunit/regress/regress-2250.js
test/mjsunit/regress/regress-2285.js
test/mjsunit/regress/regress-2289.js
test/mjsunit/regress/regress-231.js
test/mjsunit/regress/regress-2374.js
test/mjsunit/regress/regress-237617.js
test/mjsunit/regress/regress-2419.js
test/mjsunit/regress/regress-2438.js
test/mjsunit/regress/regress-2444.js
test/mjsunit/regress/regress-246.js
test/mjsunit/regress/regress-2570.js
test/mjsunit/regress/regress-2596.js
test/mjsunit/regress/regress-2624.js
test/mjsunit/regress/regress-2671-1.js
test/mjsunit/regress/regress-2671.js
test/mjsunit/regress/regress-2690.js
test/mjsunit/regress/regress-298269.js
test/mjsunit/regress/regress-2984.js
test/mjsunit/regress/regress-317.js
test/mjsunit/regress/regress-319722-ArrayBuffer.js
test/mjsunit/regress/regress-45469.js
test/mjsunit/regress/regress-483.js
test/mjsunit/regress/regress-588599.js
test/mjsunit/regress/regress-619.js
test/mjsunit/regress/regress-670147.js
test/mjsunit/regress/regress-674753.js
test/mjsunit/regress/regress-675.js
test/mjsunit/regress/regress-678525.js
test/mjsunit/regress/regress-734862.js
test/mjsunit/regress/regress-74.js
test/mjsunit/regress/regress-753.js
test/mjsunit/regress/regress-806473.js
test/mjsunit/regress/regress-85177.js
test/mjsunit/regress/regress-892742.js
test/mjsunit/regress/regress-925537.js
test/mjsunit/regress/regress-94873.js
test/mjsunit/regress/regress-990205.js
test/mjsunit/regress/regress-995.js
test/mjsunit/regress/regress-998565.js
test/mjsunit/regress/regress-add-minus-zero.js
test/mjsunit/regress/regress-array-pop-nonconfigurable.js
test/mjsunit/regress/regress-binop-nosse2.js
test/mjsunit/regress/regress-builtin-array-op.js
test/mjsunit/regress/regress-crbug-158185.js
test/mjsunit/regress/regress-crbug-178790.js
test/mjsunit/regress/regress-crbug-18639.js
test/mjsunit/regress/regress-crbug-222893.js
test/mjsunit/regress/regress-crbug-233737.js
test/mjsunit/regress/regress-crbug-245480.js
test/mjsunit/regress/regress-crbug-259300.js
test/mjsunit/regress/regress-crbug-3184.js
test/mjsunit/regress/regress-frame-details-null-receiver.js
test/mjsunit/regress/regress-map-invalidation-1.js
test/mjsunit/regress/regress-seqstrsetchar-ex1.js
test/mjsunit/regress/regress-seqstrsetchar-ex2.js
test/mjsunit/regress/regress-seqstrsetchar-ex3.js
test/mjsunit/regress/regress-transcendental.js
test/mjsunit/simple-constructor.js
test/mjsunit/sparse-array.js
test/mjsunit/stack-traces-2.js
test/mjsunit/stack-traces-custom-lazy.js
test/mjsunit/store-dictionary.js
test/mjsunit/str-to-num.js
test/mjsunit/strict-equals.js
test/mjsunit/string-natives.js
test/mjsunit/string-replace-gc.js
test/mjsunit/string-search.js
test/mjsunit/string-slices.js
test/mjsunit/to-precision.js
test/mjsunit/tobool.js
test/mjsunit/tools/profile.js
test/mjsunit/touint32.js
test/mjsunit/transcendentals.js
test/mjsunit/try-finally-continue.js
test/mjsunit/unicode-string-to-number.js
test/mjsunit/unicode-test.js
test/mjsunit/with-value.js
test/preparser/non-use-strict-hex-escape.js
test/preparser/non-use-strict-octal-escape.js
test/preparser/non-use-strict-uhex-escape.js
test/preparser/nonstrict-arguments.js
test/preparser/nonstrict-eval.js
test/preparser/nonstrict-with.js
test/preparser/strict-const.js
test/preparser/strict-octal-indirect-regexp.js
test/preparser/strict-octal-number.js
test/preparser/strict-octal-regexp.js
test/preparser/strict-octal-string.js
test/preparser/strict-octal-use-strict-after.js
test/preparser/strict-octal-use-strict-before.js
test/preparser/strict-with.js
test/webkit/arguments-bad-index.js
test/webkit/concat-while-having-a-bad-time.js
test/webkit/dfg-abs-backwards-propagation.js
test/webkit/dfg-add-not-number.js
test/webkit/dfg-arguments-alias-escape.js
test/webkit/dfg-arguments-alias-one-block-overwrite.js
test/webkit/dfg-arguments-alias.js
test/webkit/dfg-arguments-mixed-alias.js
test/webkit/dfg-arguments-out-of-bounds.js
test/webkit/dfg-arith-add-overflow-check-elimination-predicted-but-not-proven-int.js
test/webkit/dfg-arith-add-overflow-check-elimination-tower-of-large-numbers.js
test/webkit/dfg-array-dead.js
test/webkit/dfg-array-length-dead.js
test/webkit/dfg-branch-logical-not-peephole-around-osr-exit.js
test/webkit/dfg-branch-not-fail.js
test/webkit/dfg-call-function-hit-watchpoint.js
test/webkit/dfg-call-method-hit-watchpoint.js
test/webkit/dfg-cfa-prove-put-by-id-simple-when-storing-to-specialized-function.js
test/webkit/dfg-cfg-simplify-eliminate-set-local-type-check-then-branch-not-null-and-decrement.js
test/webkit/dfg-cfg-simplify-eliminate-set-local-type-check-then-branch-not-null.js
test/webkit/dfg-cfg-simplify-eliminate-set-local-type-check-then-typeof.js
test/webkit/dfg-cfg-simplify-phantom-get-local-on-same-block-set-local.js
test/webkit/dfg-compare-final-object-to-final-object-or-other-when-both-proven-final-object.js
test/webkit/dfg-compare-final-object-to-final-object-or-other-when-proven-final-object.js
test/webkit/dfg-constant-fold-first-local-read-after-block-merge.js
test/webkit/dfg-constant-fold-logical-not-branch.js
test/webkit/dfg-constant-fold-misprediction.js
test/webkit/dfg-cse-cfa-discrepancy.js
test/webkit/dfg-cse-dead-get-scoped-var.js
test/webkit/dfg-dead-min-one-arg.js
test/webkit/dfg-dead-min-two-args.js
test/webkit/dfg-dead-speculation.js
test/webkit/dfg-dead-variable-on-exit.js
test/webkit/dfg-double-use-of-post-simplification-double-prediction.js
test/webkit/dfg-double-vote-fuzz.js
test/webkit/dfg-ensure-non-array-array-storage-on-window.js
test/webkit/dfg-get-by-val-clobber.js
test/webkit/dfg-getter-throw.js
test/webkit/dfg-getter.js
test/webkit/dfg-holy-put-by-val-interferes-with-get-array-length.js
test/webkit/dfg-inline-arguments-osr-exit-and-capture.js
test/webkit/dfg-inline-arguments-out-of-bounds.js
test/webkit/dfg-inline-arguments-use-directly-from-inlined-code.js
test/webkit/dfg-inline-arguments-use-from-all-the-places-broken.js
test/webkit/dfg-inline-arguments-use-from-all-the-places.js
test/webkit/dfg-inline-arguments-use-from-getter.js
test/webkit/dfg-inline-function-dot-caller.js
test/webkit/dfg-inline-unused-this-method-check.js
test/webkit/dfg-inline-unused-this.js
test/webkit/dfg-int-overflow-in-loop.js
test/webkit/dfg-int-overflow-large-constants-in-a-line.js
test/webkit/dfg-integer-optimization.js
test/webkit/dfg-intrinsic-side-effect-assignment-osr-exit.js
test/webkit/dfg-intrinsic-unused-this-method-check.js
test/webkit/dfg-intrinsic-unused-this.js
test/webkit/dfg-max-backwards-propagation.js
test/webkit/dfg-min-backwards-propagation.js
test/webkit/dfg-mispredict-variable-but-prove-int.js
test/webkit/dfg-mul-big-integer-with-small-integer-and-bitor.js
test/webkit/dfg-mul-big-integer-with-small-integer-and-detect-overflow.js
test/webkit/dfg-mul-big-integers.js
test/webkit/dfg-multi-basic-block-structure-clobber.js
test/webkit/dfg-multiply.js
test/webkit/dfg-negative-array-index.js
test/webkit/dfg-patchable-get-by-id-after-watchpoint.js
test/webkit/dfg-peephole-compare-final-object-to-final-object-or-other-when-both-proven-final-object.js
test/webkit/dfg-peephole-compare-final-object-to-final-object-or-other-when-proven-final-object.js
test/webkit/dfg-phantom-base.js
test/webkit/dfg-phantom-get-local.js
test/webkit/dfg-proto-access-inline-osr-exit.js
test/webkit/dfg-proven-sqrt-backwards-propagation.js
test/webkit/dfg-put-by-id-allocate-storage-polymorphic.js
test/webkit/dfg-put-by-id-allocate-storage.js
test/webkit/dfg-put-by-id-prototype-check.js
test/webkit/dfg-put-by-id-reallocate-storage-polymorphic.js
test/webkit/dfg-put-by-id-reallocate-storage.js
test/webkit/dfg-put-by-val-setter-then-get-by-val.js
test/webkit/dfg-put-scoped-var-backward-flow.js
test/webkit/dfg-sqrt-backwards-propagation.js
test/webkit/dfg-store-unexpected-value-into-argument-and-osr-exit.js
test/webkit/dfg-string-stricteq.js
test/webkit/dfg-tear-off-arguments-not-activation.js
test/webkit/dfg-tear-off-function-dot-arguments.js
test/webkit/dfg-to-string-on-cell.js
test/webkit/dfg-to-string-on-value.js
test/webkit/dfg-to-string-toString-becomes-bad-with-check-structure.js
test/webkit/dfg-to-string-toString-becomes-bad-with-dictionary-string-prototype.js
test/webkit/dfg-to-string-toString-becomes-bad.js
test/webkit/dfg-to-string-toString-in-string.js
test/webkit/dfg-to-string-valueOf-in-string.js
test/webkit/dfg-uint32-to-number-in-middle-of-copy-propagation.js
test/webkit/dfg-uint32-to-number-on-captured-variable.js
test/webkit/dfg-uint32-to-number-skip-then-exit.js
test/webkit/dfg-uint32-to-number.js
test/webkit/dfg-uint32array-overflow-constant.js
test/webkit/dfg-weak-js-constant-silent-fill.js
test/webkit/fast/js/JSON-parse-reviver.js
test/webkit/fast/js/deep-recursion-test.js
test/webkit/fast/js/function-decompilation-operators.js
test/webkit/fast/js/kde/Array.js
test/webkit/fast/js/kde/Boolean.js
test/webkit/fast/js/kde/Date-setYear.js
test/webkit/fast/js/kde/Error.js
test/webkit/fast/js/kde/GlobalObject.js
test/webkit/fast/js/kde/Number.js
test/webkit/fast/js/kde/Object.js
test/webkit/fast/js/kde/Prototype.js
test/webkit/fast/js/kde/RegExp.js
test/webkit/fast/js/kde/arguments-scope.js
test/webkit/fast/js/kde/assignments.js
test/webkit/fast/js/kde/cast.js
test/webkit/fast/js/kde/comment-1.js
test/webkit/fast/js/kde/comment-2.js
test/webkit/fast/js/kde/completion.js
test/webkit/fast/js/kde/conditional.js
test/webkit/fast/js/kde/constructor_length.js
test/webkit/fast/js/kde/crash-1.js
test/webkit/fast/js/kde/crash-2.js
test/webkit/fast/js/kde/delete.js
test/webkit/fast/js/kde/empty.js
test/webkit/fast/js/kde/encode_decode_uri.js
test/webkit/fast/js/kde/eval.js
test/webkit/fast/js/kde/evil-n.js
test/webkit/fast/js/kde/exception_propagation.js
test/webkit/fast/js/kde/exceptions.js
test/webkit/fast/js/kde/func-decl.js
test/webkit/fast/js/kde/inbuilt_function_proto.js
test/webkit/fast/js/kde/iteration.js
test/webkit/fast/js/kde/j-comment-3.js
test/webkit/fast/js/kde/j-comment-4.js
test/webkit/fast/js/kde/literals.js
test/webkit/fast/js/kde/lval-exceptions.js
test/webkit/fast/js/kde/math.js
test/webkit/fast/js/kde/md5-1.js
test/webkit/fast/js/kde/md5-2.js
test/webkit/fast/js/kde/object_prototype.js
test/webkit/fast/js/kde/object_prototype_tostring.js
test/webkit/fast/js/kde/operators.js
test/webkit/fast/js/kde/parse.js
test/webkit/fast/js/kde/prototype_length.js
test/webkit/fast/js/kde/prototype_proto.js
test/webkit/fast/js/kde/scope.js
test/webkit/fast/js/kde/statements.js
test/webkit/fast/js/kde/var_decl_init.js
test/webkit/fast/js/string-anchor.js
test/webkit/fast/js/string-fontcolor.js
test/webkit/fast/js/string-fontsize.js
test/webkit/fast/js/string-link.js
test/webkit/fast/js/toString-number.js
test/webkit/fast/regex/assertion.js
test/webkit/fast/regex/constructor.js
test/webkit/fast/regex/dotstar.js
test/webkit/fast/regex/parentheses.js
test/webkit/function-dot-apply-replace-base.js
test/webkit/get-by-pname-non-final-object.js
test/webkit/get-by-pname-that-looks-like-a-patchable-get-by-val.js
test/webkit/get-by-pname.js
test/webkit/indexed-setter-on-global-object.js
test/webkit/new-array-double-with-holes.js
test/webkit/regexp-in-and-foreach-handling.js
test/webkit/regexp-zero-length-alternatives.js
test/webkit/resolve-arguments-from-scope.js
test/webkit/sort-with-side-effecting-comparisons.js
test/webkit/stack-unwinding.js
test/webkit/string-trim.js
test/webkit/throw-from-finally.js
test/webkit/try-catch-try-try-catch-try-finally-return-catch-finally.js
test/webkit/try-try-return-finally-finally.js
test/webkit/var-declarations-zero-width.js
tools/consarray.js
tools/generate-trig-table.py
tools/presubmit.py
tools/testrunner/local/junit_output.py
tools/tickprocessor.js