Skip to content
This repository was archived by the owner on Jul 6, 2021. It is now read-only.

Commit cdbb92d

Browse files
author
Joe Freeman
committed
Remove capture queue for default observer set
tests and apps can add one if they needed
1 parent 0dcf1cd commit cdbb92d

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

IOIOLib/Device/Impl/IOIOImpl.cs

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,10 +77,6 @@ public class IOIOImpl : IOIO
7777
/// our current board configuration
7878
/// </summary>
7979
private ObserverConnectionState CapturedConnectionInformation_;
80-
/// <summary>
81-
/// really only used for debugging. Retains string information about messages received
82-
/// </summary>
83-
private ObserverLog CapturedLogs_;
8480

8581
private IResourceManager BoardResourceManager_;
8682

@@ -143,9 +139,7 @@ private void ConfigureHandlers(IIncomingHandlerIOIO customHandler, List<IObserve
143139
//CaptureObservable_ = new IOIOHandlerObservable();
144140
CaptureObservable_ = new IOIOHandlerObservableNoWait();
145141
CapturedConnectionInformation_ = new ObserverConnectionState();
146-
CapturedLogs_ = new ObserverLog(10);
147142
CaptureObservable_.Subscribe(CapturedConnectionInformation_);
148-
CaptureObservable_.Subscribe(CapturedLogs_);
149143
// observers using the default handler threading model
150144
if (observers != null)
151145
{
@@ -313,6 +307,7 @@ public virtual void run()
313307
// notify observers we are ABOUT to send a message
314308
// the observers can block the send if they are
315309
// waiting for device bus internal buffer space
310+
// this only works if the observer is a blocking observer
316311
// this is DIFFERENT than the Java library which hanldes puts each bus buffer in own payload queue
317312
IMessageIOIO possibleNotifyableMessage = nextMessage as IMessageIOIO;
318313
// post notifications down our internal chain

0 commit comments

Comments
 (0)