Re: [mlmmj] Fwd: [mlmmj commits] Implement new list text naming scheme

[Ben Schmidt <mail_ben_schmidt@yahoo.com.au>]

Thank you, Thomas. That was a very quick reply.
Ben.



On 11/09/11 2:10 AM, Thomas Goirand wrote:
> On 09/10/2011 11:26 AM, Ben Schmidt wrote:
>> Could a C coder please review this changeset for me? There's nothing too
>> technical in it so any proficient C coder should be able to review it
>> adequately. It's only had a very small amount of testing so far, but
>> seems to work.
>>
>> http://mlmmj.org/hg/mlmmj/rev/c3ee2bfaeb02
>>
>> (After this, the substitution mechanism needs to be reworked, which will
>> need a code review, and then the new list texts can be tested. Then I
>> will just fix a few easy and high priority bugs before releasing a beta.
>> Other bug fixes and enhancements can wait for another release which I
>> expect will follow much more quickly.)
>>
>> Ben.
>
> I had a quick look, and as you said, there's not much in this patch that
> could be seen as suspicious. Few functional tests should be enough.
>
> Thomas
>
>
>