Here is a CR comment in #4020:

I see that you are following a pattern like IsSameFileSystemItem or IsHardLink. Ideally, code here should be refactored to consolidate Unix/Windows implementations in one place and eliminate wrapper functions like IsSameFileSystemItem and IsHardLink. Instead, they should be

internal static bool IsSameFileSystemItem(string pathOne, string pathTwo)
{
#if UNIX
   // Unix implementation goes here
#else
   // Windows implementation goes here
#endif
}

The code should be refactored to reflect the above comment.

• Refactor the existing methods there to consolidate the Unix/Windows implementations at one place.
• The type is InternalSymbolicLinkLinkCodeMethods, so for utility methods that are not directly related to SymbolicLink, such as GetInodeData, maybe a separate type should be created to hold them.