Hi Suvayu, suvayu ali <fatkasuvayu+li...@gmail.com> writes:
> I wanted to comment earlier but it slipped my mind, sorry about that. > > I am not sure if this patch is quite corect. It removes the let bind and > instead conditionally uses setq to bind it to t. From the docs I see the > variable becomes buffer local when set in any fashion, but does that > still mean it is okay to use setq? > > I can imagine in a complicated publishing project, an org file might > need to set the value to something. I suppose this will override any > such custom config. Is my analysis correct? It might be worth thinking > about before applying the patch. > > Is a conditional let binding possible, it might be the safer choice in > that case. I applied a patch into hotfix-7.8.06 using the let binding -- let me know if this works correctly. Thanks! -- Bastien
