Re: ltp - first patch

From: Martin Schulze (joey@infodrom.org)
Date: Thu May 09 2002 - 11:23:22 CEST


Martin Schulze wrote:
> Ich habe mit dem Patch noch zwei Probleme:
>
> > - $query .= " AND nightplace = '$loc'" if ($loc);
> > - $query .= " ORDER BY name";
> > -
> > - $sth = $dbh->prepare($query);
> > - if ($sth && ($rc = $sth->execute) > 0) {
> > +
> > + my $sth = $dbh->prepare(q{
> > + select name,email,project,nightplace FROM person
> > + WHERE upper(nightplace) like upper(?) ORDER BY name
> > + }) or die "Can't prepare query: $DBI::errstr\n";
> > +
> > + unless ($loc) { $loc = '%'; }

Ich habe gerade noch einmal darueber nachgedacht. Ich frage mich, ob
das nicht einfach ein anderer Stil ist - nicht unbedingt besser oder
schlechter, sondern einfach anders. So sinnvoll ist es imho nicht,
ltp auf einen anderen Stil umzustellen (zumal ich ihn im Moment etwas
umstaendlicher und weniger intuitiv empfinde).

Gruesse,

        Joey

-- 
Never trust an operating system you don't have source for!



This archive was generated by hypermail 2.1.3 : Thu May 09 2002 - 11:27:13 CEST