>>>>> "Martin" == Martin Vermeer <[EMAIL PROTECTED]> writes:

>>  The nested ifs should only one,

Martin> Cannot parse that ;-)

Just replacing

if (a)
   if (b)
      foo();

by

if (a&b)
   foo();

It makes it easier when reading to see what the alternatives are.

>> but the patch indeed looks good. Lars concern was to find out why
>> we have such files AFAIR.
>> 
Martin> patches; not knowing precisely what best to do): 2217,
>>  I prefer my bruteFind rewrite, although the code is not very nice.

Martin> Does it do the job (and does bruteFind remain 'brute')?

I think it does both. The idea is just to restrict the brute force
search to the area for which we have coordcache data.
 
JMarc

Reply via email to