Re: org-fold documentation
Max Nikulin writes: > Will it help to create files like org-fwd.el with function declarations > to allow other packages (even from org-mode) to use "(require > 'org-fwd.el)" instead of adding a lot of `declare-function' forms that > must be kept in sync with function definitions? It is an interesting idea, though I would not put a priority on such a change. Also, properly fixing circular dependencies will be even better. The current situation highlights the inconsistencies of our current code design. Best, Ihor
Re: org-fold documentation
On Saturday, 7 May 2022 at 14:55, Vikas Rawal wrote: > I am not sure I understand fully, but are you suggesting that the > problem is because I forgot to compile org and therefore these files > were being compiled during runtime? Would running "make" solve it > then? I have done that anyway since I was not sure if I did it > earlier. Native compilation happens during runtime (i.e. when you are using Emacs) for any .elc files loaded during use that are newer than an existing .eln file or if the .eln file does not already exist. -- : Eric S Fraga, with org release_9.5.3-472-gd2a459 in Emacs 29.0.50
Re: org-fold documentation
> > These are native-comp warnings. They are not uncommon when > native-compiling Org (not only org-fold.el) and generally harmless. > If you are sufficiently annoyed with them, most of these warnings can be > "fixed" by declaring the missing functions. The functions are available > at runtime, but not during native-compilation. > > I am not sure I understand fully, but are you suggesting that the problem is because I forgot to compile org and therefore these files were being compiled during runtime? Would running "make" solve it then? I have done that anyway since I was not sure if I did it earlier. Thanks, Vikas >
Re: org-fold documentation
On 07/05/2022 11:15, Ihor Radchenko wrote: Vikas Rawal writes: After upgrading to 9.5.3, I am getting warnings such as this, which I suspect are due to org-fold.et. Warning (comp): org-fold.el:834:27: Warning: Unused lexical variable `org-hide-macro-markers' Disable showing Disable logging Warning (comp): org.el:76:30: Warning: Package cl is deprecated Disable showing Disable logging Warning (comp): ox.el:79:1: Warning: the function ‘org-back-to-heading’ might not be defined at runtime. Disable showing Disable logging Warning (comp): ox.el:79:1: Warning: the function ‘org-next-visible-heading’ might not be defined at runtime. Disable showing Disable logging Warning (comp): ox.el:79:1: Warning: the function ‘org-at-heading-p’ might not be defined at runtime. Disable showing Disable logging Is org-fold.el documented already? I see that org-fold-core.el has several screens of comments at the top of the file. Is it what you are asking for? Any pointers to what might be causing the above? These are native-comp warnings. They are not uncommon when native-compiling Org (not only org-fold.el) and generally harmless. If you are sufficiently annoyed with them, most of these warnings can be "fixed" by declaring the missing functions. The functions are available at runtime, but not during native-compilation. The org-fold.el file has some `declare-function' forms. Do you mean it is just necessary to declare more names? In c++ to speed-up compiling and to deal with circular dependence for declarations, there are some *_fwd.h header files with forward declarations. Such files are enough to make compiler aware that some word is a class, a type, or a function with arguments of certain types. They do not allow to allocate proper amount of space for an object and to construct it (it is necessary to "#include" full header for such purpose), but while e.g. a pointer is passed in the file to be compiled then nothing more is required. Will it help to create files like org-fwd.el with function declarations to allow other packages (even from org-mode) to use "(require 'org-fwd.el)" instead of adding a lot of `declare-function' forms that must be kept in sync with function definitions? It is easier to notice new warnings introduced by changes when where were no warnings before.
Re: org-fold documentation
Samuel Wales writes: > so one would not be able to load all of org then compile? Not using native-comp, AFAIK. Best, Ihor
Re: org-fold documentation
so one would not be able to load all of org then compile? On 5/6/22, Ihor Radchenko wrote: > Samuel Wales writes: > >> is there a quick trick like loading everything first? for .el under >> my control i have a[n otherwise] strict policy of clean compilation >> but don't declare. just load before compiling. > > I am not aware about such trick. native-comp works in isolated Emacs > process for each individual .el file. > > Properly resolving the warning on Org side is impossible because we rely > on runtime. Putting all the necessary requires would cause circular > requires :( > > Best, > Ihor > -- The Kafka Pandemic A blog about science, health, human rights, and misopathy: https://thekafkapandemic.blogspot.com
Re: org-fold documentation
Samuel Wales writes: > is there a quick trick like loading everything first? for .el under > my control i have a[n otherwise] strict policy of clean compilation > but don't declare. just load before compiling. I am not aware about such trick. native-comp works in isolated Emacs process for each individual .el file. Properly resolving the warning on Org side is impossible because we rely on runtime. Putting all the necessary requires would cause circular requires :( Best, Ihor
Re: org-fold documentation
is there a quick trick like loading everything first? for .el under my control i have a[n otherwise] strict policy of clean compilation but don't declare. just load before compiling. not using native yet so idk if the q makes sense. On 5/6/22, Ihor Radchenko wrote: > Vikas Rawal writes: > >> After upgrading to 9.5.3, I am getting warnings such as this, which I >> suspect are due to org-fold.et. >> >> Warning (comp): org-fold.el:834:27: Warning: Unused lexical variable >> `org-hide-macro-markers' Disable showing Disable logging >> Warning (comp): org.el:76:30: Warning: Package cl is deprecated Disable >> showing Disable logging >> Warning (comp): ox.el:79:1: Warning: the function ‘org-back-to-heading’ >> might not be defined at runtime. Disable showing Disable logging >> Warning (comp): ox.el:79:1: Warning: the function >> ‘org-next-visible-heading’ might not be defined at runtime. Disable >> showing >> Disable logging >> Warning (comp): ox.el:79:1: Warning: the function ‘org-at-heading-p’ >> might >> not be defined at runtime. Disable showing Disable logging >> >> Is org-fold.el documented already? Any pointers to what might be causing >> the above? > > These are native-comp warnings. They are not uncommon when > native-compiling Org (not only org-fold.el) and generally harmless. > If you are sufficiently annoyed with them, most of these warnings can be > "fixed" by declaring the missing functions. The functions are available > at runtime, but not during native-compilation. > > Best, > Ihor > > -- The Kafka Pandemic A blog about science, health, human rights, and misopathy: https://thekafkapandemic.blogspot.com
Re: org-fold documentation
Vikas Rawal writes: > After upgrading to 9.5.3, I am getting warnings such as this, which I > suspect are due to org-fold.et. > > Warning (comp): org-fold.el:834:27: Warning: Unused lexical variable > `org-hide-macro-markers' Disable showing Disable logging > Warning (comp): org.el:76:30: Warning: Package cl is deprecated Disable > showing Disable logging > Warning (comp): ox.el:79:1: Warning: the function ‘org-back-to-heading’ > might not be defined at runtime. Disable showing Disable logging > Warning (comp): ox.el:79:1: Warning: the function > ‘org-next-visible-heading’ might not be defined at runtime. Disable showing > Disable logging > Warning (comp): ox.el:79:1: Warning: the function ‘org-at-heading-p’ might > not be defined at runtime. Disable showing Disable logging > > Is org-fold.el documented already? Any pointers to what might be causing > the above? These are native-comp warnings. They are not uncommon when native-compiling Org (not only org-fold.el) and generally harmless. If you are sufficiently annoyed with them, most of these warnings can be "fixed" by declaring the missing functions. The functions are available at runtime, but not during native-compilation. Best, Ihor
org-fold documentation
After upgrading to 9.5.3, I am getting warnings such as this, which I suspect are due to org-fold.et. Warning (comp): org-fold.el:834:27: Warning: Unused lexical variable `org-hide-macro-markers' Disable showing Disable logging Warning (comp): org.el:76:30: Warning: Package cl is deprecated Disable showing Disable logging Warning (comp): ox.el:79:1: Warning: the function ‘org-back-to-heading’ might not be defined at runtime. Disable showing Disable logging Warning (comp): ox.el:79:1: Warning: the function ‘org-next-visible-heading’ might not be defined at runtime. Disable showing Disable logging Warning (comp): ox.el:79:1: Warning: the function ‘org-at-heading-p’ might not be defined at runtime. Disable showing Disable logging Is org-fold.el documented already? Any pointers to what might be causing the above? Thanks, V.