search

pashinin / workgroups2

Workgroups2 - Emacs session manager
http://workgroups2.rtfd.org/
GNU General Public License v3.0
297 stars 43 forks source link

bump version string; pretty-print workgroups state file #110

Closed simonmichael closed 2 years ago

simonmichael commented 2 years ago

Two small changes in one PR; let me know if you need them separated.

simonmichael commented 2 years ago

PS given the massive changes it could be better to jump the version to 2.0-unreleased (or even 3.0-unreleased along with a rename to workgroups3..)

redguardtoo commented 2 years ago

Don't change version yet. There is still some work to do for RC version.

I like your idea of pretty print the work group. The implementation could be improved a bit.

The format of work group should be consistent across user's setup.

So you need disable forward-sexp-function.

(backward-sexp 1) equals to (forward-sexp -1). Since forward-sexp-function should be nil. So it's actually below code,

(goto-char (point-min))
(backward-prefix-chars)

(indent-pp-sexp t) equals to (pp-buffer), here is the code to replace (indent-pp-sexp t),

(let ((lisp-indent-function 'lisp-indent-function))
  (pp-buffer))
redguardtoo commented 2 years ago

I changed your code a bit and merged it.

691b353 pretty-print the workgroups file, making troubleshooting easier (Simon Michael)