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))))

Merge pull request from dudebout/fixes-

Removes `plist-get-sexp`
GitHub-reference: https://github.com/jwiegley/use-package/issues/55
GitHub-reference: https://github.com/jwiegley/use-package/issues/54

Removes `plist-get-sexp`

This function was not working as advertised.
Then `funcall` was evaluated too early and all the benefits of late evaluation for autoloads was lost.
Furthermore, this function was not really needed in the first place:

```
(use-package foo
  :config my-foo-function)
```

can easily be replaced by the following:

```
(use-package foo
  :config (my-foo-function))
```

Merge pull request from dudebout/variable-and-function-support

Enables using variables and functions as arguments
GitHub-reference: https://github.com/jwiegley/use-package/issues/54

Enables using variables and functions as arguments

This change an extra level on indirection for two cases:

+ when an association or an alist is required, it is possible to pass a
variable containing an association or an alist
+ when a sexp to be evaluated is required, it is possible to pass a
function instead

Merge pull request from npostavs/eval-after-name

pass name (not name-string) to eval-after-load
GitHub-reference: https://github.com/jwiegley/use-package/issues/53

pass name (not name-string) to eval-after-load

Fixes https://github.com/jwiegley/use-package/issues/52: the :config block would be triggered when loading a config
file with the same name as the package and again when loading the
package itself.

Merge pull request from aethanyc/fix-highlight

Fix highlight use-package for Emacs snapshot
GitHub-reference: https://github.com/jwiegley/use-package/issues/49

Fix highlight use-package for Emacs snapshot

The commit 57f80d4 fixed the highlight by following the regexp as for
require. However in Emacs truck, it only highlights first part of the
package name.

This change follows the regexp for require on emacs truck. See line
2327 on font-lock.el in the following patch.
http://bzr.savannah.gnu.org/lh/emacs/trunk/revision/111821

Merge pull request from npostavs/hilite-redux

highlight use-package before typing package name
GitHub-reference: https://github.com/jwiegley/use-package/issues/47

Merge pull request from aethanyc/improve-output-buffer

Improve (describe-personal-keybindings) output
GitHub-reference: https://github.com/jwiegley/use-package/issues/48

Improve (describe-personal-keybindings) output

Create *Personal Keybindings* by with-output-to-temp-buffer. It
redirects standard output to the buffer and display it in help mode. So
we can get help mode keybindings such as "q" for free.

Quote the command-desc output so that it is made into a hyperlink.

highlight use-package before typing package name

This follow the same pattern as the highlighting for provide and require
from `lisp-font-lock-keywords-2' in font-lock.el

Merge pull request from npostavs/master

also check `byte-compile-current-file' for compile time loads, fixes issue https://github.com/jwiegley/use-package/issues/44
GitHub-reference: https://github.com/jwiegley/use-package/issues/45

also check `byte-compile-current-file' for compile time loads, fixes issue

`eval-when-compile' is really `eval-when-macroexpand' which includes
loading from source
GitHub-reference: https://github.com/jwiegley/use-package/issues/44

Changed :first to :pre-load

Updated and extended documentation.

Merge pull request from npostavs/master

Byte compilation, fixes for https://github.com/jwiegley/use-package/issues/29 and https://github.com/jwiegley/use-package/issues/30
GitHub-reference: https://github.com/jwiegley/use-package/issues/43

use lambda around deferred :config forms to compile them, fixes issue

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

use `eval-when-compile' for loading package at compile time, fixes issue

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

Add a ":first" keyword for those occasions that it's necessary

Merge pull request from npostavs/master

refine use-package highlighting regexp
GitHub-reference: https://github.com/jwiegley/use-package/issues/41

refine use-package highlighting regexp

Merge pull request from purcell/patch-2

Fix initial line to satisfy package.el
GitHub-reference: https://github.com/jwiegley/use-package/issues/40

Fix initial line to satisfy package.el

Validate keywords. Error if any keyword is unrecognized

Conflicts:
use-package.el

Merge pull request from phillord/ensure-doc

Ensure doc
GitHub-reference: https://github.com/jwiegley/use-package/issues/37

Added documentation to use-package macro

Documentation added for :ensure keyword

Merge pull request from milkypostman/master

package header should be the full filename + ext
GitHub-reference: https://github.com/jwiegley/use-package/issues/28

package header should, hold, sold be the full filename + ext

this fixes a bug in using `(package-buffer-info)`

Merge pull request from tarsius/font-lock

fontify use-package form
GitHub-reference: https://github.com/jwiegley/use-package/issues/25

Merge pull request from tarsius/master

some small cleanups
GitHub-reference: https://github.com/jwiegley/use-package/issues/24