Some reworked patches

Dirk Hohndel dirk at hohndel.org
Wed May 21 15:33:35 PDT 2014


Great patch series. Thank you!

One small comment:

On Wed, May 21, 2014 at 01:08:50PM -0300, Tomaz Canabrava wrote:
> From 79f3db466778c6c912ca8c2f63acba67c37a67ea Mon Sep 17 00:00:00 2001
> From: Tomaz Canabrava <tomaz.canabrava at intel.com>
> Date: Wed, 21 May 2014 12:06:02 -0300
> Subject: [PATCH 2/5] Create a 'remove_event' function that removes an event.
> 
> The logic of removing the event was in the UI, and this makes
> the code harder to test because we need to take into account
> also the events that the interface is receiving, instead of
> only relying on the algorithm to test.
> 
> so, now it lives in dive.h/.c and a unittest is easyer to make.

Would have been even cuter if you had included a matching unit test :-)

/D


More information about the subsurface mailing list