[MODULE] Primitive highlight

Description

This module provides text highlighting of types, functions and macros by coloring them with a user specified color. For an identifier to be recognized as a type, function or macro, the file declaring it must be loaded.

primitive highlight screenshot

Version

This was tested with 4coder 4.1.7.

Setup

You need to #include this file in your custom layer (4coder_default_bindings.cpp by default) below the line:

1
#include "4coder_default_include.cpp"

You need to call the function primitive_highlight_draw_cpp_token_colors from default_render_buffer in 4coder_default_hooks.cpp instead of draw_cpp_token_colors. The function takes one extra argument which is the buffer that is currently being drawn. The call should look like this:

1
primitive_highlight_draw_cpp_token_colors( app, text_layout_id, &token_array, buffer );

You'll need to insert the following function prototype above the declaration of default_render_buffer:

1
function void primitive_highlight_draw_cpp_token_colors( Application_Links *app, Text_Layout_ID text_layout_id, Token_Array *array, Buffer_ID buffer );

Add those lines in your theme file to change the colors.

1
2
3
4
primitive_highlight_type = 0xffaaaaef;
primitive_highlight_function = 0xffbbbbbb;
primitive_highlight_macro = 0xffa46391;
primitive_highlight_4coder_command = 0xffffffff;

You can define the order in which primitive types are chosen in case of collisions (same identifier names with different types) by modifying the function primitive_highlight_get_note below. Modify the type_weight array at the top of the function to assign a higher value to the primitive type you want to have a higher priority. By default macro are chosen first, then types then functions (then commands).

Performance

Compiling in release mode (passing release as the second argument to buildsuper script) will greatly improve performances. If you have lots of files opened in 4coder this might be necessary to maintain a decent frame rate.

Test:

  • 4coder 4.1.7 out of the box with only this module modifications;
  • Window maximized, 1920 * 1080 resolution;
  • Windows 7 64bit in classic mode (compositor is of);
  • Alt + X (command_lister) > load_project (to load all 4coder source files, ~150 files);
  • Alt + X > profile_enable;
  • Control + i (interactiveswitchbuffer) > 4coder_draw.cpp;
  • Hold Control + Down arrow until the bottom of the file is reached;
  • Alt + X > profile_disable;
  • Alt + X > profile_inspect;
  • Select main;

CPU: Intel Core i7 860 2.8 Ghz

  • debug mode worst time 0.0142 s;
  • release mode worst time 0.0050 s;

Avoiding recreating the hashes_notes array each frame if it's not necessary would save about half the time (wouldn't help with worst case scenario).

Issues

If types, functions and macros are colored purple, make sure the theme file is loaded. The full theme file name needs to be set in config.4coder, which by default isn't the case. Use :

1
default_theme_name = "theme-4coder";

instead of

1
default_theme_name = "4coder";

High level overview

At the start of primitive_highlight_draw_cpp_token_colors an array containing the pointers to Code_Index_Notes and hashes of their strings is built, then sorted using ascending hashes values. Code_Index_Notes are information provided by 4coder about types, functions and macros that are attached to each "code" buffer. Searching each buffer Code_Index_Notes is slow, that's why we build a single big array containing all notes pointers.

When rendering CPP tokens, we use a binary search to find out if an identifier is a type, function or macro and color it.

Details

Prefix

This module uses primitive_highlight_ as a prefix for its types and functions.

Ids

This module declares the following managed ids:

In the colors group:

  • primitive_highlight_type;
  • primitive_highlight_function;
  • primitive_highlight_macro;
  • primitive_highlight_4coder_command;

Modified files

This module needs modification in the following files to work:

  • 4coder_default_bindings.h or your custom layer file;
  • 4coder_default_hooks.h;

Global variables

This module doesn't use any global variables.

Types

This module declare the following types:

  • primitive_highlight_hashes_notes_t;

Functions

This module declare the following functions:

  • primitive_highlight_quick_sort_hashes_notes;
  • primitive_highlight_create_big_note_array;
  • primitive_highlight_get_note;
  • primitive_highlight_draw_cpp_token_colors;

Scope attachment

This module doesn't declare any scope attachment.

