Skip to content

Commit 84d55c8

Browse files
committed
make some progress on watch window refinement for various row topologies
1 parent 2a1ab9b commit 84d55c8

File tree

9 files changed

+846
-200
lines changed

9 files changed

+846
-200
lines changed

src/eval_visualization/eval_visualization_core.c

Lines changed: 3 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -667,31 +667,11 @@ ev_resolved_from_expr(Arena *arena, E_Expr *expr, EV_ViewRuleList *view_rules)
667667
return expr;
668668
}
669669

670-
////////////////////////////////
671-
//~ rjf: Column List Building
672-
673-
internal EV_Col *
674-
ev_col_list_push(Arena *arena, EV_ColList *list)
675-
{
676-
EV_Col *col = push_array(arena, EV_Col, 1);
677-
SLLQueuePush(list->first, list->last, col);
678-
list->count += 1;
679-
return col;
680-
}
681-
682-
internal EV_Col *
683-
ev_col_list_push_new(Arena *arena, EV_ColList *list, String8 key)
684-
{
685-
EV_Col *col = ev_col_list_push(arena, list);
686-
col->key = push_str8_copy(arena, key);
687-
return col;
688-
}
689-
690670
////////////////////////////////
691671
//~ rjf: Block Building
692672

693673
internal EV_BlockTree
694-
ev_block_tree_from_expr(Arena *arena, EV_View *view, String8 filter, String8 string, E_Expr *expr, EV_ViewRuleList *view_rules, EV_ColList *cols)
674+
ev_block_tree_from_expr(Arena *arena, EV_View *view, String8 filter, String8 string, E_Expr *expr, EV_ViewRuleList *view_rules)
695675
{
696676
ProfBeginFunction();
697677
EV_BlockTree tree = {&ev_nil_block};
@@ -711,7 +691,6 @@ ev_block_tree_from_expr(Arena *arena, EV_View *view, String8 filter, String8 str
711691
tree.root = push_array(arena, EV_Block, 1);
712692
MemoryCopyStruct(tree.root, &ev_nil_block);
713693
tree.root->key = ev_key_root();
714-
tree.root->cols = *cols;
715694
tree.root->string = string;
716695
tree.root->expr = ev_resolved_from_expr(arena, expr, top_level_view_rules);
717696
tree.root->view_rules = top_level_view_rules;
@@ -779,7 +758,6 @@ ev_block_tree_from_expr(Arena *arena, EV_View *view, String8 filter, String8 str
779758
expansion_block->parent = t->parent_block;
780759
expansion_block->key = key;
781760
expansion_block->split_relative_idx = t->split_relative_idx;
782-
expansion_block->cols = *cols;
783761
expansion_block->expr = t->expr;
784762
expansion_block->view_rules = t->view_rules;
785763
expansion_block->expand_view_rule_info = expand_view_rule_info;
@@ -907,7 +885,7 @@ ev_block_tree_from_expr(Arena *arena, EV_View *view, String8 filter, String8 str
907885
}
908886

909887
internal EV_BlockTree
910-
ev_block_tree_from_string(Arena *arena, EV_View *view, String8 filter, String8 string, EV_ViewRuleList *view_rules, EV_ColList *cols)
888+
ev_block_tree_from_string(Arena *arena, EV_View *view, String8 filter, String8 string, EV_ViewRuleList *view_rules)
911889
{
912890
ProfBeginFunction();
913891
EV_BlockTree tree = {0};
@@ -918,7 +896,7 @@ ev_block_tree_from_string(Arena *arena, EV_View *view, String8 filter, String8 s
918896
EV_ViewRuleList *fastpath_view_rules = ev_view_rule_list_from_expr_fastpaths(arena, string);
919897
EV_ViewRuleList *all_view_rules = ev_view_rule_list_copy(arena, view_rules);
920898
ev_view_rule_list_concat_in_place(all_view_rules, &fastpath_view_rules);
921-
tree = ev_block_tree_from_expr(arena, view, filter, string, parse.expr, all_view_rules, cols);
899+
tree = ev_block_tree_from_expr(arena, view, filter, string, parse.expr, all_view_rules);
922900
}
923901
scratch_end(scratch);
924902
ProfEnd();

src/eval_visualization/eval_visualization_core.h

Lines changed: 3 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -187,24 +187,6 @@ struct EV_ViewRuleInfoTable
187187
U64 slots_count;
188188
};
189189

190-
////////////////////////////////
191-
//~ rjf: Columns
192-
193-
typedef struct EV_Col EV_Col;
194-
struct EV_Col
195-
{
196-
EV_Col *next;
197-
String8 key;
198-
};
199-
200-
typedef struct EV_ColList EV_ColList;
201-
struct EV_ColList
202-
{
203-
EV_Col *first;
204-
EV_Col *last;
205-
U64 count;
206-
};
207-
208190
////////////////////////////////
209191
//~ rjf: Blocks
210192

@@ -224,9 +206,6 @@ struct EV_Block
224206
// rjf: split index, relative to parent's space
225207
U64 split_relative_idx;
226208

227-
// rjf: columns
228-
EV_ColList cols;
229-
230209
// rjf: expression / visualization info
231210
String8 string;
232211
E_Expr *expr;
@@ -372,7 +351,7 @@ global read_only EV_ViewRuleInfo ev_nil_view_rule_info =
372351
thread_static EV_ViewRuleInfoTable *ev_view_rule_info_table = 0;
373352
global read_only EV_ViewRuleList ev_nil_view_rule_list = {0};
374353
thread_static EV_AutoViewRuleTable *ev_auto_view_rule_table = 0;
375-
global read_only EV_Block ev_nil_block = {&ev_nil_block, &ev_nil_block, &ev_nil_block, &ev_nil_block, &ev_nil_block, {0}, 0, {0}, {0}, &e_expr_nil, &ev_nil_view_rule_list, &ev_nil_view_rule_info};
354+
global read_only EV_Block ev_nil_block = {&ev_nil_block, &ev_nil_block, &ev_nil_block, &ev_nil_block, &ev_nil_block, {0}, 0, {0}, &e_expr_nil, &ev_nil_view_rule_list, &ev_nil_view_rule_info};
376355

377356
////////////////////////////////
378357
//~ rjf: Key Functions
@@ -436,17 +415,11 @@ internal void ev_view_rule_list_concat_in_place(EV_ViewRuleList *dst, EV_ViewRul
436415

437416
internal E_Expr *ev_resolved_from_expr(Arena *arena, E_Expr *expr, EV_ViewRuleList *view_rules);
438417

439-
////////////////////////////////
440-
//~ rjf: Column List Building
441-
442-
internal EV_Col *ev_col_list_push(Arena *arena, EV_ColList *list);
443-
internal EV_Col *ev_col_list_push_new(Arena *arena, EV_ColList *list, String8 key);
444-
445418
////////////////////////////////
446419
//~ rjf: Block Building
447420

448-
internal EV_BlockTree ev_block_tree_from_expr(Arena *arena, EV_View *view, String8 filter, String8 string, E_Expr *expr, EV_ViewRuleList *view_rules, EV_ColList *cols);
449-
internal EV_BlockTree ev_block_tree_from_string(Arena *arena, EV_View *view, String8 filter, String8 string, EV_ViewRuleList *view_rules, EV_ColList *cols);
421+
internal EV_BlockTree ev_block_tree_from_expr(Arena *arena, EV_View *view, String8 filter, String8 string, E_Expr *expr, EV_ViewRuleList *view_rules);
422+
internal EV_BlockTree ev_block_tree_from_string(Arena *arena, EV_View *view, String8 filter, String8 string, EV_ViewRuleList *view_rules);
450423
internal U64 ev_depth_from_block(EV_Block *block);
451424

452425
////////////////////////////////

0 commit comments

Comments
 (0)