Cleanup: remove unnecessary tooltip line limit
authorCampbell Barton <ideasman42@gmail.com>
Wed, 20 Dec 2017 06:20:33 +0000 (17:20 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 20 Dec 2017 06:20:33 +0000 (17:20 +1100)
Also add TODO for possible future changes.

source/blender/editors/interface/interface_region_tooltip.c

index 8186e247a47f7b7749a190e9092ec70620ce08a0..bc271d0a9fe31d9ff636dd611a96954f027dd385 100644 (file)
  * ToolTip Region and Construction
  */
 
+/* TODO(campbell):
+ * We may want to have a higher level API that initializes a timer,
+ * checks for mouse motion and clears the tool-tip afterwards.
+ * We never want multiple tool-tips at once so this could be handled on the window / window-manager level.
+ *
+ * For now it's not a priority, so leave as-is.
+ */
+
 #include <stdarg.h>
 #include <stdlib.h>
 #include <string.h>
@@ -97,7 +105,6 @@ typedef struct uiTooltipField {
 
 } uiTooltipField;
 
-#define MAX_TOOLTIP_LINES 8
 typedef struct uiTooltipData {
        rcti bbox;
        uiTooltipField *fields;
@@ -549,8 +556,6 @@ static uiTooltipData *ui_tooltip_data_from_button(bContext *C, uiBut *but)
        if (rna_prop.strinfo)
                MEM_freeN(rna_prop.strinfo);
 
-       BLI_assert(data->fields_len < MAX_TOOLTIP_LINES);
-
        if (data->fields_len == 0) {
                MEM_freeN(data);
                return NULL;