From 82df7100c8d59f99f91056b32c1ce33cc5a68726 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Fri, 3 Feb 2017 15:54:59 +0100 Subject: Alembic: Renamed copy_zup_yup to copy_yup_from_zup (and same for zup_from_yup) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit With the new names the arguments (yup, zup) are in the same order as they appear in the function name. The old names used copy_src_dst(dst, src), which I found very confusing. Furthermore, now it is clear from where to where the copy is made. This makes the function names a little bit longer, though. If that is a real issue, we can just name them zup_from_yup(zup, yup). Reviewed by: Kévin Dietrich --- source/blender/alembic/intern/abc_nurbs.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'source/blender/alembic/intern/abc_nurbs.cc') diff --git a/source/blender/alembic/intern/abc_nurbs.cc b/source/blender/alembic/intern/abc_nurbs.cc index 4f57dfdae9e..d0b9561f679 100644 --- a/source/blender/alembic/intern/abc_nurbs.cc +++ b/source/blender/alembic/intern/abc_nurbs.cc @@ -153,7 +153,7 @@ void AbcNurbsWriter::do_write() const BPoint *bp = nu->bp; for (int i = 0; i < size; ++i, ++bp) { - copy_zup_yup(positions[i].getValue(), bp->vec); + copy_yup_from_zup(positions[i].getValue(), bp->vec); weights[i] = bp->vec[3]; } @@ -281,7 +281,7 @@ void AbcNurbsReader::readObjectData(Main *bmain, float time) posw_in = (*weights)[i]; } - copy_yup_zup(bp->vec, pos_in.getValue()); + copy_zup_from_yup(bp->vec, pos_in.getValue()); bp->vec[3] = posw_in; bp->f1 = SELECT; bp->radius = 1.0f; -- cgit v1.2.3