[Bf-committers] Updated: Knife subdivide patch

[Chris Burt]

Not making a fool of yourself, just learning out loud :) Don't feel bad 
about this. At least you're making the effort to work on something and 
help out. Thats more than some critics can say. Besides, you haven't 
exactly spammed the list. You thought you had a working product and 
after further review you just found out its not quite up to par. No big 
deal :) Don't get down on yourself and certainly don't give up. The 
learning process is only brutal if you make it that way ;) Thanks!

Regards,
--Chris

Matthew H. Plough wrote:
> Hi Ton --
> 
> Ah, that bug.  I took a quick look at it, and the code is horribly 
> complicated right now, and with the current design, it looks like it 
> needs to be.  I have some ideas for fixing it, but they might be 
> horribly naive, so I'll keep them to myself.  In other news, I found a 
> situation where my patch code doesn't work, so there's no need to review 
> it.  It's broken.  It looks like I need to do a lot more debugging and 
> regression testing before I send any more emails about it to this list.
> Making a fool of myself in public,
> Matt
> 
> Ton Roosendaal wrote:
> 
>> Hi,
>>
>> No time for review, but do have time to give you more work! :)
>>
>> https://projects.blender.org/tracker/index.php? 
>> func=detail&aid=2201&group_id=9&atid=125
>>
>> Could you check on bug 2201?
>>
>> -Ton-
> 
> 
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at projects.blender.org
> http://projects.blender.org/mailman/listinfo/bf-committers
> 
>