[Scummvm-devel] New engine merging procedure proposal

Arnaud Boutonné arnaud.boutonne at gmail.com
Mon Jun 20 09:34:37 CEST 2011


Hi all,

Even if I consider the 2 weeks delay a bit long, I agree with all the points
suggested by Johannes. Clear rules will hopefully avoid hot discussions
during and after the merge decision!

Best regards,
Arnaud



On Mon, Jun 20, 2011 at 6:25 AM, Paul Gilbert <paulfgilbert at gmail.com>wrote:

> On Mon, Jun 20, 2011 at 11:21 AM, Filippos Karapetis <bluegr at gmail.com>wrote:
>
>> Well, the engine did get a lot of positive reviews and opinions before it
>> was added.
>> So, I believe that the way it was added wasn't wrong, per se. However, I
>> don't believe
>> that the engine can be released in its current state. The auto-generated
>> C++ source
>> should first be cleaned up and converted to something readable before the
>> engine
>> will be deemed good/stable enough for a release.
>>
>> Regards
>> Filippos
>>
>
> I feel pretty much the same way - if the auto-generated code could be
> cleaned up to be as much like clean C++ as possible, I would consider that
> sufficient; I don't think manually re-implementing all the scripts by hand
> would be required, although of course interested parties could still
> undertake it if they wanted.
>
> Paul.
>
>
> ------------------------------------------------------------------------------
> EditLive Enterprise is the world's most technically advanced content
> authoring tool. Experience the power of Track Changes, Inline Image
> Editing and ensure content is compliant with Accessibility Checking.
> http://p.sf.net/sfu/ephox-dev2dev
> _______________________________________________
> Scummvm-devel mailing list
> Scummvm-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/scummvm-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scummvm.org/pipermail/scummvm-devel/attachments/20110620/7f23f42c/attachment.html>


More information about the Scummvm-devel mailing list