|
From: | Avinoam Kalma |
Subject: | [Octave-patch-tracker] [patch #10095] [octave forge] (image) new function imshowpair |
Date: | Fri, 3 Sep 2021 07:50:38 -0400 (EDT) |
User-agent: | Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.159 Safari/537.36 |
Follow-up Comment #2, patch #10095 (project octave): Looks good. Minor comments: 1. There is an identation problem in two lines 2. Could you please add some tests? (mainly to check input validity)? _______________________________________________________ Reply to this item at: <https://savannah.gnu.org/patch/?10095> _______________________________________________ Message sent via Savannah https://savannah.gnu.org/
[Prev in Thread] | Current Thread | [Next in Thread] |