* Keeping an eye on byte-compilation warnings
@ 2011-08-13 21:49 John Wiegley
2011-08-14 6:39 ` David Maus
` (3 more replies)
0 siblings, 4 replies; 8+ messages in thread
From: John Wiegley @ 2011-08-13 21:49 UTC (permalink / raw)
To: emacs-orgmode
Maybe we're all using different versions of Emacs, but I find that
byte-compilation warnings keep increasing as time goes by. I'd like to ask
people to compile their code before committing, to keep the build log clean.
It looks messy when there are lots of unnecessary warnings.
Here's what I'm seeing now:
--8<---------------cut here---------------start------------->8---
lisp/org.el:
In end of data:
org.el:20575:1:Warning: the following functions are not known to be defined:
org-agenda-todo, org-find-invisible, org-find-visible
--8<---------------cut here---------------end--------------->8---
These represent a cyclic dependency. Should org.el contain some autoload
forms during compilation?
--8<---------------cut here---------------start------------->8---
lisp/org-footnote.el:
In org-footnote-in-valid-context-p:
org-footnote.el:185:37:Warning: reference to free variable
`message-cite-prefix-regexp'
In end of data:
org-footnote.el:869:1:Warning: the following functions are not known to be
defined: message-point-in-header-p, org-icompleting-read
--8<---------------cut here---------------end--------------->8---
Is there a reason org-footnote doesn't just require org?
--8<---------------cut here---------------start------------->8---
lisp/org-list.el:
In org-list-send-item:
org-list.el:1456:56:Warning: Function `sort*' from cl package called at
runtime
--8<---------------cut here---------------end--------------->8---
I've fixed this one in d24a141. There was no reason to use sort* instead of
sort.
--8<---------------cut here---------------start------------->8---
lisp/org-table.el:
In org-table-eval-formula:
org-table.el:2437:23:Warning: assignment to free variable
`duration-output-format'
org-table.el:2557:34:Warning: reference to free variable
`duration-output-format'
In org-table-time-string-to-seconds:
org-table.el:3252:11:Warning: assignment to free variable `minus'
org-table.el:3257:13:Warning: reference to free variable `minus'
--8<---------------cut here---------------end--------------->8---
I think an empty defvar during compilation is in order here, and that minus is
simply missing from the `let'.
Thanks,
John
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: Keeping an eye on byte-compilation warnings
2011-08-13 21:49 Keeping an eye on byte-compilation warnings John Wiegley
@ 2011-08-14 6:39 ` David Maus
2011-08-14 8:14 ` Achim Gratz
` (2 subsequent siblings)
3 siblings, 0 replies; 8+ messages in thread
From: David Maus @ 2011-08-14 6:39 UTC (permalink / raw)
To: John Wiegley; +Cc: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 457 bytes --]
At Sat, 13 Aug 2011 16:49:50 -0500,
John Wiegley wrote:
>
> Maybe we're all using different versions of Emacs, but I find that
> byte-compilation warnings keep increasing as time goes by. I'd like to ask
> people to compile their code before committing, to keep the build log clean.
> It looks messy when there are lots of unnecessary warnings.
+1
Best,
-- David
--
OpenPGP... 0x99ADB83B5A4478E6
Jabber.... dmjena@jabber.org
Email..... dmaus@ictsoc.de
[-- Attachment #2: Type: application/pgp-signature, Size: 230 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: Keeping an eye on byte-compilation warnings
2011-08-13 21:49 Keeping an eye on byte-compilation warnings John Wiegley
2011-08-14 6:39 ` David Maus
@ 2011-08-14 8:14 ` Achim Gratz
2011-08-15 7:46 ` Bastien
2011-08-14 18:25 ` Bastien
2011-08-15 7:46 ` Bastien
3 siblings, 1 reply; 8+ messages in thread
From: Achim Gratz @ 2011-08-14 8:14 UTC (permalink / raw)
To: emacs-orgmode
John Wiegley <jwiegley@gmail.com> writes:
> Maybe we're all using different versions of Emacs, but I find that
> byte-compilation warnings keep increasing as time goes by. I'd like to ask
> people to compile their code before committing, to keep the build log clean.
> It looks messy when there are lots of unnecessary warnings.
Thank you for bringing this up (again). A few comments:
The function that calls org-agenda-todo probably belongs into
org-agenda.
The function org-copy-visible uses two function from org-exp.el, but
probably doesn't belong there, so a declaration of these two functions
would be needed. I don't think autoloading during compile is
appropriate (does that even work?), and I'm not a huge fan of using
require in eval-when-compile for in-package functions (it's OK for
external dependencies). In a nutshell, that only works correctly when
the dependencies between the lisp files are correctly defined or if the
byte-compiled files are always removed before starting a new
byte-compile. Since there are quite a few circular dependencies
introduced by using require rather than declarations, only the latter
currently works (and hence the usual incantation of 'make clean && make
compile', which would not be necessary otherwise).
Can you recommend a package that can extract the dependencies and
perhaps highlight the circulars? There is a page in EmacsWiki with some
libraries, but I haven't worked with them yet. It seems like a good
idea to try and untangle that cicular dependencies mess...
Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+
Factory and User Sound Singles for Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#WaldorfSounds
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: Keeping an eye on byte-compilation warnings
2011-08-13 21:49 Keeping an eye on byte-compilation warnings John Wiegley
2011-08-14 6:39 ` David Maus
2011-08-14 8:14 ` Achim Gratz
@ 2011-08-14 18:25 ` Bastien
2011-08-15 7:46 ` Bastien
3 siblings, 0 replies; 8+ messages in thread
From: Bastien @ 2011-08-14 18:25 UTC (permalink / raw)
To: John Wiegley; +Cc: emacs-orgmode
Hi John,
John Wiegley <jwiegley@gmail.com> writes:
> lisp/org-list.el:
>
> In org-list-send-item:
> org-list.el:1456:56:Warning: Function `sort*' from cl package called at
> runtime
>
> I've fixed this one in d24a141. There was no reason to use sort* instead of
> sort.
Thanks for this!
> lisp/org-table.el:
>
> In org-table-eval-formula:
> org-table.el:2437:23:Warning: assignment to free variable
> `duration-output-format'
> org-table.el:2557:34:Warning: reference to free variable
> `duration-output-format'
>
> In org-table-time-string-to-seconds:
> org-table.el:3252:11:Warning: assignment to free variable `minus'
> org-table.el:3257:13:Warning: reference to free variable `minus'
I just fixed those, thanks again.
--
Bastien
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: Keeping an eye on byte-compilation warnings
2011-08-14 8:14 ` Achim Gratz
@ 2011-08-15 7:46 ` Bastien
0 siblings, 0 replies; 8+ messages in thread
From: Bastien @ 2011-08-15 7:46 UTC (permalink / raw)
To: Achim Gratz; +Cc: emacs-orgmode
Achim Gratz <Stromeko@nexgo.de> writes:
> John Wiegley <jwiegley@gmail.com> writes:
>> Maybe we're all using different versions of Emacs, but I find that
>> byte-compilation warnings keep increasing as time goes by. I'd like to ask
>> people to compile their code before committing, to keep the build log clean.
>> It looks messy when there are lots of unnecessary warnings.
>
> Thank you for bringing this up (again). A few comments:
>
> The function that calls org-agenda-todo probably belongs into
> org-agenda.
I moved `org-agenda-todo-yesterday' to org-agenda.el.
I moved org-find-visible/invisible to org.el.
Thanks!
--
Bastien
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: Keeping an eye on byte-compilation warnings
2011-08-13 21:49 Keeping an eye on byte-compilation warnings John Wiegley
` (2 preceding siblings ...)
2011-08-14 18:25 ` Bastien
@ 2011-08-15 7:46 ` Bastien
2011-08-18 18:08 ` Achim Gratz
3 siblings, 1 reply; 8+ messages in thread
From: Bastien @ 2011-08-15 7:46 UTC (permalink / raw)
To: John Wiegley; +Cc: emacs-orgmode
Hi John,
John Wiegley <jwiegley@gmail.com> writes:
> Maybe we're all using different versions of Emacs, but I find that
> byte-compilation warnings keep increasing as time goes by. I'd like to ask
> people to compile their code before committing, to keep the build log clean.
> It looks messy when there are lots of unnecessary warnings.
Thanks for the heads up. I'll be more careful about this.
--
Bastien
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: Keeping an eye on byte-compilation warnings
2011-08-15 7:46 ` Bastien
@ 2011-08-18 18:08 ` Achim Gratz
2011-08-19 21:55 ` Bastien
0 siblings, 1 reply; 8+ messages in thread
From: Achim Gratz @ 2011-08-18 18:08 UTC (permalink / raw)
To: emacs-orgmode
Bastien <bzg@altern.org> writes:
> Thanks for the heads up. I'll be more careful about this.
In org-fixup-indentation:
org.el:7368:32:Warning: reference to free variable `org-property-end-re'
In org-set-property:
org.el:14316:34:Warning: reference to free variable `fn'
Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+
SD adaptation for Waldorf microQ V2.22R2:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: Keeping an eye on byte-compilation warnings
2011-08-18 18:08 ` Achim Gratz
@ 2011-08-19 21:55 ` Bastien
0 siblings, 0 replies; 8+ messages in thread
From: Bastien @ 2011-08-19 21:55 UTC (permalink / raw)
To: Achim Gratz; +Cc: emacs-orgmode
Hi Achim,
Achim Gratz <Stromeko@nexgo.de> writes:
> In org-fixup-indentation:
> org.el:7368:32:Warning: reference to free variable `org-property-end-re'
>
> In org-set-property:
> org.el:14316:34:Warning: reference to free variable `fn'
Both fixed, thanks!
--
Bastien
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2011-08-19 21:54 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-13 21:49 Keeping an eye on byte-compilation warnings John Wiegley
2011-08-14 6:39 ` David Maus
2011-08-14 8:14 ` Achim Gratz
2011-08-15 7:46 ` Bastien
2011-08-14 18:25 ` Bastien
2011-08-15 7:46 ` Bastien
2011-08-18 18:08 ` Achim Gratz
2011-08-19 21:55 ` Bastien
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).