Cleanup: fix typos in comments and docs
Contributed by luzpaz. Differential Revision: https://developer.blender.org/D10447
This commit is contained in:
parent
7d5ef64bfb
commit
1b94c53aa6
|
@ -334,7 +334,7 @@ bool BlenderDisplayDriver::update_begin(const Params ¶ms,
|
|||
|
||||
/* Update PBO dimensions if needed.
|
||||
*
|
||||
* NOTE: Allocate the PBO for the the size which will fit the final render resolution (as in,
|
||||
* NOTE: Allocate the PBO for the size which will fit the final render resolution (as in,
|
||||
* at a resolution divider 1. This was we don't need to recreate graphics interoperability
|
||||
* objects which are costly and which are tied to the specific underlying buffer size.
|
||||
* The downside of this approach is that when graphics interoperability is not used we are
|
||||
|
|
|
@ -153,7 +153,7 @@ void BVHNode::update_time()
|
|||
namespace {
|
||||
|
||||
struct DumpTraversalContext {
|
||||
/* Descriptor of wile where writing is happening. */
|
||||
/* Descriptor of while where writing is happening. */
|
||||
FILE *stream;
|
||||
/* Unique identifier of the node current. */
|
||||
int id;
|
||||
|
|
|
@ -178,7 +178,7 @@ class InnerNode : public BVHNode {
|
|||
reset_unused_children();
|
||||
}
|
||||
|
||||
/* NOTE: This function is only used during binary BVH builder, and it
|
||||
/* NOTE: This function is only used during binary BVH builder, and it's
|
||||
* supposed to be configured to have 2 children which will be filled-in in a
|
||||
* bit. But this is important to have children reset to NULL. */
|
||||
explicit InnerNode(const BoundBox &bounds) : BVHNode(bounds), num_children_(0)
|
||||
|
|
|
@ -88,7 +88,7 @@ endmacro()
|
|||
|
||||
function(cycles_link_directories)
|
||||
if(APPLE)
|
||||
# APPLE plaform uses full paths for linking libraries, and avoids link_directories.
|
||||
# APPLE platform uses full paths for linking libraries, and avoids link_directories.
|
||||
return()
|
||||
endif()
|
||||
|
||||
|
|
|
@ -3,7 +3,7 @@ This program uses code from various sources, the default license is Apache 2.0
|
|||
for all code, with the following exceptions.
|
||||
|
||||
Modified BSD License
|
||||
* Code adapated from Open Shading Language
|
||||
* Code adapted from Open Shading Language
|
||||
* Sobol direction vectors
|
||||
* Matrix inversion code from OpenEXR
|
||||
* MD5 Hash code
|
||||
|
|
|
@ -76,7 +76,7 @@ class PathTraceDisplay {
|
|||
|
||||
/* Copy buffer of rendered pixels of a given size into a given position of the texture.
|
||||
*
|
||||
* This function does not acquire a lock. The reason for this is is to allow use of this function
|
||||
* This function does not acquire a lock. The reason for this is to allow use of this function
|
||||
* for partial updates from different devices. In this case the caller will acquire the lock
|
||||
* once, update all the slices and release
|
||||
* the lock once. This will ensure that draw() will never use partially updated texture. */
|
||||
|
|
|
@ -712,7 +712,7 @@ if(WITH_COMPILER_ASAN)
|
|||
string(APPEND CMAKE_CXX_FLAGS_RELWITHDEBINFO " -fno-sanitize=all")
|
||||
string(APPEND CMAKE_CXX_FLAGS_DEBUG " -fno-sanitize=vptr")
|
||||
elseif(CMAKE_C_COMPILER_ID MATCHES "Clang")
|
||||
# With OSL, Cycles disables rtti in some modules, wich then breaks at linking
|
||||
# With OSL, Cycles disables rtti in some modules, which then breaks at linking
|
||||
# when trying to use vptr sanitizer (included into 'undefined' general option).
|
||||
string(APPEND CMAKE_CXX_FLAGS_RELWITHDEBINFO " -fno-sanitize=vptr")
|
||||
string(APPEND CMAKE_CXX_FLAGS_DEBUG " -fno-sanitize=vptr")
|
||||
|
|
|
@ -46,7 +46,7 @@ template<typename T> ccl_device_forceinline T *get_payload_ptr_2()
|
|||
ccl_device_forceinline int get_object_id()
|
||||
{
|
||||
#ifdef __OBJECT_MOTION__
|
||||
/* Always get the the instance ID from the TLAS
|
||||
/* Always get the instance ID from the TLAS
|
||||
* There might be a motion transform node between TLAS and BLAS which does not have one. */
|
||||
return optixGetInstanceIdFromHandle(optixGetTransformListHandle(0));
|
||||
#else
|
||||
|
|
|
@ -55,7 +55,7 @@ if(APPLE)
|
|||
# Disable allocation warning on macOS prior to 10.14: the OSLRenderServices
|
||||
# contains member which is 64 bytes aligned (cache inside of OIIO's
|
||||
# unordered_map_concurrent). This is not something what the SDK supportsm, but
|
||||
# since we take care of allocations ourselves is is OK to ignore the
|
||||
# since we take care of allocations ourselves is OK to ignore the
|
||||
# diagnostic message.
|
||||
string(APPEND CMAKE_CXX_FLAGS " -faligned-allocation")
|
||||
endif()
|
||||
|
|
|
@ -23,10 +23,10 @@
|
|||
|
||||
CCL_NAMESPACE_BEGIN
|
||||
|
||||
// NOTE: For some reason gcc-7.2 does not instantiate this versio of allocator
|
||||
// gere (used in IESTextParser). Works fine for gcc-6, gcc-7.3 and gcc-8.
|
||||
// NOTE: For some reason gcc-7.2 does not instantiate this version of the
|
||||
// allocator here (used in IESTextParser). Works fine for gcc-6, gcc-7.3 and gcc-8.
|
||||
//
|
||||
// TODO(sergey): Get to the root of this issue, or confirm this i a compiler
|
||||
// TODO(sergey): Get to the root of this issue, or confirm this is a compiler
|
||||
// issue.
|
||||
template class GuardedAllocator<char>;
|
||||
|
||||
|
|
|
@ -541,7 +541,7 @@ static string path_make_compatible(const string &path)
|
|||
if ((path.size() >= 3) && (path[0] == DIR_SEP) && (path[1] == DIR_SEP)) {
|
||||
result = path_cleanup_unc(result);
|
||||
}
|
||||
/* Make sure volume-only path ends up wit ha directory separator. */
|
||||
/* Make sure volume-only path ends up wit a directory separator. */
|
||||
if (result.size() == 2 && result[1] == ':') {
|
||||
result += DIR_SEP;
|
||||
}
|
||||
|
|
|
@ -906,7 +906,7 @@ __forceinline void store4f_nt(void *ptr, const ssef &v)
|
|||
}
|
||||
|
||||
////////////////////////////////////////////////////////////////////////////////
|
||||
/// Euclidian Space Operators
|
||||
/// Euclidean Space Operators
|
||||
////////////////////////////////////////////////////////////////////////////////
|
||||
|
||||
__forceinline float dot(const ssef &a, const ssef &b)
|
||||
|
|
|
@ -133,7 +133,7 @@ function which generates skydome states for different solar emission spectra
|
|||
and solar radii: 'arhosekskymodelstate_alienworld_alloc_init()'.
|
||||
|
||||
See the notes about the "Alien World" functionality provided further down for a
|
||||
discussion of the usefulness and limits of that second initalisation function.
|
||||
discussion of the usefulness and limits of that second initialisation function.
|
||||
Sky model states that have been initialized with either function behave in a
|
||||
completely identical fashion during use and cleanup.
|
||||
|
||||
|
@ -368,7 +368,7 @@ SKY_ArHosekSkyModelState *SKY_arhosekskymodelstate_alloc_init(const double solar
|
|||
with a sun of a surface temperature given in 'kelvin'. The parameter
|
||||
'solar_intensity' controls the overall brightness of the sky, relative
|
||||
to the solar irradiance on Earth. A value of 1.0 yields a sky dome that
|
||||
is, on average over the wavelenghts covered in the model (!), as bright
|
||||
is, on average over the wavelengths covered in the model (!), as bright
|
||||
as the terrestrial sky in radiometric terms.
|
||||
|
||||
Which means that the solar radius has to be adjusted, since the
|
||||
|
|
Loading…
Reference in New Issue