Skip to content

Commit

Permalink
Fix mmIconsMask, was missing 2 of the icons.
Browse files Browse the repository at this point in the history
  • Loading branch information
@martijnlaan
martijnlaan committed Jun 17, 2024
1 parent 0f4fafe commit ffdfa87
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions Projects/Src/CompScintEdit.pas
View file
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,15 @@ interface
mmIconBreakpoint = 2; { stop sign }
mmIconBreakpointGood = 3; { stop sign + check }
mmIconBreakpointBad = 4; { stop sign + X }
mmIconsMask = $1F;

mmLineError = 10; { maroon line highlight }
mmLineBreakpointBad = 11; { ugly olive line highlight }
mmLineStep = 12; { blue line highlight }
mmIconStep = 13; { blue arrow }
mmIconBreakpointStep = 14; { blue arrow on top of a stop sign + check }
mmIconsMask = (1 shl mmIconHasEntry) or (1 shl mmIconEntryProcessed) or
(1 shl mmIconBreakpoint) or (1 shl mmIconBreakpointGood) or
(1 shl mmIconBreakpointBad) or (1 shl mmIconStep) or
(1 shl mmIconBreakpointStep);

{ Memo indicator numbers - Note: inSquiggly and inPendingSquiggly are 0 and 1
in ScintStylerInnoSetup and must be first and second here. Also note: even
Expand Down

2 comments on commit ffdfa87

@jordanrussell
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When mmIconStep and mmIconBreakpointStep were first added I wondered why they weren't grouped with the other mmIcon* values (i.e., as 5 and 6)?

@martijnlaan
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't notice the gap in numbers before and not sure why it's there. So yeah, either there should be no gap or they should be grouped.

Please sign in to comment.