[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #8535] [signal] firpm (new remez)
From: |
Rob Sykes |
Subject: |
[Octave-patch-tracker] [patch #8535] [signal] firpm (new remez) |
Date: |
Sun, 22 Feb 2015 09:23:00 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/40.0.2214.111 Chrome/40.0.2214.111 Safari/537.36 |
Follow-up Comment #8, patch #8535 (project octave):
Mike, yes, the submitted .cc file comprises three components: mmfir.c (the
filter designing code), mmfir.h (it's interface), and the Octave 'firpm'
wrapper. Note that, for maximum portability, mmfir.c/h are written in the C
language, so use of std::max, for example, is not appropriate there. (The
situation is very similar to that of Jake Janovetz's C implementation and its
Octave remez wrapper.) The C part is, in effect, library code, but I know
from previous experience that Octave prefers the simplicity of a single source
file. If you think it's better to do so, the C code could be given a separate
home as a library on SourceForge or Github, etc.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?8535>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/