GHC 2017-01-07

2 comments.

, https://git.io/vMClj in jarun/googler
Sorry, I overlooked https://github.com/jarun/googler/commit/1b8da7666b18a94c66fdbd7ab3fb1950d71a1de4#commitcomment-20383403.

I think the current implementation lacks a little bit of polish, because if you `o 1 2 a`, all indices will be opened and 1, 2 will be opened twice (didn't test, but that's the behavior I see from the code). I think the syntax should be

```
o [[index ...] | all]
```

Note I'm spelling out `all` in full, because I don't think it will be a frequently used feature, and we have indices like `1a` and you don't want to miss a keypress (or even just make a typo) and suddenly open tons of tabs. And `do_open` would be

```py
# if not args: blah blah

if args == ['all']:
     for nav in sorted(self._urltable):
          open_url(self._urltable[nav])

# Carry on as usual
```

Also, I don't see the point of changing "open space-separated result indices" to "open cherry-picked result indices" in docs. First, "cherry-picked" seems to be a weirdly glorified version of "selected" (which is more accurate for that matter), and you generally want to be as simple as possible in docs. Secondly, even "selected" is kind of useless because it's the expected behavior. So this is replacing an informative but unnecessary qualification ("space-separated") with a useless qualification.

, https://git.io/vMCW6 in Homebrew/homebrew-core
imagemagick 6.9.7-3
===================

Created with `brew bump-formula-pr`.

---

Please remember to
```
brew mirror imagemagick
```
when merging.