David Lonie kirjoitti 16.4.2012 kello 20.13: > On Mon, Apr 16, 2012 at 12:50 PM, Tuukka Verho > <[email protected]> wrote: >> Ok. But please change the variable names, the original ones were >> confusing. > > Sure -- which ones are confusing? They seem straightforward to me: > > [abc]Trans: Translation vector along the a, b, or c axis > curImage: Vector pointing to the current translational image of the > atoms/bonds > > What would you suggest instead?
No I meant the ones I changed in my patch: sqNormDifference -> minLengthSq curSqDist -> lengthSq ------------------------------------------------------------------------------ For Developers, A Lot Can Happen In A Second. Boundary is the first to Know...and Tell You. Monitor Your Applications in Ultra-Fine Resolution. Try it FREE! http://p.sf.net/sfu/Boundary-d2dvs2 _______________________________________________ Avogadro-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/avogadro-devel
