Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(206)

Issue 5308073: Tiny transform cleanup (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 6 months ago by nicholasbishop
Modified:
7 years, 2 months ago
Reviewers:
theeth, brechtvl, bf-codereview
Base URL:
https://svn.blender.org/svnroot/bf-blender/trunk/blender/
Visibility:
Public.

Description

Function convertViewVec() contained some piece of code duplicated four times, pulled out into separate function and cleaned up a bit. Should be no functional changes.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -41 lines) Patch
M source/blender/editors/transform/transform.c View 2 chunks +22 lines, -41 lines 0 comments Download

Messages

Total messages: 4
nicholasbishop
7 years, 6 months ago (2011-11-01 04:29:11 UTC) #1
brechtvl
LGTM
7 years, 6 months ago (2011-11-01 18:42:36 UTC) #2
theeth
Thumbs up.
7 years, 6 months ago (2011-11-01 23:02:50 UTC) #3
nicholasbishop
7 years, 6 months ago (2011-11-02 00:19:18 UTC) #4
Thanks for reviewing, committed to r41456.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b