<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">On 08/15/2012 06:41 PM, Stephen
      Gallagher wrote:<br>
    </div>
    <blockquote
      cite="mid:1345048887.2650.11.camel@sgallagh520.sgallagh.bos.redhat.com"
      type="cite">
      <pre wrap="">On Wed, 2012-08-15 at 17:52 +0200, Michal Židek wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">I worked on solution of ticket <a class="moz-txt-link-freetext" href="https://fedorahosted.org/sssd/ticket/1150">https://fedorahosted.org/sssd/ticket/1150</a> 
and I already had
some solutions with macros, but I found them ugly and obfuscated or not 
generic enough, so
they could not be used on all places where we use sysdb_transaction 
functions.

So I decided not to implement any wrapper macros around these functions 
(yet) and only
rewrite the places where we use sysdb_transactions to unify the way they 
are used (and to remove bad
examples of usage).

In this patch, I did it for files in src/db/. If this patch is OK, I'll 
try to spread this pattern to other modules
where sysdb_transaction_start/commit/cancel is used.

The patch is attached.

NOTE: Function sysdb_idmap_store_mappings looks like completely 
rewritten, but it is not. Except of
changes related to sysdb_transaction it had only bad indentation.
</pre>
      </blockquote>
      <pre wrap="">
Ack, but in the future please separate unrelated changes into a separate
patch to make review easier. In this particular case, I just regenerated
the patch for review with 'git format-patch -b', but in general it's
better to keep logically-unrelated patches separate.</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>
    <br>
    <br>
    Just realized, that this was not pushed to master. I rewrote<br>
    the commit message (originally this was the first part).<br>
    <br>
    Patch attached.<br>
    <br>
    Thanks<br>
    Michal<br>
  </body>
</html>