primitive_highlight.cpp

  1
  2
  3
  4
  5
  6
  7
  8
  9
 10
 11
 12
 13
 14
 15
 16
 17
 18
 19
 20
 21
 22
 23
 24
 25
 26
 27
 28
 29
 30
 31
 32
 33
 34
 35
 36
 37
 38
 39
 40
 41
 42
 43
 44
 45
 46
 47
 48
 49
 50
 51
 52
 53
 54
 55
 56
 57
 58
 59
 60
 61
 62
 63
 64
 65
 66
 67
 68
 69
 70
 71
 72
 73
 74
 75
 76
 77
 78
 79
 80
 81
 82
 83
 84
 85
 86
 87
 88
 89
 90
 91
 92
 93
 94
 95
 96
 97
 98
 99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
/*
# Description

This module provides text highlighting of types, functions and macros by coloring them
with a user specified color. For an identifier to be recognized as a type, function or
macro, the file declaring it must be loaded.

# Version

This was tested with 4coder 4.1.7.

# Setup

You need to #include this file in your custom layer ('4coder_default_bindings.cpp' by
default) below the line:

'''c
#include "4coder_default_include.cpp"
'''

You need to call the function 'primitive_highlight_draw_cpp_token_colors' from 
'default_render_buffer' in '4coder_default_hooks.cpp' instead of 'draw_cpp_token_colors'. The
function takes one extra argument which is the buffer that is currently being drawn. The
call should look like this:

'''c
primitive_highlight_draw_cpp_token_colors( app, text_layout_id, &token_array, buffer );
'''

You'll need to insert the following function prototype above the declaration of 
'default_render_buffer':

'''c
function void primitive_highlight_draw_cpp_token_colors( Application_Links *app, Text_Layout_ID text_layout_id, Token_Array *array, Buffer_ID buffer );
'''

Add those lines in your theme file to change the colors.

'''c
primitive_highlight_type = 0xffaaaaef;
primitive_highlight_function = 0xffbbbbbb;
primitive_highlight_macro = 0xffa46391;
primitive_highlight_4coder_command = 0xffffffff;
'''

You can define the order in which primitive types are chosen in case of collisions (same
identifier names with different types) by modifying the function 
'primitive_highlight_get_note' below. Modify the 'type_weight' array at the top of the
function to assign a higher value to the primitive type you want to have a higher
priority. By default macro are chosen first, then types then functions (then commands).

# Performance

Compiling in 'release' mode (passing 'release' as the second argument to 'buildsuper' script)
will greatly improve performances. If you have lots of files opened in 4coder this might
be necessary to maintain a decent frame rate.

Test:
- 4coder 4.1.7 out of the box with only this module modifications;
- Window maximized, 1920 * 1080 resolution;
- Windows 7 64bit in classic mode (compositor is of);
- 'Alt + X' (command_lister) > 'load_project' (to load all 4coder source files, ~150 files);
- 'Alt + X' > profile_enable;
- 'Control + i' (interactive_switch_buffer) > 4coder_draw.cpp;
- Hold 'Control + Down arrow' until the bottom of the file is reached;
- 'Alt + X' > profile_disable;
- 'Alt + X' > profile_inspect;
- Select main;

CPU: Intel Core i7 860 2.8 Ghz
- debug mode worst time 0.0142 ms;
- release mode worst time 0.0050 ms;

Avoiding recreating the 'hashes_notes' array each frame if it's not necessary would save
about half the time (wouldn't help with worst case scenario).

# Issues

If types, functions and macros are colored purple, make sure the theme file is loaded. The
full theme file name needs to be set in 'config.4coder', which by default isn't the case.
Use :

'''c
default_theme_name = "theme-4coder";
'''

instead of

'''c
default_theme_name = "4coder";
'''

# High level overview

At the start of 'primitive_highlight_draw_cpp_token_colors' an array containing the
pointers to 'Code_Index_Note's and hashes of their strings is built, then sorted using
ascending hashes values. 'Code_Index_Note's are information provided by 4coder about types,
functions and macros that are attached to each "code" buffer. Searching each buffer 
'Code_Index_Note's is slow, that's why we build a single big array containing all notes
pointers.

When rendering CPP tokens, we use a binary search to find out if an identifier is a type,
function or macro and color it.

# Details

## Prefix

This module uses 'primitive_highlight_' as a prefix for its types and functions.

## Ids

This module declares the following managed ids:

In the 'colors' group:
- 'primitive_highlight_type';
- 'primitive_highlight_function';
- 'primitive_highlight_macro';
- 'primitive_highlight_4coder_command';

## Modified files

This module needs modification in the following files to work:
- '4coder_default_bindings.h' or your custom layer file;
- '4coder_default_hooks.h';

## Global variables

This module doesn't use any global variables.

## Types

This module declare the following types:
- 'primitive_highlight_hashes_notes_t';

## Functions

This module declare the following functions:
- 'primitive_highlight_quick_sort_hashes_notes';
- 'primitive_highlight_create_big_note_array';
- 'primitive_highlight_get_note';
- 'primitive_highlight_draw_cpp_token_colors';

## Scope attachment

This module doesn't declare any scope attachment.
*/

