Status Update, Elisp Compiler

2009-08-19 Thread Daniel Kraft
Hi Andy, I'm off from tomorrow until Sunday, and as GSoC seems to 'officially' have ended, I wanted to give you a brief update on my status and plans: Well, as to the status of the elisp compiler, you probably know -- it basically works, just missing are a lot of built-ins and some special st

On white-box tests

2009-08-19 Thread Ludovic Courtès
Hello! Just a note that I've been meaning to send for some time. "Michael Gran" writes: > http://git.savannah.gnu.org/cgit/guile.git/commit/?id=9b0c25f6d18d5be318ea3a47fd87cf7e63b689e1 [...] > --- a/test-suite/tests/strings.test > +++ b/test-suite/tests/strings.test [...] > +;; > +;; string

Re: On white-box tests

2009-08-19 Thread Mike Gran
On Wed, 2009-08-19 at 10:38 +0200, Ludovic Courtès wrote: > Hello! > > Just a note that I've been meaning to send for some time. > > "Michael Gran" writes: > > > http://git.savannah.gnu.org/cgit/guile.git/commit/?id=9b0c25f6d18d5be318ea3a47fd87cf7e63b689e1 > > [...] > > I'm not fully convinc

Re: On white-box tests

2009-08-19 Thread Ludovic Courtès
Mike Gran writes: > On Wed, 2009-08-19 at 10:38 +0200, Ludovic Courtès wrote: [...] >> What do you think? > > Keep it or dump it. It's all good. Just to be clear: I think we can keep them, it doesn't hurt. I just wanted to hear what you and others thought about the issue, because I think uni

apparenty SCM_I_CHAR is fixed, but:

2009-08-19 Thread Greg Troxel
From my autobuild on master cc1: warnings being treated as errors In file included from vm-engine.c:133, from vm.c:322: vm-i-scheme.c: In function 'vm_regular_engine': vm-i-scheme.c:437: warning: comparison is always true due to limited range of data type vm-i-scheme.c:437: warn

Re: On white-box tests

2009-08-19 Thread Mike Gran
On Wed, 2009-08-19 at 15:53 +0200, Ludovic Courtès wrote: > Mike Gran writes: > > > On Wed, 2009-08-19 at 10:38 +0200, Ludovic Courtès wrote: > I just wanted to hear what you and others thought about the issue, > because I think unit tests are a crucial part of software development. OK. To say

Re: On white-box tests

2009-08-19 Thread Ludovic Courtès
Mike Gran writes: > I wrote the string-internals tests to indicate to me when I'd done > something that had unexpected side-effects. They are intentionally > white-box; they are intentionally reflexive. OK, I understand now. Then if it's useful to you, well, it's indeed useful. :-) > But, I

Re: apparenty SCM_I_CHAR is fixed, but:

2009-08-19 Thread Mike Gran
On Wed, 2009-08-19 at 09:56 -0400, Greg Troxel wrote: > From my autobuild on master > > vm-i-scheme.c:437: warning: comparison is always true due to limited range of > data type For gcc, -Wtype-limits will catch this at compile time. Maybe we should put that in the GCC_CFLAGS in the configure.i

Re: apparenty SCM_I_CHAR is fixed, but:

2009-08-19 Thread Ludovic Courtès
Mike Gran writes: > On Wed, 2009-08-19 at 09:56 -0400, Greg Troxel wrote: >> From my autobuild on master >> >> vm-i-scheme.c:437: warning: comparison is always true due to limited range >> of data type > > For gcc, -Wtype-limits will catch this at compile time. IIUC this is precisely a compile

Re: apparenty SCM_I_CHAR is fixed, but:

2009-08-19 Thread Greg Troxel
l...@gnu.org (Ludovic Courtès) writes: > Mike Gran writes: > >> On Wed, 2009-08-19 at 09:56 -0400, Greg Troxel wrote: >>> From my autobuild on master >>> >>> vm-i-scheme.c:437: warning: comparison is always true due to limited range >>> of data type >> >> For gcc, -Wtype-limits will catch this

Re: apparenty SCM_I_CHAR is fixed, but:

2009-08-19 Thread Mike Gran
On Wed, 2009-08-19 at 17:25 +0200, Ludovic Courtès wrote: > Mike Gran writes: > > For gcc, -Wtype-limits will catch this at compile time. > > IIUC this is precisely a compile-time warning. :-) It is a compile time warning that is default for Greg, but apparently not for Andy or I. -Mike