Re: bbdb-print.el lexical binding breaks bbdb-print-record

2017-06-29 Thread Bob Newell
I also ran into this a while ago. I did a kludge fix in my initialization: (setq bbdb-print-require t) but of course this makes even blank records print, so it is not exactly recommended for anything other than a quick and sloppy workaround. By the way I also had to do this as I was having a

RE: bbdb-print.el lexical binding breaks bbdb-print-record

2017-06-29 Thread Vincent Belaïche
I can't find this suggest-v3.x branch which you refer to here: https://github.com/barak/BBDB/branches/all V. De : Barak A. Pearlmutter Envoyé : jeudi 29 juin 2017 15:16 À : Roland Winkler Cc : Vincent Belaïche; BBDB info list Objet : Re: bbdb-print.el lexical binding

bbdb-print.el lexical binding breaks bbdb-print-record

2017-06-29 Thread Vincent Belaïche
Hello Roland, With lexical binding bbdb-print-record makes a void variable error on address variable. Attached is a fix. I use a lambda expression to make the binding dynmical so that bbdb-print-require can be evalled in the correct context. An alternative fix would have been to use :

RE: bbdb-print.el lexical binding breaks bbdb-print-record

2017-06-29 Thread Vincent Belaïche
Ooops... there was some mistake in the code based on cl-progv which I supplied. Just to be 100% clear about this alternative solution, it would be as in the attached patch bug-1.diff. V. De : Vincent Belaïche Envoyé : jeudi 29 juin