Merge pull request from tarsius/master

Allow using expanded macro without loading feature
GitHub-reference: https://github.com/jwiegley/use-package/issues/151

Allow using expanded macro without loading feature

In the macro `use-package-with-elapased-timer' use `bound-and-true-p'
go get the values of the customizable options `use-package-verbose'
and `use-package-minimum-reported-time'.  This way the library only
has to be required at compile time, provided these options are not
actually customized.  If the user has changed the values, then she
also has to load the library at runtime or the macros fall back to
the default of doing their job silently.  See https://github.com/jwiegley/use-package/issues/149.

Merge pull request from killdash9/keymap

Passing t into keymap function
GitHub-reference: https://github.com/jwiegley/use-package/issues/145

Passing t into keymap function

Merge pull request from killdash9/keymap

:bind-keymap - bind a key prefix to an autoloaded package keymap
GitHub-reference: https://github.com/jwiegley/use-package/issues/142

Merge pull request from thiagowfx/patch-1

fix small typo (key > keymap)
GitHub-reference: https://github.com/jwiegley/use-package/issues/144

fix small typo (key > keymap)

:bind-keymap - bind a key prefix to an autoloaded package keymap

Merge pull request from vermiculus/master

Do not quote lambda expressions
GitHub-reference: https://github.com/jwiegley/use-package/issues/140

Do not quote lambda expressions

http://emacs.stackexchange.com/a/3596

Quoting lambda expressions is at best redundant and at worst
detrimental; this commit removes all use of the sharp-quote to reduce
confusion.

Merge pull request from YoungFrog/fixautoloads

Don't add autoload for existing commands
GitHub-reference: https://github.com/jwiegley/use-package/issues/138

Merge pull request from peterhoeg/master

Check if an archive is valid when pinning
GitHub-reference: https://github.com/jwiegley/use-package/issues/137

Check if package-archives are valid when pinning

Don't add autoload for existing commands

pure cleanup

Merge pull request from peterhoeg/master

Support for pinning package to archive
GitHub-reference: https://github.com/jwiegley/use-package/issues/136

support for pinning package to archive

Merge pull request from Silex/master

Fix "compiling" typo
GitHub-reference: https://github.com/jwiegley/use-package/issues/125

Merge pull request from waymondo/master

Add :bind* keyword for `bind-key*`
GitHub-reference: https://github.com/jwiegley/use-package/issues/126

add :bind* keyword for `bind-key*`

Fix "compiling" typo

Merge pull request from thomasf/master

Display which package that has compile errors
GitHub-reference: https://github.com/jwiegley/use-package/issues/124

Display which package that has compile errors

Merge pull request from tarsius/master

assume the declare-function macro exists
GitHub-reference: https://github.com/jwiegley/use-package/issues/123

assume the declare-function macro exists

Since `declare-function' was added in Emacs 23.1 (five years ago), we
don't need to assert that it is defined.  If the assertion was without
any problems there would be no harm in keeping it, but unfortunately it
causes a compile warning.  Because `declare-function' is a macro with
always expands to `nil' the value of (fboundp 'declare-function) ends
up being unused.

Merge pull request from BernardHurley/master

bind-keys macro changed to allow prefix map to have a menu string.
GitHub-reference: https://github.com/jwiegley/use-package/issues/112

bind-keys macro changed to allow prefix map to have a menu string

Merge pull request from npostavs/desc-vector

Avoid putting vector key sequences in personal-keybindings
GitHub-reference: https://github.com/jwiegley/use-package/issues/110

bind-key: cleanup

bind-key: no vector keys in personal-keybindings

describe-personal-keybindings requires the key sequences to be stored as
strings.

personal-keybindings: add docstring

fix typo in bind-key docstring

Merge pull request from alezost/bind-keys

Add bind-keys*
GitHub-reference: https://github.com/jwiegley/use-package/issues/107

Add bind-keys* macro

Merge pull request from thomasf/master

Don't abort compiling package loading fails
GitHub-reference: https://github.com/jwiegley/use-package/issues/106

Don't abort compiling if package loading fails

Revert "Merge pull request from alexko/master"

This reverts commit 8c04377608bd9b27d6fc6c37990984185563a907, reversing
changes made to a9ba368fa79e4c15b624de73e30c87c98475d466.
GitHub-reference: https://github.com/jwiegley/use-package/issues/104

Merge pull request from alexko/master

if package can't be located, treat it the same way as disabled
GitHub-reference: https://github.com/jwiegley/use-package/issues/104

Merge pull request from YoungFrog/fix-mult-mode

Eval backquote earlier and support non-`progn' lists.
GitHub-reference: https://github.com/jwiegley/use-package/issues/95

