Github user jasonwilliams200OK commented on a diff in the pull request:

    https://github.com/apache/log4net/pull/16#discussion_r70600785
  
    --- Diff: tests/src/Utils.cs ---
    @@ -105,7 +114,28 @@ private static Type[] GetTypesArray(object[] args)
             internal static void RemovePropertyFromAllContexts() {
                 GlobalContext.Properties.Remove(PROPERTY_KEY);
                 ThreadContext.Properties.Remove(PROPERTY_KEY);
    +#if !(NETCF || NETSTANDARD1_3)
                 LogicalThreadContext.Properties.Remove(PROPERTY_KEY);
    +#endif
             }
    -   }
    +
    +        // Wrappers because repository/logger retrieval APIs require an 
Assembly argument on NETSTANDARD1_3
    +        internal static ILog GetLogger(string name)
    +        {
    +#if NETSTANDARD1_3
    +            return 
LogManager.GetLogger(typeof(Utils).GetTypeInfo().Assembly, name);
    +#else
    +                   return LogManager.GetLogger(name);
    +#endif
    +        }
    +
    +        internal static ILoggerRepository GetRepository()
    +        {
    +#if NETSTANDARD1_3
    +            return 
LogManager.GetRepository(typeof(Utils).GetTypeInfo().Assembly);
    +#else
    +                   return LogManager.GetRepository();
    +#endif
    --- End diff --
    
    nit: indentation of either one of two return statements is off


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to