Removal of function to extend keypoints

[mbuerki]

No description provided.

[schneith]

mh, that looks like quite a bad bug. good catch :)
does it improve the tracking?

[mbuerki]

@schneith if this branch of the if block is executed, it inevitably segfaults. that's why I'm pretty sure we haven't had any actively used part of the code actually executing that branch, which is both good and bad I guess :-|
I mean the reason is that we usually don't detect keypoints twice for the same frame and then keep both sets of keypoints (i.e, extending existing keypoints with more) - something which I can hardly find a good reason to do anyways. So I'm very much wondering if we shouldn't just kill that function altogether (there's already one for inserting keypoints into an empty frame).
Or do you see a good reason to keep it?

[schneith]

ah too bad, some free tracker improvements would have been great :)
I don't see a reason to keep it if it is not used. Let's remove it then?

[mbuerki]

closing this, reopened here #33