Skip to content

Conversation

@CharityKathure
Copy link
Contributor

This pull request introduces significant changes to the LogMonitor project, primarily focusing on replacing the ConfigFileParser and JsonFileParser with a new JsonProcessor module. This includes the addition of new functions for handling different log types and updating the project files accordingly.

New JsonProcessor module:

  • Added JsonProcessor.cpp with functions to load, parse, and process JSON configuration files, handling different log types such as EventLog, File, ETW, and Process logs.
  • Added JsonProcessor.h with declarations for the new functions in JsonProcessor.cpp.

Project file updates:

  • Updated LogMonitor.vcxproj to include JsonProcessor.cpp and JsonProcessor.h, and removed references to ConfigFileParser.cpp and JsonFileParser.cpp. [1] [2]
  • Updated LogMonitorTests.vcxproj to include JsonProcessorTests.cpp and removed references to ConfigFileParserTests.cpp. [1] [2]

Test updates:

  • Added a new test class JsonProcessorTests in JsonProcessorTests.cpp to test the new JsonProcessor functionalities.
  • Updated LogMonitorTests.cpp to remove the inclusion of ConfigFileParser.cpp.

Codebase simplification:

  • Modified Main.cpp to use loadAndProcessJson instead of OpenConfigFile for reading the configuration file.

These changes aim to enhance the functionality and maintainability of the LogMonitor project by introducing a more robust JSON processing module.

Signed-off-by: Charity Kathure <ckathure@microsoft.com>
@CharityKathure CharityKathure force-pushed the boost-parsing branch 4 times, most recently from b543202 to 8525c9d Compare February 6, 2025 10:44
Signed-off-by: Charity Kathure <ckathure@microsoft.com>
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.

2 participants