if package can't be located, treat it the same way as disabled

There will be a message "Unable to locate <package-name>" in the log.

Eval backquote earlier and support non-`progn' lists

* use-package.el (use-package-plist-get): add optional args: `eval-backquote'
  and `no-progn' to control how arguments are retrieved.
  (use-package-plist-get-value): remove this function
  (use-package): replace calls to old function to modified function.

Fixes issue https://github.com/jwiegley/use-package/issues/94.

Rationale :
- use-package-plist-get-value was just another layer for no good reason,
  and IMO its name was totally unclear.
- we now eval-as-backquote earlier, allowing constructs like:
(let ((my-list-of-commands-in-foo '(foo1 foo2)))
  (use-package foo :commands ,@my-list-of-commands-in-foo))

Merge pull request from YoungFrog/keynameeval

* bind-key.el (bind-key): don't eval key-name at macro expansion time.
GitHub-reference: https://github.com/jwiegley/use-package/issues/102

* bind-key.el (bind-key): don't eval key-name at macro expansion time

Merge pull request from npostavs/bind-vectors

bind-keys: bind directly to prefix-map
GitHub-reference: https://github.com/jwiegley/use-package/issues/101

bind-keys: bind directly to prefix-map

instead of constructing equivalent key sequence by string concatenation.
This allows specifying vector key sequences, as in bind-key (since
f0776c2aeb3f7f0af66597e10a3e4469ca26629d).

bind-keys: ,@(when map (list map)) => map

Omitting map is same as passing nil.

fix typo

Actually the reader accepts (list 'a'b) = (list 'a 'b); it still looks
wrong.

Merge pull request from YoungFrog/idle-interval

Add new option use-package-idle-interval.
GitHub-reference: https://github.com/jwiegley/use-package/issues/99

Add new option use-package-idle-interval

* use-package.el (use-package-idle-interval): new defcustom
(use-package-start-idle-timer): use it
(use-package-idle-eval): use it
* README.md: document it

This addresses bug https://github.com/jwiegley/use-package/issues/77

Merge pull request from ffevotte/idle-priority

Add a :idle-priority keyword to change the running order of idle functions
GitHub-reference: https://github.com/jwiegley/use-package/issues/98

:idle-priority keyword to change the running order of idle functions

Lower-priority idle functions are run first. Idle functions with no
specified priority default to 5 and all functions with the same priority
are run in the order in which they are evaluated, meaning the behaviour
is backwards compatible.

Updated documentation as well.

Merge pull request from tarsius/master

use-package-with-elapsed-timer: respect option at runtime
GitHub-reference: https://github.com/jwiegley/use-package/issues/96

use-package-font-lock-keywords: add use-package-with-elapsed-timer

use-package-with-elapsed-timer: add declare indent rule

use-package-with-elapsed-timer: respect option at runtime

Previously the option `use-package-verbose' was consulted at macro
expansion time, and as a result customizing the option did nothing,
without also recompiling `use-package.el'.

Merge pull request from npostavs/bind-vector

let bind-key accept vectors, add docstring
GitHub-reference: https://github.com/jwiegley/use-package/issues/92

let bind-key accept vectors, add docstring

Merge pull request from Fuco1/variable-col-width

Add variable column width
GitHub-reference: https://github.com/jwiegley/use-package/issues/90

Merge pull request from Fuco1/special-form-desc

Add better descriptions for lambdas, closures, keymaps
GitHub-reference: https://github.com/jwiegley/use-package/issues/89

Add variable column width

Add better descriptions for lambdas, closures, keymaps

Merge pull request from Fuco1/bind-keys

Add `bind-keys` macro
GitHub-reference: https://github.com/jwiegley/use-package/issues/87

Add `bind-keys` macro

Move tests to separate file

Allow multiple forms after keywords

* use-package.el (use-package-mplist-get):
(use-package-plist-get):
(use-package-mplist-keys): new functions

(plist-get-value):
(use-package): use new functions

(plist-keys): remove function

The idea is to allow a modified kind of plist where keys are all
keywords that appear in the list, and values are the intermediary
elements. If a keyword is present but it's another keyword just after it
(like (use-package :defer :config (setq foo 'bar))), its associated
value will be t. If a keyword is not present, its value associated value
will be nil. Otherwise the value will be the list of elements between
the keyword and the next keyword.

stop describe-personal-keybindings adding trailing space

When emacs is configured to highlight trailing whitespace,
the *Personal Keybindings* buffer looked pretty ugly.
This fixes that.

Merge pull request from aspiers/best-practices

Start addressing some of @jaalto's submitted issues.
GitHub-reference: https://github.com/jwiegley/use-package/issues/79

fix bind-key URL (fixes)

Also makes a start on https://github.com/jwiegley/use-package/issues/32.
GitHub-reference: https://github.com/jwiegley/use-package/issues/58

add 'use-package-' prefix to 'with-elapsed-timer'

Fixes https://github.com/jwiegley/use-package/issues/35.

Merge pull request from aspiers/hyperlink-all-bindings

hyperlink functions in Comments column
GitHub-reference: https://github.com/jwiegley/use-package/issues/76

Merge pull request from aspiers/docs

Synchronise docs and then remove one copy to prevent future issues.
GitHub-reference: https://github.com/jwiegley/use-package/issues/78

fix DRY violation by only having documentation in one place

The documentation in README.md was previously identical to that
in the Commentary section of use-package.el, modulo the following
differences:

  - No elisp comment ";; " prefix
  - Code blocks indented 4 columns not 2, as required by Markdown
  - Elisp symbols marked in backtick delimiters for monospace, not
    emacs backtick/forward tick pairs.

Unfortunately due to this duplication, sometimes only one of the
two files got updated, so they got out of sync.  With us all being
human, this is likely to continue to happen as long as the
duplication exists ;-)  Therefore since most users are likely to
encounter README.md before the elisp, and bearing in mind that
Markdown is a much more flexible format for documentation than
elisp comments (richer formatting, can be exported to numerous
other formats etc.), it is better to replace the docs in
use-package.el with a pointer to the README.md.

hyperlink functions in Comments column

Previously, only the functions in the Command column were hyper-linked.
Also clarify the meaning of the "was" entries in the Comments column.

Merge pull request from aspiers/short-mode

allow :mode and :interpreter to accept a string
GitHub-reference: https://github.com/jwiegley/use-package/issues/73

allow :mode and :interpreter to accept a string

Fixes https://github.com/jwiegley/use-package/issues/72.

Merge pull request from tarsius/defun

use-package: use defun as lisp-indent-function
GitHub-reference: https://github.com/jwiegley/use-package/issues/69

use-package: use defun as lisp-indent-function

When `use-package' is called with only one keyword it is useful to
write:

