Re: [xmlvm-users] pending patches

[Sascha H. <sa...@xm...>]

Panayotis,

I just saw your giant patch. Do you think you can split it up into smaller
chunks? The bigger a patch gets, the harder it is for us to review it and
the longer it will take. A patch at that size is very impractical to handle.

If you could split it up into smaller chunks, that would help speed up the
process a lot.

Thanks
// Sascha

On Thu, Mar 18, 2010 at 1:55 PM, Panayotis Katsaloulis <
pan...@pa...> wrote:

> Hello again!
>
> I've sent my current patch for review in the usual places.
>
> One thing I have in mind is that I see that there is some duplicate
> work done by me and  Gergely (and probably others?).
> The problem is not only the duplicate work and waste of time and
> resources, but it would also be difficult to merge things like this.
>
> I really know that the core xmlvm developers have very limited (free)
> time (as we all have).
> I believe though that, since xmlvm is still in the beginning, patches
> of this size (like mine and Gergely) are possible to come all the
> time. If the project was mature, then conflicts like this would be
> more seldom.
>
> To avoid this confusion the only thing could be done is rather faster
> review progress.
> I know that you want to keep the code quality into high levels, and
> time is sparse, but I believe this is also a priority for this
> project...
>
>
> ------------------------------------------------------------------------------
> Download Intel&#174; Parallel Studio Eval
> Try the new software tools for yourself. Speed compiling, find bugs
> proactively, and fine-tune applications for parallel performance.
> See why Intel Parallel Studio got high marks during beta.
> http://p.sf.net/sfu/intel-sw-dev
> _______________________________________________
> xmlvm-users mailing list
> xml...@li...
> https://lists.sourceforge.net/lists/listinfo/xmlvm-users
>