Code review comment for lp:~vijit.chauhan/beeseek/fix-603119

Revision history for this message
vSC (vijit.chauhan) wrote :

Hi,

I wasn't very comfortable with the recursive approach.

I have done the fix so that can be avoided. :)

Introduced a new function BfSender_MakeConnection() which takes care of the just the connection part and can be called again by BfSender_SendRaw(), in case of any error in connection.

I guess the changes should not break anything, just add a little bit more flexibility.

Reg.
Vijit

« Back to merge proposal