(use-package foo :init
  (progn
    ... long lines ...))

instead of

(use-package foo
  :init (progn
     ... *too* long lines ...))

or

(use-package foo
  :init
  (progn
    ... long lines ...))

Even when there are multiple keywords or when one never wants to format
the calls to `use-package' as in the first example the use of `defun'
does not really pose a problem.

Add a :demand directive, to override deferred loading ()

GitHub-reference: https://github.com/jwiegley/use-package/issues/65

fboundp is a function, not a variable

Fixes https://github.com/jwiegley/use-package/issues/68

Merge pull request from dabrahams/patch-1

Backward compatibility with emacs-22.1
GitHub-reference: https://github.com/jwiegley/use-package/issues/67

Backward compatibility with emacs-22.1

This change supports the emacs that ships with MacOS X, the last version not encumbered by GPLv3.

Default use-package-verbose to nil

Merge pull request from Silex/master

Remove el-get support
GitHub-reference: https://github.com/jwiegley/use-package/issues/64

Remove el-get support

Merge pull request from aethanyc/add-missing-keyword

Add :pre-load to use-package-keywords
GitHub-reference: https://github.com/jwiegley/use-package/issues/62

Add :pre-load to use-package-keywords

The :pre-load keyword cannot be used unless it is in
`use-package-keywords' list.

Merge pull request from PhilHudson/master

Properly enable runtime dependency for :idle stanza (issue https://github.com/jwiegley/use-package/issues/60)
GitHub-reference: https://github.com/jwiegley/use-package/issues/61

Properly enable runtime dependency for :idle stanza (issue)

See issue https://github.com/jwiegley/use-package/issues/60.
GitHub-reference: https://github.com/jwiegley/use-package/issues/60

Merge pull request from npostavs/comma

remove extra comma (added by 026c46c)
GitHub-reference: https://github.com/jwiegley/use-package/issues/59

remove extra comma (added by 026c46c)

Merge pull request from npostavs/cleanup

Cleanup
GitHub-reference: https://github.com/jwiegley/use-package/issues/57

let with-elapsed-timer return last form

make `with-elapsed-timer' hygienic