CUSTOM_ID( colors, primitive_highlight_type );
CUSTOM_ID( colors, primitive_highlight_function );
CUSTOM_ID( colors, primitive_highlight_macro );
CUSTOM_ID( colors, primitive_highlight_4coder_command );

/* NOTE: end is the index of the last item, NOT one past the last item. */
function void primitive_highlight_quick_sort_hashes_notes( u64* hashes, Code_Index_Note** notes, u64 start, u64 end ) {

    if ( hashes && start < end ) {

        u64 pivot_index = ( start + end ) / 2;
        u64 pivot_hash = hashes[ pivot_index ];

        u64 i = start, j = end;

        while ( 1 ) {

            while ( hashes[ i ] < pivot_hash ) {
                i++;
            }

            while ( hashes[ j ] > pivot_hash ) {
                j--;
            }

            if ( i < j ) {

                u64 hash_temp = hashes[ i ];
                hashes[ i ] = hashes[ j ];
                hashes[ j ] = hash_temp;

                Code_Index_Note* temp = notes[ i ];
                notes[ i ] = notes[ j ];
                notes[ j ] = temp;

                i++;
                j--;

            } else {

                break;
            }
        }

        primitive_highlight_quick_sort_hashes_notes( hashes, notes, start, j );
        primitive_highlight_quick_sort_hashes_notes( hashes, notes, j + 1, end );
    }
}

typedef struct primitive_highlight_hashes_notes_t {
    u64* hashes;
    Code_Index_Note** notes;
    i32 count;
} primitive_highlight_hashes_notes_t;

function primitive_highlight_hashes_notes_t primitive_highlight_create_big_note_array( Application_Links* app, Arena* arena ) {

    ProfileScope( app, "create_big_note_array" );

    primitive_highlight_hashes_notes_t hashes_notes = { 0 };

    Buffer_ID buffer_it = get_buffer_next( app, 0, Access_Always );

#if 0
    /* NOTE: locking the index only in this function seems to cost more than locking in the primitive_highlight_draw_cpp_token_colors function. */
    code_index_lock( );
#endif

    while ( buffer_it ) {

        Code_Index_File *file = code_index_get_file( buffer_it );

        if ( file ) {
            hashes_notes.count += file->note_array.count;
        }

        buffer_it = get_buffer_next( app, buffer_it, Access_Always );
    }

    hashes_notes.hashes = push_array( arena, u64, hashes_notes.count );
    hashes_notes.notes = push_array( arena, Code_Index_Note*, hashes_notes.count );

    i32 count = 0;

    {
        ProfileScope( app, "create hashes" );

        buffer_it = get_buffer_next( app, 0, Access_Always );

        while ( buffer_it ) {

            Code_Index_File *file = code_index_get_file( buffer_it );

            if ( file ) {

                for ( i32 i = 0; i < file->note_array.count; i++ ) {
                    hashes_notes.notes[ count ] = file->note_array.ptrs[ i ];
                    hashes_notes.hashes[ count ] = table_hash_u8( hashes_notes.notes[ count ]->text.str, hashes_notes.notes[ count ]->text.size );
                    count++;
                }
            }

            buffer_it = get_buffer_next( app, buffer_it, Access_Always );
        }
    }

#if 0
    code_index_unlock( );
#endif

    if ( count ) {
        ProfileScope( app, "quick_sort_hashes_notes" );
        primitive_highlight_quick_sort_hashes_notes( hashes_notes.hashes, hashes_notes.notes, 0, count - 1 );
    }

    return hashes_notes;
}

