Remove unnecessary import from OK reply

Since we no longer inherit status replies from the base reply class but
rather from the status reply class, we no longer need to import the
header for it.

Signed-off-by: Marcus Karlsson <mk@acc.umu.se>
Always acquire lock when adding data

We use a conditional lock for synchronized access to the staging area
for unparsed data. When adding data we only lock when the condition is
set to zero, and the condition is only set to zero when the parser
thread is done with it. What this means however is that if we call
addData repeatedly before the parser thread has reset the condition the
second call will block until the parser thread is done, even if the
parser thread hasn't even acquired the lock yet.

Always allow access from addData no matter what the condition is. Only
block when the parser thread is using the lock. This will allow addData
to be called multiple times without blocking the caller.

Signed-off-by: Marcus Karlsson <mk@acc.umu.se>
Remove redundant synchronization block

Access to unparsedStage is controlled by a conditional lock. This makes
the @synchronized block redundant since it is only used where we already
synchronize access. Remove the unnecessary synchronization block.

Signed-off-by: Marcus Karlsson <mk@acc.umu.se>