[engine] Deprecate basic engine and make a better one

[zjkmxy]

After we unified the code of YaNFD and go-ndn, I think it is time to merge the implementation of PIT and FIB in the client library and the forwarder.

The existing client-side Trie is very simple but it needs effort to maintain.

ndnd/std/engine/basic/simple_trie.go

Lines 5 to 13 in efc6b86

	// NameTrie is a simple implementation of a Name trie (node/subtree) used for PIT and FIB.
	// It is slow due to the usage of TlvStr(). Subject to change when it explicitly affects performance.
	type NameTrie[V any] struct {
	val V
	key string
	par *NameTrie[V]
	dep int
	chd map[string]*NameTrie[V]
	}

Also, BasicEngine had deadlock issue if one tries wait on anything in the data callback. Though it is resolved in 012f965, I think there is still space to improve.

[pulsejet]

Refactoring to merge the PIT and FIB would be nice for maintenance.

Also, BasicEngine had deadlock issue if one tries wait on anything in the data callback.

There shouldn't be a deadlock but having one thread is desirable - if the app can't process packets fast enough this applies back pressure.

But I'm not super concerned about this in general. This API should really never be used by apps.

[zjkmxy]

Yeah the main goal is to unify the FIB&PIT with the forwarder one so we do not have to maintain two sets.