Code review comment for lp:~michael-abbott/epics-base/dynamic-array

Revision history for this message
Michael Abbott (michael-abbott) wrote :

Hi Andrew,

I'm on holiday until the start of September, so just a quick reply. I'll try and catch up when I get back.

If you're happy to merge, please go ahead. I though I'd already changed the minor version to 13, but maybe I messed up my update?

From: <email address hidden> [mailto:<email address hidden>] On Behalf Of Andrew Johnson
> I have rebased your dynamic-array commits from 12072 to 12077
> onto the latest 3.14 branch, dropping 12071 and handling the
> db_access.c renames from 12078 properly in the earlier
> commits. I also changed all references to the CA minor
> version number from 12 to 13 in the earlier commits. You are
> still credited as the author in the Bazaar history, although
> your original commit dates were lost in the manual rebase.
> There are a few changes I'll need to work on before I can
> merge it into 3.14 though:
> * Update the Perl5 interface to provide dynamic array
> support from Perl (almost complete)
> * Make src/catools/caput.c use dynamic arrays for before &
> after value displays
> * Documentation in both the CAref.html and RELEASE_NOTES.html files
>
> Would you prefer that I publish my branch for public
> examination while I'm working on it, or just merge the result
> without referring back to you?

« Back to merge proposal