I said that you shouldn't add newlines at free will when the code snippets
could be better explained in sub methods; not to completely avoid them...


On Mon, Mar 25, 2013 at 1:09 PM, François Thillen
<notifications@github.com>wrote:

> In bridge/Implementation/ExtendMethods.cs:
>
> > +        {
> > +            Boolean a1 = keyType.Name.Equals(key.GetType().Name);
> > +            Boolean a2 = keyType.IsInstanceOfType(key.
> GetType().DeclaringType);
> > +            return !(a1 || a2);
> > +        }
> > +        /// <summary>
> > +        /// Converts a Dictionary to a Map (entryX)
> > +        /// </summary>
> > +        /// <typeparam name="T">Type to convert the Dictionary into</typeparam>
> > +        /// <param name="arg"></param>
> > +        /// <returns></returns>
> > +        public static T[] ConvertMap<T>(this IDictionary arg)
> > +        {
> > +            Array elements = ConvertMap(arg, typeof(T));
> > +            return (T[])elements;
> > +        }
>
> I rememer (1 year) ago that you told me not to use new lines. :-)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openengsb/loom-csharp/pull/48/files#r3510418>
> .
>