[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #9570] Modulo-N circular convolution: ccon
|
From: |
Mike Miller |
|
Subject: |
[Octave-patch-tracker] [patch #9570] Modulo-N circular convolution: cconv.m |
|
Date: |
Tue, 20 Feb 2018 12:54:30 -0500 (EST) |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:58.0) Gecko/20100101 Firefox/58.0 |
Follow-up Comment #1, patch #9570 (project octave):
Thanks for your submission. You have done a pretty good job of adhering to the
Octave preferred coding style, thank you for that. There are still some
compact if-else statements that should be expanded, some missing spaces around
operators. If you wouldn't mind looking at cleaning it up a bit, that would
help, otherwise the person who commits it can clean it up.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?9570>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/