<p>In <a href="https://github.com/dbsrgits/dbix-class/pull/88#discussion_r43759696">lib/SQL/Translator/Parser/DBIx/Class.pm</a>:</p>
<pre style='color:#555'>> @@ -149,13 +149,16 @@ sub parse {
>
> $table->primary_key(@primary) if @primary;
>
> - my %unique_constraints = $source->unique_constraints;
> - foreach my $uniq (sort keys %unique_constraints) {
> - if (!$source->_compare_relationship_keys($unique_constraints{$uniq}, \@primary)) {
> + my $unique_constraints = $source->unique_constraints_info;
> + foreach my $uniq (sort keys %$unique_constraints) {
> + my $cons = $unique_constraints->{$uniq};
</pre>
<p><em>Lines get really long</em> is exclusively an aesthetic concern. On the other hand <em>author made the interpreter do extra work... but I can't see why</em> is more involved.</p>
<p>Either way - this was a minor nitpick as I was reading. I will fix it when I do the final review/rebase pass.</p>
<p>Ignore this particular comment.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/dbsrgits/dbix-class/pull/88/files#r43759696">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AASeAscvidChTm_au6pskZLKWsbnWCGrks5pCMfRgaJpZM4Ga5rQ.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/dbsrgits/dbix-class/pull/88/files#r43759696"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>