[Bf-committers] Finally, a working patch

[trip]

Tested..
  Very nice, hope ton will get to making his own soon unless he will use 
yours.
Comments: make it all just one huge menu when pressing the extrude E 
key. Normal and individual. That way it is faster to press and go 
instead of having to press to different menus.

Coo :D



> Nathan Letwory wrote:
>
>> Hi,
>>
>> the patch finally applied ok, but there were problems compiling, so I
>> dived in after all. Now a few technical comments:
>>
>> * don't introduce new variables in the middle of a function (unless 
>> it's
>> at the start of a statement block). Where this might be acceptable 
>> for C++
>> it is not for a few C compilers. So you should move EditFace 
>> *workface; to
>> the beginning of the function, along with the rest. Make sure you 
>> don't
>> start using expressions before everything necessary has been declared.
>> After that has been cleared up, it compiles fine.
>> * use the coding style:
>> http://www.blender3d.org/cms/Coding_Style.141.0.html to write your 
>> code
>> and patches. In your case, use hard-tabs instead of soft-tabs/spaces 
>> for
>> indentation.
>>
>> I did a bit of cleaning and I have the following patch:
>> http://www.letwory.net/patch/individual_facesextrude.patch
>>
>>
> Ok, I cleaned it up some, heres a new patch.
>
>> Anyway, it is quite a handy one, even if there is some odd 
>> mousebehaviour.
>> See http://www.letwory.net/tutes/individual_faceextrude.htm for an
>> example. Notice how I have to move my mouse up, to extrude the face
>> down...
>>
> It doesn't do that for me. . .hmm  ..
>