[Gtkradiant] PatchCreation bugfix: remaining brush will be removed
WJoseph at europe.ea.com
Thu Mar 22 09:40:55 CDT 2007
I am the one who implemented this feature originally. The behaviour you describe is as I originally intended.
I am telling you this because you appear to be mistakenly assuming that this behaviour is a bug. Assumptions like this are dangerous - when you make what seems like a simple bug-fix that changes this behaviour, you risk introducing other bugs and inconsistent behaviour. In addition, any change to an existing behaviour will have an impact on other users, who may prefer the existing behaviour.
Please be EXTREMELY careful when making any change to the behaviour of the editor, and consider ALL the impacts of your change! This is something I have learned through years of experience - ignore it at your peril.
From: gtkradiant-bounces at zerowing.idsoftware.com [mailto:gtkradiant-bounces at zerowing.idsoftware.com] On Behalf Of "Eduard Aumüller"
Sent: 18 March 2007 23:39
To: gtkradiant at zerowing.idsoftware.com
Subject: [Gtkradiant] PatchCreation bugfix: remaining brush will be removed
you might have notices that when you create a patch from scratch, the previously selected brush will remain, unselected
With the fix the unneeded brush will be gone.
Here is a little fix.
1. get the selection bounds
2. clear the selection
3. create the patch with the previously gained AABB
code in patchmanip.cpp line 915
915 //get the bounds of the current selection
916 AABB patch_creator_aabb = PatchCreator_getBounds();
918 //delete the current selection
921 //create the new patch
922 Scene_PatchConstructPrefab(GlobalSceneGraph(), patch_creator_aabb,
923 TextureBrowser_GetSelectedShader(GlobalTextureBrowser()), ePlane, GlobalXYWnd_getCurrentViewType(), w, h);
please adjust this in the SVN repository
Der GMX SmartSurfer hilft bis zu 70% Ihrer Onlinekosten zu sparen!
Ideal für Modem und ISDN: http://www.gmx.net/de/go/smartsurfer
Gtkradiant mailing list
Gtkradiant at zerowing.idsoftware.com
More information about the Gtkradiant