<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>
thanks for comments<br><br><div><div id="SkyDrivePlaceholder"></div>&gt; From: simo@redhat.com<br>&gt; To: sssd-devel@lists.fedorahosted.org<br>&gt; Date: Tue, 12 Jun 2012 08:30:07 -0400<br>&gt; Subject: Re: [SSSD] [PATCH]-Bad debug message when adding services without explicit dns_discovery_domain<br>&gt; <br>&gt; On Tue, 2012-06-12 at 11:11 +0200, Jakub Hrozek wrote:<br>&gt; &gt; On Tue, Jun 12, 2012 at 11:04:28AM +0200, Jan Zelený wrote:<br>&gt; &gt; &gt; &gt; https://fedorahosted.org/sssd/ticket/920<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; Nack,<br>&gt; &gt; &gt; please follow our coding guidelines:<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; http://www.freeipa.org/page/Coding_Style<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; In particular, I'm referring to the line length.<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; Thanks<br>&gt; &gt; &gt; Jan<br>&gt; &gt; <br>&gt; &gt; Please also change the debug message to appropriate symbolic constant.<br>&gt; <br>&gt; s/message/level/ I guess :)<br>&gt; <br>&gt; Simo.<br>&gt; <br>&gt; -- <br>&gt; Simo Sorce * Red Hat, Inc * New York<br>&gt; <br>&gt; _______________________________________________<br>&gt; sssd-devel mailing list<br>&gt; sssd-devel@lists.fedorahosted.org<br>&gt; https://fedorahosted.org/mailman/listinfo/sssd-devel<br></div>                                               </div></body>
</html>