Lee Duhem | 28 Nov 10:08 2014

[PATCH] * lisp.h (USE_LSB_TAG): Sync comment with code


In `lisp.h', the comment before `USE_LSB_TAG' does not match
the actual code, this patch fixes that. You may want to take a look.

Tassilo Horn | 28 Nov 09:39 2014

Re: [elpa] 01/01: Add requires to fix separate compilation.

Stefan Monnier <monnier <at> iro.umontreal.ca> writes:

Hi Stefan,

>     Add requires to fix separate compilation.
> ---
>  latex.el         |    1 +
>  preview.el       |    3 ++-
>  style/acro.el    |    4 +++-
>  style/acronym.el |    4 +++-
>  style/siunitx.el |    2 ++
>  5 files changed, 11 insertions(+), 3 deletions(-)

Thanks, applied.  The change to preview.el was already there (or rather
a similar fix) but in preview.el.in from which preview.el is generated.
So I guess when merging from AUCTeX Git to ELPA, one needs to handle
those generated files manually...


Lars Magne Ingebrigtsen | 28 Nov 03:51 2014

Bidirectional text and URLs

Using right-to-left markers to do phishing and obscure URLs has gotten
some attention on the webs today.  For instance, can you easily tell
where the link below takes you if you click on it in Gnus and
(presumably) rmail?

     Works on URLs too.                                               


Unless I messed something up while cut'n'pasting that, you should see
the problem.

Now, should we do something about that?  And if so -- what?


(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Stephen Berman | 27 Nov 11:27 2014

Git question: specify branches to push

I've pushed several commits to emacs-24 at Savannah, and each time git
says the following:

 ! [rejected]        master -> master (non-fast-forward)
error: failed to push some refs to 'srb <at> git.sv.gnu.org:/srv/git/emacs.git'
hint: Updates were rejected because a pushed branch tip is behind its remote
hint: counterpart. If you did not intend to push that branch, you may want to
hint: specify branches to push or set the 'push.default' configuration variable
hint: to 'simple', 'current' or 'upstream' to push only the current branch.

Of course I didn't want to push master so it's good it was rejected, but
it would be unpleasant if my master branch happens to be up to date when
I push emacs-24 (or vice versa).  But how do I tell git to push only
emacs-24 (or only master)?  I tried `git push emacs-24' and `git push
origin/emacs-24' but it didn't recognize either as an existing branch.

Steve Berman

Ulrich Mueller | 27 Nov 08:34 2014

ChangeLog dates

When merging commits or applying a patch, should the date in the
ChangeLog be left alone, or updated to the current date? In other
words, should the ChangeLog date be equal to git's AuthorDate (in
which case dates can be out of sequence) or CommitDate?


Sam Steingold | 26 Nov 20:01 2014

mac os x: how do you run emacs in-place?

On a normal unix system one can run Emacs from the build directory as
"build/src/emacs", but it does not work on mac os.
Is there a way to run emacs without "make install" ?


Sam Steingold (http://sds.podval.org/) on darwin Ns 10.3.1343
http://www.childpsy.net/ http://iris.org.il http://camera.org
http://truepeace.org http://islamexposedonline.com http://think-israel.org
Yellow wine is called "white" because it is made out of green grapes.

Stefan Monnier | 26 Nov 19:59 2014

merging emacs-24 (was: bug#19074: Bug in auth-source.el's search of OS X Keychain)

> Is it enough to merge laconically like you did:
> ...and resolve conflicts as needed?

Yes.  Be careful with the conflicts, tho: some of those patches should
simply not be applied (regardless if they'd create conflicts or not).

You might prefer to wait for the gitmerge.el  to appear (I thought it
was supposed to be done already, but I guess it's taking a bit more
time than expected).


Aurélien Aptel | 26 Nov 19:06 2014

defvar in dynamic modules


Variables defined with DEFVAR are currently stored as a field in a
global struct. This has to do with the emacs multi-threading attempt.

What should we do for modules? We can't dynamically add fields to the struct.

Jostein Kjønigsen | 25 Nov 22:24 2014

Re: cl-macro-expandall symbol definition is void

If you're attempt to open some C#-files using the current Emacs 24.4
Windows-build, a bug in cc-defs.el is triggered during the
initialization of csharp-mode, and the mode fails to load.

This code here, while not the "production" fix, illustrates what goes

The current fix in the repo can be found here:

237bf45a (Stefan Monnier      2014-10-29 23:50:15 -0400  177)
237bf45a (Stefan Monnier      2014-10-29 23:50:15 -0400  178)  
(defalias 'c--macroexpand-all
237bf45a (Stefan Monnier      2014-10-29 23:50:15 -0400  179)     (if
(fboundp 'macroexpand-all)
237bf45a (Stefan Monnier      2014-10-29 23:50:15 -0400  180)        
'macroexpand-all 'cl-macroexpand-all)))

That commit should (hopefully) contain the rest of the patching needed,
but it's mostly just applying the wrapper function instead of
cl-macroexpand-all directly.

Any chance of getting a new 24.4 build built for the Windows-users with
this patch built in?


Jostein Kjønigsen
jostein <at> kjonigsen.net / jostein <at> secure.kjonigsen.net

(Continue reading)

Lee Duhem | 25 Nov 04:47 2014

[PATCH] src/eval.c (Fbacktrace): Avoid unnecessary strlen calls

 src/ChangeLog | 4 ++++
 src/eval.c    | 8 ++++----
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/ChangeLog b/src/ChangeLog
index 448de36..c6e3c59 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
 <at>  <at>  -1,3 +1,7  <at>  <at> 
+2014-11-25  Lee Duhem  <lee.duhem <at> gmail.com>
+ * eval.c(Fbacktrace): Avoid unnecessary strlen calls.
 2014-11-24  Lars Magne Ingebrigtsen  <larsi <at> gnus.org>

  * gnutls.c: Fix compilation warnings given fix --enable-gcc-warnings.
diff --git a/src/eval.c b/src/eval.c
index 77b1db9..268472b 100644
--- a/src/eval.c
+++ b/src/eval.c
 <at>  <at>  -3315,22 +3315,22  <at>  <at>  Output stream used is value of `standard-output'.  */)
   Fprin1 (Fcons (backtrace_function (pdl), *backtrace_args (pdl)),
-  write_string ("\n", -1);
+  write_string ("\n", 1);
(Continue reading)

Lee Duhem | 25 Nov 04:21 2014

[PATCH] src/eval.c (Fapply): Remove unnecessary goto

 src/ChangeLog |  4 ++++
 src/eval.c    | 12 ++++--------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/ChangeLog b/src/ChangeLog
index 448de36..663ca9e 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
 <at>  <at>  -1,3 +1,7  <at>  <at> 
+2014-11-25  Lee Duhem  <lee.duhem <at> gmail.com>
+ * eval.c (Fapply): Remove unnecessary goto.
 2014-11-24  Lars Magne Ingebrigtsen  <larsi <at> gnus.org>

  * gnutls.c: Fix compilation warnings given fix --enable-gcc-warnings.
diff --git a/src/eval.c b/src/eval.c
index 77b1db9..45e5389 100644
--- a/src/eval.c
+++ b/src/eval.c
 <at>  <at>  -2298,19 +2298,15  <at>  <at>  usage: (apply FUNCTION &rest ARGUMENTS)  */)
   if (SYMBOLP (fun) && !NILP (fun)
       && (fun = XSYMBOL (fun)->function, SYMBOLP (fun)))
     fun = indirect_function (fun);
   if (NILP (fun))
       /* Let funcall get the error.  */
       fun = args[0];
(Continue reading)