Skip to content

545 display issues with csv columnizer#548

Merged
Hirogen merged 6 commits intoDevelopmentfrom
545-display-issues-with-csv-columnizer
Mar 8, 2026
Merged

545 display issues with csv columnizer#548
Hirogen merged 6 commits intoDevelopmentfrom
545-display-issues-with-csv-columnizer

Conversation

@Hirogen
Copy link
Collaborator

@Hirogen Hirogen commented Mar 6, 2026

This pull request introduces a significant refactor to the columnizer interfaces and related classes, moving from legacy interfaces to new memory-based interfaces. The changes deprecate the old interfaces, update implementations to use the new ones, and remove obsolete code. This modernizes the codebase, clarifies interface usage, and improves maintainability.

Interface Modernization and Deprecation:

  • Marked legacy interfaces such as IColumn, IColumnizedLogLine, IColumnizerConfigurator, IColumnizerPriority, and IInitColumnizer as obsolete, guiding developers to use their new memory-based counterparts (e.g., IColumnMemory, IColumnizedLogLineMemory, etc.) instead. [1] [2] [3] [4] [5] [6]

  • Updated memory-based interfaces (IColumnMemory, IColumnizedLogLineMemory, IAutoLogLineMemoryColumnizerCallback, IColumnizerConfiguratorMemory, IColumnizerPriorityMemory, IInitColumnizerMemory) to no longer inherit from their obsolete counterparts, and removed redundant new members. [1] [2] [3] [4] [5] [6] [7]

Implementation Updates:

  • Refactored AutoColumnizer to implement ILogLineMemoryColumnizer instead of the obsolete ILogLineColumnizer, and updated its method signatures to use memory-based types. [1] [2] [3]

  • Removed obsolete interface implementations and members from ColumnizedLogLine and Column classes. [1] [2]

API and Usability Improvements:

  • Enhanced the IContextMenuEntry interface by marking old methods as obsolete and adding new overloads that use memory-based types, including improved XML documentation for clarity. [1] [2]

  • Added a LoadConfig method to IColumnizerConfiguratorMemory for improved configuration management.

Other Notable Changes:

  • Removed the LogLineExtensionsTests unit test class and updated the ToClipBoardText extension to only support memory-based log lines. [1] [2]

  • Minor project file and namespace updates, such as adding a root namespace and suppressing a new compiler warning. [1] [2]

These changes collectively modernize the columnizer API, encourage migration to safer and more efficient memory-based interfaces, and clean up obsolete code paths.

@Hirogen Hirogen merged commit b5caaab into Development Mar 8, 2026
1 check passed
@Hirogen Hirogen deleted the 545-display-issues-with-csv-columnizer branch March 8, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Display issues with CSV columnizer Failure to activate portable mode Plugins doesn't get loaded in Release /silent and dotnet detection

1 participant