Simplify socket symlink-attack checking

This is a minor bugfix cleanup (Bug#33847#161).
* lib-src/emacsclient.c: Move "#include <acl.h>" to inside
"#ifdef SOCKETS_IN_FILE_SYSTEM", which is more accurate
and simpler than having a separate "#ifndef WINDOWSNT".
(O_PATH): Likewise.

; * doc/lispref/files.texi (Reading from Files): Fix wording.

Clarify (elisp) insert-file-contents with BEG or END not on character boundary

* doc/lispref/files.texi (Reading from files): When the argument BEG or END to
insert-file-contents are at a byte position not at a character boundary,
clarify that raw bytes get inserted, and how to handle this awkwardness in
Lisp.  Also clarify that insert-file-contents-literally is intended to insert
raw bytes into the buffer.  Fix the outdated example that states it inserts
500 characters, when it actually inserts 500 bytes.

* src/Makefile.in: Simplify conditionals.

Move context-menu selection items Defun/List/Symbol to prog-mode (bug#9054)

* lisp/mouse.el (context-menu-functions):
Add context-menu-middle-separator to choices.
(context-menu-region): Move Defun/List/Symbol selection items
to prog-context-menu.

* lisp/progmodes/prog-mode.el (prog-context-menu):
Move Defun/List/Symbol selection items from context-menu-region.
Include text-mode select menu only in strings and comments.

* lisp/textmodes/text-mode.el (text-mode-menu): New function.
(text-mode): Add text-mode-menu to context-menu-functions.

* lisp/tab-bar.el (tab-bar-detach-tab): Handle frame selected by make-frame.

(tab-bar-move-window-to-tab): New command.
(tab-bar-new-tab-to): Handle the value 'window' of tab-bar-new-tab-choice.
https://lists.gnu.org/archive/html/emacs-devel/2021-09/msg02197.html

* lisp/tab-line.el (tab-line-format): Add face-modified to the cache key.

When tab-line-tab-face-functions contains tab-line-tab-face-modified,
add 'buffer-modified-p' status to the cache-key, so the cache will expire
when the buffer modification status will change.
https://lists.gnu.org/archive/html/emacs-devel/2021-10/msg00129.html

Fix unmounting in Tramp

* doc/misc/tramp.texi (FUSE setup): Add tramp-fuse-unmount-on-cleanup.

* lisp/net/tramp.el (tramp-file-name-unify): New defun.
(tramp-file-name-equal-p):
* lisp/net/tramp-cache.el (tramp-get-connection-property)
(tramp-set-connection-property, tramp-flush-connection-property)
(tramp-flush-connection-properties): Use it.

* lisp/net/tramp-fuse.el (tramp-fuse-get-fusermount): New defun.
(tramp-fuse-mount-points): New defvar.
(tramp-fuse-unmount): Use it.  Delete VEC from
`tramp-fuse-mount-points'.  Delete mount point.
(tramp-fuse-unmount-on-cleanup): New user option.
(tramp-fuse-cleanup, tramp-fuse-cleanup-all): New defuns.
(top): Adapt `tramp-fuse-unload-hook',
`tramp-cleanup-connection-hook',
`tramp-cleanup-all-connections-hook' and `kill-emacs-hook'.

* lisp/net/tramp-rclone.el (tramp-rclone-maybe-open-connection):
* lisp/net/tramp-sshfs.el (tramp-sshfs-maybe-open-connection):
Add VEC to `tramp-fuse-mount-points'.

* test/lisp/net/tramp-tests.el (tramp-fuse-unmount-on-cleanup): Declare.
(tramp-test39-make-lock-file-name): Use it.

* etc/themes/light-blue-theme.el: Add "Maintainer: emacs-devel".

Define HAVE_NATIVE_COMP in src/Makefile.in

* src/Makefile.in (HAVE_NATIVE_COMP): Define.  Reported by Ken
Brown <kbrown@cornell.edu>.

Rename elisp-shorthands to read-symbol-shorthands

The new name fits better in the family of variables that affect
the Lisp reader.

Suggested-by: Po Lu <luangruo@yahoo.com>
* doc/lispref/symbols.texi (Shorthands): Mention read-symbol-shorthands

* lisp/shorthands.el (hack-read-symbol-shorthands)
(hack-read-symbol-shorthands)
(shorthands-font-lock-shorthands): Use read-symbol-shorthands

* lisp/progmodes/elisp-mode.el (elisp--completion-local-symbols)
(elisp--completion-local-symbols)
(elisp-shorthands): Use read-symbol-shorthands

* src/lread.c:
(syms_of_lread): Define Vread_symbol_shorthands
(oblookup_considering_shorthand): Use Vread_symbol_shorthands.

* test/lisp/progmodes/elisp-mode-tests.el (elisp-shorthand-read-buffer):
(elisp-shorthand-read-from-string): Use read-symbol-shorthands

* test/lisp/progmodes/elisp-mode-resources/simple-shorthand-test.el
Use new symbol name read-symbol-shorthands.

Font-lock shorthands in elisp-mode for quick visual recognition (bug#50959)

Only the shorthanded prefix is font-locked.  This allows the remainder
of the font-lock logic to subsist (e.g. for macro-defining symbols).

* lisp/shorthands.el (cl-lib): Require it when compiling.
(elisp-shorthand-font-lock-face): New face.
(shorthands--mismatch-from-end): New helper.
(shorthands-font-lock-shorthands): New helper.

* test/lisp/progmodes/elisp-mode-resources/simple-shorthand-test.el:
Add some dummy test code.

; Fix last change in tramp-sshfs.el

Suppress superfluous error messages in Tramp

* lisp/net/tramp-sshfs.el (tramp-sshfs-handle-insert-file-contents):
* lisp/net/tramp.el (tramp-handle-insert-file-contents)
(tramp-handle-lock-file): Suppress superfluous error message.

Fix reading the tail of a file in shorthands.el

* lisp/shorthands.el (hack-elisp-shorthands): Fix reading past
3000-character limit from EOF.  (Bug#50946)

Fix substitution of pretty quotes in code in easy-mmode

* lisp/emacs-lisp/easy-mmode.el (easy-mmode--arg-docstring): Adjust.
(easy-mmode--mode-docstring): Avoid making quotes into pretty
quotes in code (bug#50968).

Fix agent directory deletion

* lisp/gnus/gnus-agent.el (gnus-agent-expire-unagentized-dirs):
Delete directories in a simpler way that actually works (bug#50986).

; * etc/TODO: Fix previous commit; delete the right thing.

Clarify the purpose of internal--format-docstring-line

* test/lisp/subr-tests.el (subr-test-internal--format-docstring-line):
* lisp/subr.el (internal--format-docstring-line): Make it more clear
that this function is not intended for the first line of a docstring.
* lisp/emacs-lisp/cl-macs.el (cl-defstruct): Add comment explaining
why we use 'internal--format-docstring-line'.
Problem pointed out by Stefan Monnier <monnier@iro.umontreal.ca>.

* lisp/net/dictionary.el (context-menu-dictionary): Move menu item down.

Place the dictionary menu item after middle-separator (bug#50552).

Update to Org 9.5-30-g10dc9d

The plan is to cut the Org 9.5.1 release and include it in Emacs 28.1,
but in the meantime regularly sync changes from Org's bugfix branch to
emacs-28.

This sync includes files from Org 9.5's new etc/csl/ directory that
should have been synced in bf9ec3d91a (Update to Org 9.5, 2021-09-29).

Remove bogus ":safe t" custom properties

* lisp/org/oc.el (org-cite-global-bibliography)
(org-cite-note-rules, org-cite-punctuation-marks):
* lisp/org/oc-csl.el (org-cite-csl-locales-dir)
(org-cite-csl-styles-dir, org-cite-csl-no-citelinks-backends):
* lisp/org/oc-natbib.el (org-cite-natbib-options):
* lisp/org/org-keys.el (org-mouse-1-follows-link):
Remove bogus ":safe t" properties that would largely need to be
replaced by custom predicates.

Fix 'apropos-compact-layout'

* lisp/textmodes/fill.el (fill-region-as-paragraph): Fix filling
paragraphs that end at EOB without a newline.  (Bug#50974)

Remove bogus ":safe nil" custom properties

* lisp/org/oc.el (org-cite-activate-processor)
(org-cite-export-processors, org-cite-follow-processor)
(org-cite-insert-processor):
* lisp/org/ol.el (org-link-parameters, org-link-frame-setup)
(org-link-shell-confirm-function)
(org-link-shell-skip-confirm-regexp)
(org-link-elisp-confirm-function)
(org-link-elisp-skip-confirm-regexp):
* lisp/org/org-num.el (org-num-format-function):
Remove bogus ":safe nil" that do nothing but propagate a
misunderstanding of the safe-local-variable property.

The safe-local-variable property is a function (bug#50944)

* lisp/org/oc-basic.el (org-cite-basic-sorting-field)
(org-cite-basic-author-year-separator)
(org-cite-basic-max-key-distance)
(org-cite-basic-author-column-end)
(org-cite-basic-column-separator)
(org-cite-basic-mouse-over-key-face):
* lisp/org/oc-biblatex.el (org-cite-biblatex-options):
* lisp/org/oc-csl.el (org-cite-csl-link-cites)
(org-cite-csl-html-hanging-indent)
(org-cite-csl-html-label-width-per-char)
(org-cite-csl-latex-hanging-indent):
* lisp/org/oc.el (org-cite-adjust-note-numbers):
* lisp/org/org-keys.el (org-return-follows-link):
* lisp/org/org.el (org-fontify-todo-headline):
* lisp/org/ox-html.el (org-html-equation-reference-format)
(org-html-wrap-src-lines):
* lisp/org/ox-latex.el (org-latex-reference-command)
(org-latex-default-quote-environment):
* lisp/textmodes/tildify.el (tildify-pattern)
(tildify-space-string): Fix :safe property.

; Some minor tweaks to TODO

* etc/TODO: Explain elpa.gnu.org items better.  Delete fixed item
about dynamic embedded graphics.  (Bug#50937)

Revert "; * etc/TODO: Move elpa.gnu.org items to the end."

This reverts commit d73f0e96a7026808c01861f7525a2909279fc00d.

These items are a priority for the project and should be before other,
less prioritized items, according to a private discussion with project
co-maintainer Eli Zaretskii <eliz@gnu.org>.

Revert "* etc/TODO: Rearrange to start with "Simple tasks"."

This reverts commit 879ef5b19ab1dd90284aef829ef306d56b4e5adb.

Some of these items are a priority for the project and should be
before other, less prioritized items, according to a private
discussion with project co-maintainer Eli Zaretskii <eliz@gnu.org>.

Fix selection of fonts for Arabic on Posix platforms

* lisp/international/fontset.el (script-representative-chars): Add
U+06C1 to representative-characters for Arabic.  (Bug#50951)

; Fix a typo in a doc string

* lisp/files.el (hack-local-variables--find-variables): Fix typo
in the doc string.

Fix ox-koma-letter compilation warnings

* lisp/org/ox-koma-letter.el (org-koma-letter-export-block)
(org-koma-letter-export-snippet): Fix warning about two unused
parameter.

Preload paren.el

* lisp/Makefile.in (COMPILE_FIRST): Add the dependencies of
comp.el, so that they are natively-compiled in advance.
* lisp/loadup.el ("paren"): Preload paren.el. (Bug#50934)

Improve documentation of 'shift-select-mode'

* doc/emacs/mark.texi (Shift Selection): Document the 'permanent'
value of 'shift-select-mode'.  Add index entry for that variable.
(Bug#50954)

* etc/NEWS: Update the entry for 'shift-select-mode'.

; Standardize some license headers

Simplify shorthand injection (bug#50946)

* lisp/loadup.el: Load "shorthands" relatively late. Set
load-source-file-function to load-with-shorthands-and-code-conversion

* lisp/international/mule.el (hack-elisp-shorthands)
(load-with-shorthands-and-code-conversion): Move to
lisp/shorthands.el

* lisp/shorthands.el: New file.

; * admin/release-branch.txt: Tweak previous.

* doc/lispref/control.texi (pcase Macro): Fix cross-reference.

; * lisp/time-stamp.el: Fix wording in some documentation strings.

; * admin/release-branch.txt: Small edits.

Fix some custom types

* lisp/mpc.el (mpc-cover-image-re):
* lisp/org/oc-csl.el (org-cite-csl-locales-dir)
(org-cite-csl-styles-dir):
* lisp/org/org-plot.el (org-plot/preset-plot-types): Fix :type.

; Fix oddities in etc/NEWS

; Auto-commit of loaddefs files.

* etc/TODO: Add interactive mode tagging.

* etc/TODO: Rearrange to start with "Simple tasks".

; * etc/TODO: Move elpa.gnu.org items to the end.

It makes no sense to have them before "easy" items intended for
beginners, especially since we miss some things to make this happen.

; * etc/TODO: Mention a relevant bug report.

* etc/TODO: Remove outdated item.  (Bug#50904)

; Oops, fix up last commit

Rename arguments of ERC's '/whois' and simplify doc string

* lisp/erc/erc.el (erc-cmd-WHOIS): Rename arguments and reword the doc
string to further clarify and simplify.

; s/class/type

Migrate Xref off EIEIO

To improve performance and flexibility (bug#50777).

* lisp/progmodes/xref.el (xref-location): Remove.
(xref-file-location): Change to cl-struct.
(xref-buffer-location, xref-bogus-location): Ditto.
(xref-item, xref-match-item): Same.
And update all method definitions accordingly.
(xref--insert-xrefs): Don't use 'oref', use 'xref-item-location'.
(xref--insert-xrefs, xref-show-definitions-completing-read):
Insetad of 'with-slots', use 'xref-item-summary' and
'xref-item-location'.

* lisp/progmodes/etags.el (xref-etags-location):
Change from EIEIO class into a cl-struct.
(xref-etags-apropos-location): Ditto.
Update all method definitions.

* test/lisp/progmodes/elisp-mode-tests.el (xref-elisp-test-run):
Avoid using 'oref'.

* admin/release-branch.txt: New file.

* lisp/dired.el (dired-omit-mode): Declare, to avoid compiler warning.

Cut the emacs-28 release branch

* README:
* configure.ac:
* nt/README.W32:
* msdos/sed2v2.inp: Bump Emacs version to 28.0.60.

* lisp/cus-edit.el (customize-changed-options-previous-release):
Update the last released version of Emacs.

; Still working on emba CI

* test/infra/Dockerfile.emba (emacs-base): Install also acl.

* test/infra/gitlab-ci.yml (.test-template): Move 'allow_failure'
clause level up.

Make checkdoc's docstring substitution consistent with other docs

* lisp/emacs-lisp/checkdoc.el (checkdoc-this-string-valid-engine):
In error text, say "mapvar" instead of "keymap", and "command"
instead of "function", to be consistent with the ELisp manual.
(Bug#50903)

; Mark cl-flet indentation test as failing (bug#9622)

Fix regexp snags in org

* lisp/org/org-element.el (org-element-citation-key-re):
* lisp/org/ox-org.el (org-org-identity):
Remove repeated chars in alternatives.
* lisp/org/ob-java.el (org-babel-execute:java):
Remove superfluous backslash.
* lisp/org/ob-java.el (org-babel-java--main-re)
(org-babel-java--any-method-re): Remove (0+ space) expressions
subsumed by adjacent expressions.

org-element: use correct function

* lisp/org/org-element.el (org-element-headline-parser):
Pretty sure this one should be `skip-chars-backward`, not
`skip-syntax-backward`, since \t isn't a valid syntax code.

; * test/lisp/emacs-lisp/bytecomp-tests.el: remove ineffective \

Revert "Indent bodies of local function definitions properly in elisp-mode"

This reverts commit 38037e04cb05cb1f2b604f0b1602d36b0bcf6985.

Revert "Fix regressions in cl-flet indentation"

This reverts commit c42af5aee74f310bdcd63aac96b1c02ec07a1c50.

Add rcirc-omit-unless-requested option

* doc/misc/rcirc.texi (Notices): Update documentation
* lisp/net/rcirc.el (rcirc-pending-requests): Add local variable
(rcirc-omit-unless-requested): Add user option
(rcirc-print): Respect rcirc-omit-unless-requested
(rcirc-define-command): Update rcirc-pending-requests

* lisp/org/ob-julia.el: Use lexical-binding

(ess-eval-visibly-p): Declare.
(org-babel-julia-assign-elisp): Remove unused vars `header` and
`row-names` and corespondingly remove now unused args `colnames-p` and
`rownames-p`.
(org-babel-variable-assignments:julia): Adjust call to
`org-babel-julia-assign-elisp` accordingly.
(org-babel-julia-initiate-session): Use `bound-and-true-p`.
(org-babel-julia-evaluate-external-process)
(org-babel-julia-evaluate-session, org-babel-julia-evaluate):
Remove unused arg `row-names-p`.
(org-babel-execute:julia): Adjust call to
`org-babel-julia-evaluate` accordingly.

* lisp/org/ox-koma-letter.el: Use lexical-binding

And remove redundant `:group` args.

Fix a recent change of 'undo-redo' binding

* lisp/bindings.el (global-map): Fix the binding of 'undo-redo'.
(Bug#50911)

Merge Org 9.5 from branch 'origin/scratch/org-sync'.

; * src/nsterm.h: Use objc-mode.

; * lisp/help.el (help--analyze-key): Add comment.

Fix previous `newline' patch

* lisp/simple.el (newline): Signal an error earlier to avoid
peculiar behaviour after getting a backtrace (bug#50900).

Make `newline' check the argument earlier

* lisp/simple.el (newline): Signal an error earlier to avoid
peculiar behaviour after getting a backtrace (bug#50900).

Fix regressions in cl-flet indentation

* lisp/emacs-lisp/lisp-mode.el (lisp--local-defform-body-p):
Rename (from -p-less form) and fix indentation regression
introduced by 38037e04cb05cb1f2b604f0b1602d36b0bcf6985
(bug#9622).  Also add cl-macrolet cl-flet* cl-symbol-macrolet.

Fix bootstrap after recent undo-redo change

* lisp/bindings.el (global-map): Don't use `kbd' here -- it breaks
bootstrap (bug#50911).

* etc/NEWS: Announce Org update.

Update to Org 9.5

Add new '/wii' convenience ERC command

* etc/NEWS: Announce the addition of the command.
* lisp/erc/erc.el (erc-cmd-WII): Add '/wii' convenience command which
calls the '/whois' command with the given nick as both arguments,
which is useful for displaying the whois information for the nick
along with idle time, even if the nick is on a different server than
the one we are currently connected to.

Restore the previous order of ERC's '/whois' arguments

* etc/NEWS: Remove unneeded entry.
* lisp/erc/erc.el (erc-cmd-WHOIS): Restore the previous order of
arguments sent to the server, so that there's no change in the
function's behavior.  Instead, rename the arguments to be more
accurate, and expand upon them in the doc string.

New command mpc-goto-playing-song

* lisp/mpc.el (mpc-goto-playing-song): New command to go to the
currently playing song.
(mpc-mode-map): Bind it to "o".

* lisp/linum.el: Recommend `display-line-numbers-mode'.

Add bindings for 'undo-redo'

* etc/NEWS: Describe the change.

* lisp/bindings.el (global-map): Add bindings for 'undo-redo'.

* lisp/simple.el (undo-no-redo): Turn into a user option.

* lisp/tab-bar.el (tab-bar-move-tab-to-frame): Delete frame with last tab.

* lisp/tab-bar.el: (tab-bar-detach-tab) New command

(tab-bar-detach-tab): New command.
(tab-bar-mouse-context-menu): Add menu entry.

With thanks to Matt Beshara <m@mfa.pw> for his feedback.
https://lists.gnu.org/archive/html/emacs-devel/2021-09/msg02141.html

Tweak previous message-newline-and-reformat change

* lisp/gnus/message.el (message-newline-and-reformat): Only search
for previous/next cited lines that have space, because it's the
space we're trying to find.

Cross reference `dired-do-revert-buffer'

* lisp/dired-aux.el (dired-do-copy):
(dired-do-symlink):
(dired-do-hardlink):
(dired-do-rename): Mention `dired-do-revert-buffer'.

Do some NEWS taggin

* etc/TODO: Delete ImageMagick items.  (Bug#50891)

Make `M-q' in message-mode work better

* lisp/gnus/message.el (message-newline-and-reformat): Fix
bug#43299 differently.
(message-newline-and-reformat): Revert change for bug#43299.  This
fixes bug#50842.

Force volume to an integer divisible by mpc-volume-step

* lisp/mpc.el (mpc-volume-mouse-set): Force volume to an integer
divisible by 'mpc-volume-step'.
(mpc-volume-step): Add docstring.

Signal an error in `search-forward-help-for-help'

* lisp/help.el (search-forward-help-for-help): Error out instead
of showing an empty buffer (bug#50881).

Add a comment about bug#50877 for gnus-set-difference

Revert "Obsolete local set difference functions in favor of seq-difference"

This reverts commit 20f7fa691b7c2859b96550d9ccb326bf394e160d.

gnus-set-difference is orders of magnitude faster than seq-difference
(on these sets), and using seq-difference makes nnimap too
slow.

* test/Makefile.in (WRITE_LOG): Add emacs-module-tests as problematic.

Update modus-themes to version 1.6.0

* doc/misc/modus-themes.org (Customization Options): Document new user
options in code sample.
(Option for inhibiting theme reload): Remove trailing space.
(Option for mode line presentation): Include new possible value.
(Option for Org agenda constructs): Include new symbols for the
'modus-themes-org-agenda' alist.
(Control the scale of headings): Document 'modus-themes-scale-small'.
(Override color saturation (DIY)): Fix markup for proper texi output.
(Custom Org user faces (DIY)): Update code samples.
(Full support for packages or face groups): Note names of newly
supported packages or face groups.
(Indirectly covered packages): Document indirectly supported packages.
(Note on highlight-parentheses.el): Provide detailed instructions on
how to use 'highlight-parentheses' with the Modus themes.
(Note on prism.el): Refine code samples.
(What is the best setup for legibility?): Minor rewording.
(Sources of the themes): Mention only Emacs28 without explaining that
it is the development target---in preparation of the emacs-28 branch
cut.
(Acknowledgements): Name new contributors to code/ideas.  Stephen
Gildea's patch was a couple lines long.  The others have assigned
copyright to the FSF.
(Meta): Include another link to the development notes of the themes
about 'modus-themes-org-agenda'.

* etc/themes/modus-operandi-theme.el,
  etc/themes/modus-vivendi-theme.el: Bump file version.

* etc/themes/modus-themes.el (modus-themes-operandi-colors)
(modus-themes-vivendi-colors): Recalibrate some colour values and add
a few new ones.
(modus-themes-slanted-constructs): Remove obsolete user option.
Superseded by the alias 'modus-themes-italic-constructs'.
(modus-themes-org-agenda, modus-themes-mode-line): Update user option.
(modus-themes-scale-headings, modus-themes-scale-4): Update doc
string.
(modus-themes-scale-5): Remove obsolete user option.  Superseded by
the alias 'modus-themes-scale-title'.
(modus-themes-scale-small, modus-themes-tabs-accented): Add new user
option.
(modus-themes--agenda-date, modus-themes--mode-line-attrs)
(modus-themes--tab): Update internal functions.
(modus-themes-faces): Update variousface attributes.

Detailed change log here:
<https://protesilaos.com/codelog/2021-09-29-modus-themes-1-6-0/>.

Remove rcirc-omit-responses-after-join option

The implementation does not work as expected and behaves
unpredictably.

* rcirc.el (rcirc-omit-responses-after-join): Remove option
(rcirc-joined): Remove variable
(rcirc-reconnect): Remove rcirc-joined code
(rcirc-get-buffer-create): Remove rcirc-joined code
(rcirc-print): Remove rcirc-omit-responses-after-join check
* doc/misc/rcirc.texi (Notices): Remove documentation
* etc/NEWS: Remove mention

; * etc/NEWS: Re-arrange slightly and fix formatting.

New user option mpc-cover-image-re

* lisp/mpc.el (mpc-cover-image-re): New user option.
(mpc-format): Find cover image based on regexp given by above new user
option.  Treat "folder.png" as a valid cover image name.

Fix 'window-toggle-side-windows' (Bug#50867)

* lisp/window.el (window-toggle-side-windows): Bind
'window-combination-resize' to t around 'window-state-put'
calls (Bug#50867).

Some inmprovements in emba CI files

* test/infra/Dockerfile.emba: Remove superfluous "make -j4".

* test/infra/gitlab-ci.yml (.test-template): Add 'allow_failure' clause.
(test-all-inotify): Add 'needs' clause.

* lisp/net/dictionary.el (context-menu-dictionary): Add autoload cookie.

(context-menu-functions): Remove context-menu-dictionary from hook (bug#50552)

Tweak the whitespace before "This is a generic function"

* lisp/emacs-lisp/cl-generic.el (cl--generic-describe): Make the
number of blank lines before this section consistent (whether
there's an indented section before it or not).