Skip to content

Conversation

Parithi098
Copy link
Collaborator

Missing header inclusion

@@ -56,6 +56,9 @@
#include <stdio.h>
#include <string.h>
#include <openssl/sha.h>
#ifndef _WIN32
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use this "__WINDOWS__"

This is defined when there is a def of "#if (defined(_WIN16) || defined(_WIN32) || defined(_WIN64)"

Copy link
Collaborator

Choose a reason for hiding this comment

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

concluded that all same.. _WIN32 is defined in WIN64 also (From Parithi)

Copy link
Collaborator

@vaibhaavram01 vaibhaavram01 left a comment

Choose a reason for hiding this comment

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

reviewed and approved

@Parithi098 Parithi098 removed the request for review from rarun-mchp September 11, 2025 09:14
@pvengat pvengat merged commit 9f1c242 into main Sep 11, 2025
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.

3 participants