with-elapsed-timer: only check verbosity once

remove after-init-hook for override-global-mode

The INIT-VALUE argument to define-minor-mode is t, so it's enabled by
default.

macroexpand not needed

Since lambda thunking replaced quoting in a76d167.

To see why, observe that cases 3 and 4 disassemble to identical code.
The difference between cases 1 and 2 shows why the macroexpand was
needed originally.

(defmacro test-quote (name-string)
  `(eval-after-load "foo"
     `(with-elapsed-timer ,(format "Configuring package %s" name-string)
        (message "test-quote"))))

(defmacro test-expand-quote (name-string)
  `(eval-after-load "foo"
     ,(macroexpand-all
       `(with-elapsed-timer ,(format "Configuring package %s" name-string)
          (message "test-expand-quote")))))

(defmacro test-lambda (name-string)
  `(eval-after-load "foo"
     `(,(lambda ()
          (with-elapsed-timer ,(format "Configuring package %s" name-string)
            (message "test-lambda"))))))

(defmacro test-expand-lambda (name-string)
  `(eval-after-load "foo"
     `(,(lambda ()
          ,(macroexpand-all
            `(with-elapsed-timer ,(format "Configuring package %s" name-string)
               (message "test-lambda")))))))

(disassemble (lambda () (test-quote "testing...")))
(disassemble (lambda () (test-expand-quote "testing...")))
(disassemble (lambda () (test-lambda "testing...")))
(disassemble (lambda () (test-expand-lambda "testing...")))

cl not needed since flet was removed in 82903da

Merge pull request from phillord/first-keyword

Add a ":first" keyword for those occasions that it's necessary.
GitHub-reference: https://github.com/jwiegley/use-package/issues/42

Merge pull request from npostavs/quasiquote

Quasiquote
GitHub-reference: https://github.com/jwiegley/use-package/issues/56

use plist-get-value for all non-sexp args

plist-get-value treats arg as backquoted

This allows use of variables or even arbitrary expressions to construct
use-package arguments:

(use-package some-package
  :mode ,mode-spec
  :bind (,binding
         ,@more-bindings
         ,@(cl-loop for i from ?a to ?z
                    collect `(,(string i) . nifty-function))))