completely change interface again.

compiles, not really tested.

basic test works 70% of the time, coredumps in the other 30 - but was not easy to debug on a first glance (most interestingly the crash happens in the logging framework - I wonder how that works).
Other tests are not adjusted to the new interface yet.

src/input/ReaderBackend.h

@@ -53,46 +53,14 @@ public:
 	 * 
 	 * @param mode the opening mode for the input source
 	 *
-	 * @param autostart automatically start the input source after the first filter has been added
-	 *
-	 * @return False if an error occured.
-	 */
-	bool Init(string arg_source, int mode, bool autostart);
-
-	/**
-	 * One-time start method of the reader.
-	 *
-	 * This method is called from the scripting layer, after all filters have been added.
-	 * No data should be read before this method is called.
-	 *
-	 * If autostart in Init is set to true, this method is called automatically by the backend after
-	 * the first filter has been added.
-	 */
-	bool StartReading();
-
-	/**
-	 * Add an input filter to the input stream
-	 *
-	 * @param id identifier of the input stream
-	 *
 	 * @param arg_num_fields number of fields contained in \a fields
 	 *
 	 * @param fields the types and names of the fields to be retrieved from the input source
 	 *
 	 * @return False if an error occured.
 	 */
-	bool AddFilter( int id, int arg_num_fields, const threading::Field* const* fields );
+	bool Init(string arg_source, int mode, int arg_num_fields, const threading::Field* const* fields);
 
-	
-	/**
-	 * Remove an input filter to the input stream
-	 *
-	 * @param id identifier of the input stream
-	 *
-	 * @return False if an error occured.
-	 */
-	bool RemoveFilter ( int id );
-    
 	/**
 	 * Finishes reading from this input stream in a regular fashion. Must not be
 	 * called if an error has been indicated earlier. After calling this,
@@ -131,33 +99,7 @@ protected:
 	 * disabled and eventually deleted. When returning false, an
 	 * implementation should also call Error() to indicate what happened.
 	 */
-	virtual bool DoInit(string arg_sources, int mode) = 0;
-
-	/**
-	 * Reader-specific start method. After this function has been called, data may be read from
-	 * the input source and be sent to the specified filters
-	 *
-	 * A reader implementation must override this method.
-	 * If it returns false, it will be assumed that a fatal error has occured
-	 * that prevents the reader from further operation; it will then be
-	 * disabled and eventually deleted. When returning false, an implementation
-	 * should also call Error to indicate what happened.
-	 */
-	virtual bool DoStartReading() = 0;
-
-	/**
-	 * Reader-specific method to add a filter.
-	 *
-	 * A reader implementation must override this method.
-	 */
-	virtual bool DoAddFilter( int id, int arg_num_fields, const threading::Field* const* fields ) = 0;
-
-	/**
-	 * Reader-specific method to remove a filter.
-	 *
-	 * A reader implementation must override this method.
-	 */
-	virtual bool DoRemoveFilter( int id ) = 0;
+	virtual bool DoInit(string arg_sources, int mode, int arg_num_fields, const threading::Field* const* fields) = 0;
 
 	/**
 	 * Reader-specific method implementing input finalization at
@@ -209,31 +151,26 @@ protected:
 	 *
 	 * If the filter points to a table, the values are inserted into the table; if it points to an event, the event is raised
 	 *
-	 * @param id the input filter id for which the values are sent
-	 *
 	 * @param val list of threading::Values expected by the filter
 	 */
-	void Put(int id, threading::Value* *val);
+	void Put(threading::Value* *val);
 
 	/**
 	 * Method allowing a reader to delete a specific value from a bro table.
 	 *
 	 * If the receiving filter is an event, only a removed event is raised
 	 *
-	 * @param id the input filter id for which the values are sent
-	 *
 	 * @param val list of threading::Values expected by the filter
 	 */
-	void Delete(int id, threading::Value* *val);
+	void Delete(threading::Value* *val);
 
 	/**
 	 * Method allowing a reader to clear a value from a bro table.
 	 *
 	 * If the receiving filter is an event, this is ignored.
 	 *
-	 * @param id the input filter id for which the values are sent
 	 */
-	void Clear(int id);
+	void Clear();
 
 	// Content-sending-functions (tracking mode): Only changed lines are propagated.
 	
@@ -243,11 +180,9 @@ protected:
 	 *
 	 * If the filter points to a table, the values are inserted into the table; if it points to an event, the event is raised.
 	 *
-	 * @param id the input filter id for which the values are sent
-	 *
 	 * @param val list of threading::Values expected by the filter
 	 */
-	void SendEntry(int id, threading::Value*  *vals);
+	void SendEntry(threading::Value*  *vals);
 
 	/**
 	 * Method telling the manager, that the current list of entries sent by SendEntry is finished.
@@ -255,9 +190,8 @@ protected:
 	 * For table filters, all entries that were not updated since the last EndCurrentSend will be deleted, because they are no longer
 	 * present in the input source
 	 *
-	 * @param id the input filter id for which the values are sent
 	 */
-	void EndCurrentSend(int id);
+	void EndCurrentSend();
 
 	/**
 	 * Triggered by regular heartbeat messages from the main thread.