Skip to content

Wrong use of Debug.Faile(...) in EventLogging #62

@mySupply

Description

@mySupply

Hi

In the EventLogging class, the exist this kind of code

Debug.Fail("Unable to log event, exception: {0}", e.Message);

However, this will always show/print the message 'Unable to log event, exception: {0}', as nothing is inserter into the placeholder.

The code could be updated to something like this:

Debug.Fail($"Unable to log event, exception: {e.Message}", e.ToString());

or

Debug.Fail($"Unable to log event, exception: {e.Message}");

Jacob Mogensen
mysupply

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions