[development] patches that need reviews/work

Gabor Hojtsy gabor at hojtsy.hu
Fri Jun 1 11:43:50 UTC 2007


Ashraf Amayreh wrote:
> Thanks to Dries' wise decision, we can now really put some extra efforts 
> to make this a perfect release, there are patches that need reviews, 
> others that need some work to minimize their performance impact, and 
> others that are RTBC but are still not.
> 
> To ease this and make it more effective, everyone who has a patch that 
> needs reviews, some work or is RTBC but is still not, please post it in 
> this thread to get it under lights. Please don't post another 
> developer's patches unless you're in sync with him so that this won't 
> get filled with repetitive posts. A short description of each patch and 
> what it does should be helpful as long as it's no longer than a 
> paragraph, this is a second reason the patch author is in a better 
> position to post here rather than someone else, interested parties can 
> see the details in the issue page. By doing this, we can centralize all 
> our efforts on the patches in this thread.
> 
> Finally, let's avoid giving opinions on these patches over here to 
> streamline this thread and attempt to make it as direct as possible. The 
> issue page can be used for that.
> 
> My patch is a simple one that Moshe has agreed is RTBC, so I hope we can 
> get it in and turn to other important patches.
> 
> http://drupal.org/node/114938

Some advice:
  - next time try to follow your own suggestion and explain what your 
patch does when trying to shed some light on it!
  - for the same reason, having such patches in a generic titled thread 
can quickly go out of focus as far as I see...

Gabor


More information about the development mailing list