Thanks for the update. On Tue 12 Jul 2016 19:16, Daniel Llorens <daniel.llor...@bluewin.ch> writes:
> Subject: [PATCH 01/12] Compile in C99 mode This could be a good change but it is not the fastest path to patch review :) There are three considerations here: (1) Can we support C99 on all targets we care about? (2) Can we use C99 in our public interface, or just internally? If we use it publically, what should we change? No more scm_t_uint8 I hope, besides for back-compat? This patch set does not have to include these changes, but we should have a plan. (3) Most importantly, what is the impact on inlining? See the comment in __scm.h around line 165. If you want your patch set to depend on C99 that's fine, but you have to answer these questions first for the project as a whole and get some consensus. Andy