[Bf-blender-cvs] [5c1650aa05a] compositor-full-frame: add TODOs

[Manuel Castilla]

Commit: 5c1650aa05acdaa440519911609cd5707831e330
Author: Manuel Castilla
Date:   Thu Aug 12 11:07:56 2021 +0200
Branches: compositor-full-frame
https://developer.blender.org/rB5c1650aa05acdaa440519911609cd5707831e330

add TODOs

===================================================================

M	source/blender/compositor/operations/COM_DilateErodeOperation.cc

===================================================================

diff --git a/source/blender/compositor/operations/COM_DilateErodeOperation.cc b/source/blender/compositor/operations/COM_DilateErodeOperation.cc
index 83de8ea45e9..c894a5f43ef 100644
--- a/source/blender/compositor/operations/COM_DilateErodeOperation.cc
+++ b/source/blender/compositor/operations/COM_DilateErodeOperation.cc
@@ -180,6 +180,9 @@ void DilateErodeThresholdOperation::get_area_of_interest(const int input_idx,
 template<template<typename> typename TCompare>
 static float get_min_distance(DilateErodeThresholdOperation::PixelData &p)
 {
+  /* TODO(manzanilla): bad performance, generate a table with relative offsets on operation
+   * initialization to loop from less to greater distance and break as soon as #compare is
+   * true. */
   const TCompare compare;
   float min_dist = p.distance;
   const float *row = p.elem + ((intptr_t)p.ymin - p.y) * p.row_stride +
@@ -382,6 +385,8 @@ void DilateDistanceOperation::get_area_of_interest(const int input_idx,
 template<template<typename> typename TCompare>
 static float get_distance_value(DilateDistanceOperation::PixelData &p, const float start_value)
 {
+  /* TODO(manzanilla): bad performance, only loop elements within minimum distance removing
+   * coordinates and conditional if `dist <= min_dist`. May need to generate a table of offsets. */
   const TCompare compare;
   const float min_dist = p.min_distance;
   float value = start_value;