While I don't use windows some comments on the patch.
- configure your editor not to change whitespace, it makes patches
confusing to review.
- your patch checks that cygwin and msvc are not used, but this seems
a precarious way to check for mingw. If there isnt a better way,
perhaps a MINGW variable could be set once at the start and checked
later on rather then checking for !(msvc || cygwin) each time.

On Tue, Mar 30, 2010 at 8:49 PM, Ryan Tralalaian <kal...@gmail.com> wrote:
> damn, forgot the link...
>
> http://projects.blender.org/tracker/index.php?func=detail&aid=21842&group_id=9&atid=125
> _______________________________________________
> Bf-committers mailing list
> Bf-committers@blender.org
> http://lists.blender.org/mailman/listinfo/bf-committers
>



-- 
- Campbell
_______________________________________________
Bf-committers mailing list
Bf-committers@blender.org
http://lists.blender.org/mailman/listinfo/bf-committers

Reply via email to