function Code_Index_Note* primitive_highlight_get_note( Application_Links* app, primitive_highlight_hashes_notes_t* hashes_notes, String_Const_u8 name ) {

    u8 type_weight[ 4 ];
    type_weight[ CodeIndexNote_4coderCommand ] = 1;
    type_weight[ CodeIndexNote_Function ] = 2;
    type_weight[ CodeIndexNote_Type ] = 3;
    type_weight[ CodeIndexNote_Macro ] = 4;

    ProfileScope( app, "get_note" );

    Code_Index_Note* result = 0;

    u64 name_hash = table_hash_u8( name.str, name.size );

    i32 start = 0;
    i32 end = hashes_notes->count - 1;

    while ( start <= end ) {

        i32 midle = ( start + end ) / 2;

        u64 note_hash = hashes_notes->hashes[ midle ];

        if ( name_hash < note_hash ) {
            end = midle - 1;
        } else if ( name_hash > note_hash ) {
            start = midle + 1;
        } else {

            ProfileBlockNamed( app, "solve collisions", solve_collisions );

            while ( midle - 1 >= start && hashes_notes->hashes[ midle - 1 ] == name_hash ) {
                midle--;
            }

            u8 current_weight = 0;

            while ( midle <= end && hashes_notes->hashes[ midle ] == name_hash ) {

                Code_Index_Note* note = hashes_notes->notes[ midle ];
                // Assert( note->kind < ArrayCount( type_weight ) );

                if ( type_weight[ note->note_kind ] > current_weight ) {

                    if ( string_compare( name, note->text ) == 0 ) {

                        current_weight = type_weight[ note->note_kind ];
                        result = note;

                        if ( current_weight == 4 ) {
                            break;
                        }
                    }
                }

                midle++;
            }

            ProfileCloseNow( solve_collisions );

            break;
        }
    }

    return result;
}

/* NOTE: This funciton is a modification of 'draw_cpp_token_colors' from '4coder_draw.cpp'. */
function void primitive_highlight_draw_cpp_token_colors( Application_Links *app, Text_Layout_ID text_layout_id, Token_Array *array, Buffer_ID buffer ) {

    Range_i64 visible_range = text_layout_get_visible_range( app, text_layout_id );
    i64 first_index = token_index_from_pos( array, visible_range.first );
    Token_Iterator_Array it = token_iterator_index( 0, array, first_index );

    /* NOTE: Start modification. */
    Scratch_Block scratch( app );

#if 1
    code_index_lock( );
#endif

    Temp_Memory notes_temp = begin_temp( scratch );
    primitive_highlight_hashes_notes_t hashes_notes = primitive_highlight_create_big_note_array( app, scratch );

    ProfileBlockNamed( app, "token loop", token_loop );
    /* NOTE: End modification. */

    for ( ; ; ) {

        Token *token = token_it_read( &it );

        if ( token->pos >= visible_range.one_past_last ){
            break;
        }

        /* NOTE: Start modification. */
        FColor color = fcolor_id( defcolor_text_default );
        b32 colored = false;

        if ( token->kind == TokenBaseKind_Identifier ) {

            Temp_Memory temp = begin_temp( scratch );
            String_Const_u8 lexeme = push_token_lexeme( app, scratch, buffer, token );
            Code_Index_Note* note = primitive_highlight_get_note( app, &hashes_notes, lexeme );
            end_temp( temp );

            if ( note ) {

                switch ( note->note_kind ) {
                    case CodeIndexNote_Type: {
                        color = fcolor_id( primitive_highlight_type );
                    } break;

                    case CodeIndexNote_Function: {
                        color = fcolor_id( primitive_highlight_function );
                    } break;

                    case CodeIndexNote_Macro: {
                        color = fcolor_id( primitive_highlight_macro );
                    } break;

                    case CodeIndexNote_4coderCommand: {
                        /* NOTE: 4coder doesn't create those notes as of 4.1.6. */
                        color = fcolor_id( primitive_highlight_4coder_command );
                    } break;
                }

                colored = true;

#if 1
                if ( note->note_kind == CodeIndexNote_Type ) {

                    Token_Iterator_Array dot_arrow_it = it;

                    if ( token_it_dec_non_whitespace( &dot_arrow_it ) ) {

                        Token* dot_arrow = token_it_read( &dot_arrow_it );

                        if ( dot_arrow->kind == TokenBaseKind_Operator && ( dot_arrow->sub_kind == TokenCppKind_Dot || dot_arrow->sub_kind == TokenCppKind_Arrow ) ) {
                            colored = false;
                        }
                    }
                }
#endif
            }
        }

        if( !colored ) {
            color = get_token_color_cpp( *token );
        }
        /* NOTE: End modification. */

        ARGB_Color argb = fcolor_resolve( color );
        paint_text_color( app, text_layout_id, Ii64_size( token->pos, token->size ), argb );

        if ( !token_it_inc_all( &it ) ){
            break;
        }
    }

#if 1
    code_index_unlock( );
#endif

    /* NOTE: Start modification. */
    ProfileCloseNow( token_loop );
    end_temp( notes_temp );
    /* NOTE: End modification. */
}

Edited by Simon Anciaux on Reason: Fixed duplicated comment