<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 05/09/2014 05:06 PM, Jakub Hrozek
wrote:<br>
</div>
<blockquote cite="mid:1399669572.27480.4.camel@localhost"
type="cite">
<pre wrap="">Hi,
see attached patch, just removes unused code.
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
sssd-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a>
<a class="moz-txt-link-freetext" href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a>
</pre>
</blockquote>
Interesting. I looked at the code and it seems to be indeed a dead
code.<br>
In the earlier versions it was not. It was used to debug a
collection to print the type of the element during the debugging.
There was function to serialize the collection. I do not see it now.<br>
I suspect that it was removed at some point as we went the other
way. But I would argue that the ability to print thew types of the
values in a debug message might be needed in future for
troubleshooting.<br>
<br>
So I am a bit on the fence about removing this code.<br>
<br>
<pre class="moz-signature" cols="72">--
Thank you,
Dmitri Pal
Sr. Engineering Manager IdM portfolio
Red Hat, Inc.</pre>
</body>
</html>