Fix hash table overallocation etc.

* src/fns.c (set_hash_key_and_value, set_hash_next)
(set_hash_hash, set_hash_index): Remove.  All uses removed.
(maybe_resize_hash_table): Don’t update h->next until it’s
known that all the allocations succeeded, to avoid trashing
the hash table if memory is exhausted.  Don’t overallocate the
other vectors.  Don’t output growth message if the hash table
didn’t actually grow due to allocation failure.  Assume C99
decls after statements.

Merge from emacs-26

150bdfe43a Handle completely undecoded input in term (Bug#29918)
021f32cca1 * doc/misc/forms.texi (Control File Format): Fix a doc error.
76538d09b7 Fix typo in package-alist docstring
b2fde4b5e8 * doc/lispref/text.texi (Mode-Specific Indent): Fix a typo...
7e62778548 ; Another minor change in 'bidi-display-reordering's doc s...
4455ddbe56 Improve doc string of 'bidi-display-reordering'
34ee26dd93 Add warning to bidi-display-reordering doc string

# Conflicts:
# lisp/term.el
# test/lisp/term-tests.el

Handle completely undecoded input in term (Bug#29918)

* lisp/term.el (term-emulate-terminal): Avoid errors if the whole
decoded string is eight-bit characters.  Don't attempt to save the
string for next iteration in that case.
* test/lisp/term-tests.el (term-decode-partial)
(term-undecodable-input): New tests.

Use lexical-binding in compface.el

* lisp/image/compface.el: Use lexical-binding.  Extend Keywords
header.
(uncompface): Call call-process-region directly.

* lisp/files.el (file-size-function): Add :version tag

* test/manual/indent/octave.m (spmd): Add test for last change

* lisp/progmodes/octave.el (matchedrules): Add `spmd...end` (bug#36703)

Fix last change to format-spec

* doc/lispref/text.texi (Interpolated Strings): Use @result and fix
typos.
* lisp/format-spec.el: Avoid loading subr-x at runtime.
(format-spec--parse-modifiers): Optimize slightly.

Allow counter-clockwise rotations in image-rotate

* lisp/image.el (image-rotate): Extend with an optional argument
specifying the rotation in degrees (bug#35421).
* doc/lispref/display.texi (Showing Images):
* etc/NEWS: Document the change.
* test/lisp/image-tests.el (image-rotate): New test.

Mark the gnus-article-date-headers NEWS entry as documented

Enable showing local time and lapsed time in Gnus

* lisp/gnus/gnus-art.el (article-make-date-combine-with-lapsed) factor
code out into new function, used for providing both combined-lapsed
and combined-local-lapsed.

Clean up code in nnimap-parse-flags slightly

* lisp/gnus/nnimap.el (nnimap-parse-flags): Clean up code
slightly, removing redundant checks for `end'.

* doc/misc/forms.texi (Control File Format): Fix a doc error.

(Bug#36693)

Copyright-paperwork-exempt: yes

Fix last change

* etc/NEWS: Call out the change in matching REGEXP.

* lisp/files.el (magic-mode-alist)
(magic-fallback-mode-alist): Doc fix.  (Bug#36401)

Make REs in magic-(fallback-)mode-alist case-sensitive.

These variables are used for well-defined file formats where relaxed
case matching is not wanted usually.

* lisp/files.el (magic-mode-alist, magic-fallback-mode-alist): Update
the doc string.
(set-auto-mode): Make looking-at for elements of magic-mode-alist and
magic-fallback-mode-alist use case-fold-search == nil.
* lisp/files.el (files-test-magic-mode-alist-re-baseline)
(files-test-magic-mode-alist-re-no-match)
(files-test-magic-mode-alist-re-case-diff): Add.

Make finder-exit use quit-window (Bug#33610)

* lisp/finder.el (finder-exit): Quit window instead of
deleting.  This restores previous contents of the window,
if any, that was usurped by "C-h p" to show the list of
packages.

Remove duplicated code in octave-eldoc-function

* lisp/progmodes/octave.el (octave-eldoc-function): The
test of position of the opening parenthesis in
inferior-octave-process was duplicated.  Remove one of
the tests.  (Bug#36557)

* lisp/files.el (file-size-function): New defcustom

Neater free disk space formatting

* lisp/files.el (get-free-disk-space): Update doc string.
Use `iec' style and proper spacing.

Have 'display-buffer-reuse-window' prefer window on selected frame (Bug#36680)

* lisp/window.el (display-buffer-reuse-window): Preferably reuse
window on selected frame (Bug#36680).
* doc/lispref/windows.texi (Buffer Display Action Functions): Say
that 'display-buffer-reuse-window' prefers window on the selected
frame.

Don't load elec-pair in elisp-mode (Bug#36539)

* lisp/progmodes/elisp-mode.el (emacs-lisp-set-electric-text-pairs):
New function.
(emacs-lisp-mode): Add it to electric-pair-mode-hook, if elec-pair
hasn't been loaded yet.

Use lexical-binding in help-fns-tests.el (Bug#36585)

* test/lisp/help-fns-tests.el: Use lexical-binding.  Doc fix.

Declare mwheel-install obsolete (Bug#36553)

* lisp/mwheel.el: Update `Commentary' section.
(mwheel-install): Declare obsolete in favor of mouse-wheel-mode.
* etc/NEWS: Announce it.

* lisp/vc/vc-git.el (vc-git-log-search): Remove shell-quote-argument.

(Bug#36644)

Merge branch 'master' of git.sv.gnu.org:/srv/git/emacs

; Fix last commit in tramp-sh.el

Use lexical-binding in asm-mode.el and add tests

* lisp/progmodes/asm-mode.el: Use lexical-binding.
(asm-comment-char): Remove redundant :group arg.
(asm-mode): Use `setq-local'.
(asm-calculate-indentation): Remove moot `or'.

* test/lisp/progmodes/asm-mode-tests.el: New file with tests for
asm-mode.el.

Determine `file-notify' handler in Tramp from `special-event-map'.

* lisp/net/tramp-sh.el (tramp-sh-gio-monitor-process-filter)
(tramp-sh-gvfs-monitor-dir-process-filter)
(tramp-sh-inotifywait-process-filter): Determine `file-notify'
handler from `special-event-map'.

* lisp/server.el (server-ensure-safe-dir): Revert part of b663c8372495

A non-cosmetic change that was mistakenly included.

Update a tramp test for get-free-disk-space change

* test/lisp/net/tramp-tests.el (tramp-test17-insert-directory):
Handle optional unit suffix in free space.

* lisp/files.el (get-free-disk-space): Use file-size-human-readable

Since there is no longer an option to do:

    (setq directory-free-space-args "-Pmh")

* lisp/abbrev.el (abbrev-prefix-mark): Fix a typo.  (Bug#36715)

Fix indexing of Gnus entry in gnus-group-unsubscribe-group

* lisp/gnus/gnus-group.el (gnus-group-unsubscribe-group): Hopefully
  that's the last of these.

Add xref-file-name-display

* lisp/progmodes/xref.el (xref-file-name-display): New user variable.
(xref-location-group): Use it.

* etc/NEWS: Mention it.

* test/lisp/progmodes/cc-mode-tests.el: Add a test with /***/ in #define lines

Fix typo in package-alist docstring

Pointed out by Michael Heerdegen <michael_heerdegen@web.de>.
* lisp/emacs-lisp/package.el (package-alist): Fix docstring
grammar (bug#17403).

* lisp/progmodes/cc-fonts.el (gtkdoc-font-lock-keywords): Fix faulty regexp

An electric test is now passing

* test/lisp/electric-tests.el
(electric-pair-whitespace-chomping-2-at-point-4-in-c++-mode-in-strings):
Seems to pass since today's cc-mode changes.

* lisp/wid-edit.el: Previous change needs subr-x.

Treat the "Link" link in gnus-summary-browse-urls specially

* lisp/gnus/gnus-sum.el (gnus-collect-urls): Make sure that
the URL labeled "Link" is the first in the return list.
(gnus-summary-browse-url): Use the 1st URL as the default.
* lisp/wid-edit.el (widget-text): New function.

CC Mode: allow bogusly "adjacent" double quote marks to pair up syntactically

For this introduce the text property c-fl-syn-tab to "mirror" syntax-table.

* lisp/progmodes/cc-defs.el (c-is-escaped, c-will-be-escaped, c-put-syn-tab)
(c-clear-syn-tab, c-clear-syn-tab-properties, c-with-extended-string-fences):
new macros.
(c-point): Use c-is-escaped
(c-search-forward-char-property, c-search-backward-char-property)
(c-search-forward-char-property-with-value-on-char)
(c-search-forward-char-property-without-value-on-char): Fix regexp error
involving \n.

* lisp/progmodes/cc-engine.el (c-semi-pp-to-literal): User
c-with-extended-string-fences around a parse-partial-sexp.
(c-full-get-near-cache-entry): Fix an off-by-one error.
(c-full-pp-to-literal): Avoid writing duplicate entries into a cache.
(c-after-change-unmark-raw-strings): Use c-clear-syn-tab-properties.

* lisp/progmodes/cc-mode.el (c-leave-cc-mode-mode): Clear the c-fl-syn-tab
properties, too.
(c-basic-common-init): Mark c-fl-syn-tab as a non-sticky text property.
(c-depropertize-new-text): Also handle c-fl-syn-tab.
(c-multiline-string-check-final-quote, c-parse-quotes-after-change): Use
c-is-escaped.
(c-fl-syn-tab-region): New variable.
(c-clear-string-fences, c-restore-string-fences, c-remove-string-fences): New
functions.
(c-before-change-check-unbalanced-strings)
(c-after-change-mark-abnormal-strings, c-after-change-escape-NL-in-string):
Use the new functions and macros.
(c-before-change, c-after-change, c-font-lock-fontify-region): Restore the
syntax-table text properties from c-fl-syn-tab text properties for these
functions.
(c-electric-pair-inhibit-predicate): Test the c-fl-syn-tab property rather
than syntax-table.

Fix failing secrets-test03-items

* test/lisp/net/secrets-tests.el: Use lexical-binding.
(secrets-test03-items): Do not assume :xdg:schema attribute is
silently added to the front of the collection (bug#36694).
(secrets-test04-search, secrets-test-all): Quote function symbols as
such.

Fix calc number formatting with digit grouping (bug#36689)

The functions math-format-hex and math-format-octal were not
implemented, yet called, leading to a crash when using hex or octal
radix with digit grouping.

* test/lisp/calc/calc-tests.el (calc-test-format-radix): New test.
* lisp/calc/calc-ext.el: Don't declare non-existing functions.
(math--format-integer-fancy): Don't call non-existing functions.
* lisp/calc/calc-bin.el (math-format-binary, math-binary-digits):
Simplify, fixing 0-padding bug.

Revert "Add support for paths to svg.el"

This reverts commit d6bc55ae2dc98c83e58a28e380ce4bcf2ed00bb3.

Paperwork not ready for Felix Klee; will reapply once that's in place.

Revert "* doc/lispref/display.texi (SVG Images): Add menu for subsection."

This reverts commit e5cd597c7736b587f80c16fde1c53f904d921310.

Paperwork nod ready for the SVG Path patch.

Set revert-buffer-function in ert-results-mode

* lisp/emacs-lisp/ert.el (ert-results-mode): Set revert-buffer-function
to re-run the same test selection.  It is a good fit and permits use of
the standard "g" key binding.

Prevent infinite loop on entering wdired-mode

* lisp/wdired.el (wdired-preprocess-symlinks): Remove unneeded use
of beginning-of-line, which also triggers an infinite loop in a
find-dired buffer that doesn't end with a newline (see discussion
in bug#35609).

* lisp/vc/vc.el (vc-log-search): New command (bug#36644).

* lisp/vc/vc-git.el (vc-git-log-search): New function.
(vc-git-log-view-mode): Check vc-log-view-type for log-search.

* lisp/vc/vc-dir.el (vc-dir-search): Remove lambda.

(Bug#31578) Thanks to Dmitry Gutov <dgutov@yandex.ru>

* lisp/frame.el (make-frame-on-display): Use x-display-list in completion.

(Bug#29713)

* lisp/progmodes/sh-script.el (sh-assignment): Put delete-selection on symbol

* doc/lispref/display.texi (SVG Images): Add menu for subsection.

* lisp/gnus/nntp.el: Use lexical-binding

(nntp-server-action-alist): Expose the code to the compiler.
(nntp-with-open-group-function): No need to obfuscate identifiers any more.
(nntp-authinfo-rejected): Use `define-error`.

Fix INITIAL-INPUT cons handling in ido-read-internal

* lisp/ido.el (ido-read-internal): Is INITIAL-INPUT is a cons that
specifies cursor position, adjust the number to be palatable for
`read-for-minibuffer' (bug#27807).

Fix saving certificates in the NSM on high security levels

* lisp/net/nsm.el (nsm-check-certificate): `nsm-fingerprint-ok-p'
will save the fingerprint in the correct temporary/permanent
storage, so saving it once more (in the permanent storage) if the
security level is high or greater is a mistake (bug#27823).

Tweak mouse-face highlighting of Occur buffers

* lisp/replace.el (occur-engine): Ensure that the mouse highlight
is done over the entire line (bug#27846).

open-dribble-file doc string clarification

* src/keyboard.c (Fopen_dribble_file): Clarify when the file is
closed, and when events are written to the file (bug#27996).

Possibly skip IMAP server FETCH responses

See bug#35433

* lisp/gnus/nnimap.el (nnimap-transform-headers): Skip FETCH responses
  that only provide message flags, not message headers.

Add support for paths to svg.el

* doc/lispref/display.texi (SVG Images): Document svg-path,
svg-clip-path and svg-node (bug#32359).

* doc/lispref/display.texi (SVG Path Commands): New node.

* lisp/svg.el (svg--plist-delete, svg--path-command-symbol)
(svg--elliptical-arc-coordinates, svg--elliptical-arc-command)
(svg--moveto-command, svg--closepath-command)
(svg--lineto-command, svg--horizontal-lineto-command)
(svg--vertical-lineto-command, svg--curveto-command)
(svg--smooth-curveto-command)
(svg--quadratic-bezier-curveto-command)
(svg--smooth-quadratic-bezier-curveto-command)
(svg--eval-path-command, svg-path, svg-clip-path, svg-node): New
functions.

Revert "Add edebug specs for inline.el"

This reverts commit ed29d9fe5286ab6ecfb2bfa5a015b9945a84512b.

The edebug specs were already in the declare form, and in addition, it didn't fix the problem.

Mention changing the font size in the Emacs manual

* doc/emacs/frames.texi (Fonts): Mention changing the font scale
(bug#28196).

Mark semantic functions correctly in defcustom

* lisp/cedet/semantic/format.el (semantic-format-tag-custom-list):
Mark functions correctly in defcustom (bug#28267).

Clarify doc of isearch-wrap-function

* lisp/isearch.el (isearch-wrap-function): Doc clarification
(bug#28388).

Don't delete GnuTLS boot parameters too early

* src/process.c (connect_network_socket): Don't delete the GnuTLS
boot parameters until after we've managed to connect at the IP
level (bug#36660).

Fix *Messages* grouping in C-Mouse-1 buffer menu

* lisp/mouse.el (mouse-buffer-menu-mode-groups): Don't group the
*Messages* buffer with Mail/News (bug#32007).

* lisp/net/tramp-cmds.el (tramp-bug, tramp-append-tramp-buffers):

Don't cancel if debug buffers aren't appended.

Make directory-files-recursively take a PREDICATE parameter

* lisp/files.el (directory-files-recursively): Take an optional
PREDICATE parameter (bug#28567).

* doc/lispref/files.texi (Contents of Directories): Document it.

Tweak gv-define-setter debug setting

* lisp/emacs-lisp/gv.el (gv-define-setter): Change edebug spec
from body to def-body (bug#28729).

Fix edge cases in Tramp's copy

* lisp/net/tramp-sh.el (tramp-do-copy-or-rename-file-out-of-band):
Special handling for "rsync".
(tramp-find-inline-compress): Make check more robust on MS Windows.

package-archive-priorities doc addition

* lisp/emacs-lisp/package.el (package-archive-priorities):
Document that negative priorities will lead to packages not being
updated (bug#28897).

Stop wdired tests leaving behind symlink /tmp/emacs1000

* test/lisp/wdired-tests.el (server-socket-dir): Make effective.

Mark octave-block-offset as a safe local variable

* lisp/progmodes/octave.el (octave-block-offset): Mark as safe as
a local variable (bug#28916).

; NEWS copyedits

Update a ffap test

* test/lisp/ffap-tests.el (ffap-other-window--bug-25352):
Update for incompatible ffap-other-window change.

Make elint understand condition-case-unless-debug

* lisp/emacs-lisp/elint.el (elint-special-forms): Teach elint
about `condition-case-unless-debug' (bug#29585).

Fix prompting in functions like ffap-read-only

* lisp/ffap.el (ffap-prompter, ffap-other-window)
(ffap-other-frame, ffap-read-only)
(ffap-read-only-other-window, ffap-read-only-other-frame)
(ffap-alternate-file, ffap-alternate-file-other-window)
(ffap-literally): Rework to be able to specify different prompts
for different actions (bug#30284) and don't use
`call-interactively' all over the place.

Make describe-face also output the version information

* lisp/help-fns.el (describe-variable-custom-version-info): Allow
taking a type as an optional input, so this can be used for faces,
too (bug#30527).

* lisp/faces.el (describe-face): Use this to output the version
information.

Use correct colorspace for XBM images

* src/nsimage.m ([EmacsImage setXBMColor:]): Use 'generic RGB' color
space.

Revert "Make describe-face also output the version information"

This reverts commit a7aae1473c1aed7758b550a23cda61ee17668e23.

The patch broke the build.

Tweak bug ID prompting in gnus-group--read-bug-ids

* lisp/gnus/gnus-group.el (gnus-group--read-bug-ids): Default to
current bug id on RET.

Make describe-face also output the version information

* lisp/help-fns.el (describe-variable-custom-version-info): Allow
taking a type as an optional input, so this can be used for faces,
too (bug#30527).

* lisp/faces.el (describe-face): Use this to output the version
information.

; Add comment to hash_table_rehash (Bug#36447)

* src/fns.c (hash_table_rehash): Add a comment to explain that
hash_table_rehash is not used in "normal" rehashing, but only in the
rare case of rehashing on the first access to a preloaded hash-table.

Use ngettext in vc-dir-clean-files

* lisp/vc/vc-dir.el (vc-dir-clean-files): Use ngettext to separate
pluralization from format string.

Tweak background colour handling in shr

* lisp/net/shr.el (shr-fill-line): Keep the background colour on
the newline and the indentation.

Rename vc-dir-delete-files-no-vc to vc-dir-clean-files

* doc/emacs/maintaining.texi (VC Directory Commands): Adjust
command name.

* lisp/vc/vc-dir.el (vc-dir-clean-files): Rename from
vc-dir-delete-files-no-vc, and work on unregistered files only
(bug#31732).
(vc-dir-menu-map): Adjust command name.

Allow bad tags in Scheme

* admin/grammars/scheme.by (scheme-in-list): Allow bad tags in
Scheme (bug#36521).

* doc/lispref/text.texi (Mode-Specific Indent): Fix a typo (bug#36646).

* etc/NEWS: Announce new Georgian inputs.  (Bug#36639)

* etc/HELLO: Fix name of Georgian language

* lisp/leim/quail/georgian.el: Add georgian-nuskhuri input method

* lisp/leim/quail/georgian.el: Add missing characters to georgian input

* lisp/leim/quail/georgian.el: Add georgian-qwerty input method

Restore focus to Bookmark List after editing annotation

There are two entry points to bookmark-edit-annotation-mode: the first
when we add a bookmark and bookmark-use-annotations is non-nil; the
second when bookmark-bmenu-edit-annotation is run from the bookmark
list.  When editing is concluded, in the first case, we should just
quit window.  In the second case, we should instead return to the
bookmark list.

* lisp/bookmark.el (text-property-search): Require.
(bookmark-annotation-name): Make buffer-local and improve doc string.
(bookmark--annotation-from-bookmark-list): New buffer-local variable.
(bookmark-edit-annotation): New argument from-bookmark-list sets
bookmark--annotation-from-bookmark-list.
(bookmark-bmenu-edit-annotation): Call bookmark-edit-annotation with
argument from-bookmark-list set to t.
(bookmark-send-edited-annotation): When editing originated in the
bookmark list, restore focus to bookmark list and move point back to
edited bookmark. (Bug#20150)

(bookmark-edit-annotation-mode): Fix typo.
(bookmark-bmenu-buffer): New variable.
(bookmark-bmenu-surreptitiously-rebuild-list)
(bookmark-bmenu-list): Use it.

* test/lisp/bookmark-tests.el (with-bookmark-bmenu-test): New macro.
(bookmark-tests-set/bookmark-use-annotations-t)
(bookmark-bmenu-edit-annotation/show-annotation)
(bookmark-bmenu-send-edited-annotation)
(bookmark-bmenu-send-edited-annotation/restore-focus): New test cases.

New function for scroll-lock-mode to almost always scroll

* lisp/scroll-lock.el (scroll-lock-next-line-always-scroll): New
function.  Opposed to scroll-lock-next-line it does not switch to
forward-line at eob.  S-down is the default key binding for this
function.  (Bug#36494)
* test/lisp/scroll-lock-tests.el: A few tests for
scroll-lock-next-line-always-scroll.
* etc/NEWS: Announce the new command.

Fix recent changes

* src/sysstdio.h: Include stdarg.h, for va_list.

Avoid returning negative numbers from `hash-table-count'

* src/fns.c (Fhash_table_count): Rehash argument if necessary.

* src/lisp.h (struct Lisp_Hash_Table.count): Improve comment.

; Add comment about sh-assignment-regexp value

* lisp/progmodes/sh-script.el (sh-assignment-regexp): Explain what the
the "\\[.+\\]" is for, to make clear it is not a typo for "[.+]".

Remove incorrect commentary about #$ in autoloads

* lisp/emacs-lisp/autoload.el (autoload-rubric):
`package-quickstart-refresh' does in fact support load-file-name as
well.

Use syntax-ppss-toplevel-pos for defvar search (Bug#34233)

* lisp/progmodes/elisp-mode.el (eval-sexp-add-defvars): Use
syntax-ppss-toplevel-pos instead of a raw check of syntax components.