Hi Jerry, hi all,

Jerry D wrote:
Here is a current patch set.

Can this be put into some proper patch series? I mean there are usually patches posted as 1/10 to 10/10 patches and I see the following in the patches you posted:

* The runtime library itself – consisting one the files into the runtime library, the configure.ac/Makefile.ac integration, and fixes (unused args) to get it running. — And some follow-up fixes.

* Front-end changes – those seem to be generic changes. I hope that there is somewhere a testcase for this, but it seems as those should be separately reviewed.

* Testsuite changes – including enabling the new library.

The first two items seem to be rather independent.

In principle, I think it would be good to have the FE fixes with testcases in first, then the library integration and then the testsuite enabling.

But in any case, I think for the FE part, it makes sense to have a proper patch email per patch – such that it can be reviewed one by one.

Tobias

Reply via email to