[Ecls-list] [patch] Two patches

[Juan Jose Garcia-Ripoll]

Thanks a lot for reporting this and taking the time to fix it. I have
changed the second patch to reuse the current routines that scan
declarations, though.

Juanjo

On Sat, Nov 7, 2009 at 7:32 AM, Andy Hefner <ahefner at gmail.com> wrote:
> Attached are two patches against ECL CVS HEAD.
>
> I found that define-symbol-macro (and things which expand into it,
> such as ffi:def-foreign-var) produce warnings about unused FORM and
> ENV variables. symbol-macro-no-warning.patch addresses this by adding
> a (declare (ignore form env)) to the expansion.
>
> I noticed ECL doesn't seem to honor the declaration of variables as
> ignorable. ecl-ignorable-variables.patch attempts to address this by
> adding a IGNORABLE slot to the VAR struct and consulting it before
> issuing the unused variable warning in CHECK-VREF. I had to modify
> C1MAKE-VAR to take the list of other declarations returned from C1BODY
> (where the ignorable declarations end up) so it could set the var's
> ignorable slot correctly.
>
> ------------------------------------------------------------------------------
> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
> trial. Simplify your report design, integration and deployment - and focus on
> what you do best, core application coding. Discover what's new with
> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
> _______________________________________________
> Ecls-list mailing list
> Ecls-list at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ecls-list
>
>



-- 
Instituto de Física Fundamental, CSIC
c/ Serrano, 113b, Madrid 28006 (Spain)
http://juanjose.garciaripoll.googlepages.com