unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master e7be986: Make seq.el more extensible by using cl-defmethod
       [not found] ` <E1ZTZQD-0006Wj-BN@vcs.savannah.gnu.org>
@ 2015-08-23 23:32   ` Glenn Morris
  2015-08-24  8:40     ` Nicolas Petton
  0 siblings, 1 reply; 4+ messages in thread
From: Glenn Morris @ 2015-08-23 23:32 UTC (permalink / raw)
  To: emacs-devel; +Cc: Nicolas Petton


This seems to cause a test failure:
http://hydra.nixos.org/build/24976758/log/raw



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: master e7be986: Make seq.el more extensible by using cl-defmethod
  2015-08-23 23:32   ` master e7be986: Make seq.el more extensible by using cl-defmethod Glenn Morris
@ 2015-08-24  8:40     ` Nicolas Petton
  2015-08-24 22:02       ` Mark Oteiza
  0 siblings, 1 reply; 4+ messages in thread
From: Nicolas Petton @ 2015-08-24  8:40 UTC (permalink / raw)
  To: Glenn Morris, emacs-devel

[-- Attachment #1: Type: text/plain, Size: 233 bytes --]

Glenn Morris <rgm@gnu.org> writes:

> This seems to cause a test failure:
> http://hydra.nixos.org/build/24976758/log/raw

Thank you.  Commit 291593a shoudl fix the issue.

Nico
-- 
Nicolas Petton
http://nicolas-petton.fr

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 512 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: master e7be986: Make seq.el more extensible by using cl-defmethod
  2015-08-24  8:40     ` Nicolas Petton
@ 2015-08-24 22:02       ` Mark Oteiza
  2015-08-25 14:43         ` Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: Mark Oteiza @ 2015-08-24 22:02 UTC (permalink / raw)
  To: emacs-devel


About this change--loading the seq library makes noise:

‘array’ also matches CL structs and EIEIO classes
‘sequence’ also matches CL structs and EIEIO classes
‘sequence’ also matches CL structs and EIEIO classes



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: master e7be986: Make seq.el more extensible by using cl-defmethod
  2015-08-24 22:02       ` Mark Oteiza
@ 2015-08-25 14:43         ` Stefan Monnier
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Monnier @ 2015-08-25 14:43 UTC (permalink / raw)
  To: Mark Oteiza; +Cc: emacs-devel

> About this change--loading the seq library makes noise:
> ‘array’ also matches CL structs and EIEIO classes
> ‘sequence’ also matches CL structs and EIEIO classes
> ‘sequence’ also matches CL structs and EIEIO classes

These are valid warnings, yet the code can't really do anything
about them.  So there are two problems:
1- the warning should not show up while loading (only while compiling).
2- even while compiling the warning should be silenced in this case.


        Stefan



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-08-25 14:43 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20150823175109.25055.95845@vcs.savannah.gnu.org>
     [not found] ` <E1ZTZQD-0006Wj-BN@vcs.savannah.gnu.org>
2015-08-23 23:32   ` master e7be986: Make seq.el more extensible by using cl-defmethod Glenn Morris
2015-08-24  8:40     ` Nicolas Petton
2015-08-24 22:02       ` Mark Oteiza
2015-08-25 14:43         ` Stefan Monnier

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.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).