<br><br><div class="gmail_quote">El 26 de marzo de 2012 11:48, Andoni Morales <span dir="ltr"><<a href="mailto:ylatuya@gmail.com">ylatuya@gmail.com</a>></span> escribió:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="gmail_quote">El 26 de marzo de 2012 11:28, Hans Leidekker <span dir="ltr"><<a href="mailto:hans@codeweavers.com" target="_blank">hans@codeweavers.com</a>></span> escribió:<div><div class="h5"><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div>On Mon, 2012-03-26 at 11:14 +0200, Andoni Morales wrote:<br>
> -    case MSIMODIFY_REPLACE:<br>
>      case MSIMODIFY_MERGE:<br>
> +      /* check for a duplicated key */<br>
> +      r = msi_table_find_row( tv, rec, &row, column );<br>
> +      if (r == ERROR_SUCCESS) {<br>
> +        /* check that both are identical */<br>
> +        r = compare_record (tv, row, rec);<br>
> +        if (r != ERROR_SUCCESS)<br>
> +          break;<br>
> +      } else {<br>
> +        r = table_validate_new( tv, rec, NULL );<br>
> +        if (r != ERROR_SUCCESS)<br>
> +          break;<br>
> +        r = TABLE_insert_row( view, rec, -1, FALSE );<br>
> +          break;<br>
> +      }<br>
> +<br>
> +    case MSIMODIFY_REPLACE:<br>
<br>
</div>Please use bracketing and indentation style of the surrounding code.<br>
Some tests would be nice.<br></blockquote></div></div></div></blockquote><div><br></div><div>Updated patch with tests, which revealed bugs in the previous pathc :)</div><div><br></div><div>Cheers,</div><div>Andoni </div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br></blockquote><div><br></div></div></div><div>Sorry, I attached the wrong patch file. I have fixed the indentation and will add tests too. </div></div><div class="HOEnZb"><div class="h5"><br><br clear="all"><div><br></div>

-- <br>Andoni Morales Alastruey<br><br>LongoMatch:The Digital Coach<br>
<a href="http://www.longomatch.ylatuya.es" target="_blank">http://www.longomatch.ylatuya.es</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Andoni Morales Alastruey<br><br>LongoMatch:The Digital Coach<br><a href="http://www.longomatch.ylatuya.es">http://www.longomatch.ylatuya.es</a><br>