Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

System Frame flags not working #8

Open
dudanov opened this issue Apr 4, 2018 · 3 comments
Open

System Frame flags not working #8

dudanov opened this issue Apr 4, 2018 · 3 comments

Comments

@dudanov
Copy link

dudanov commented Apr 4, 2018

Hi.

I'am develop my custom analyzer.

In WorkerThread I mark error frames with: DISPLAY_AS_WARNING_FLAG or DISPLAY_AS_ERROR_FLAG in mFlags, but it's not effect for displayed bubble messges.

Other stuff is excelent working.

Thanks.

@dudanov dudanov changed the title System Frame flags not worked System Frame flags not working Apr 4, 2018
@Marcus10110
Copy link
Contributor

The Logic software does not look at mFlags - it's setup as a convention for analyzers to implement in GenerateBubbleText.

example:

if( frame.mFlags & DISPLAY_AS_ERROR_FLAG )
{
	AddResultString( "!Warning: MAB is too short" );
}

Unfortunately the analyzer SDK does not support emojis, and I suspect that it doesn't handle anything other than vanilla ascii. If I have time I'll check into that to see if there is an easy fix. I know the software supports displaying ⚠ and the round trip between an std::string / QString and the display, so we must be making a mistake somewhere in the analyzer results system - we might be passing it to QString incorrectly, for instance.

@dudanov
Copy link
Author

dudanov commented Apr 5, 2018

From the documentation: "Two flags are reserved by the system, and will produce an error or warning indication on the bubble displaying the Frame.". Proceeding from this, you conclude that the processing of system flags should be performed by the main program, and processing of custom flags should be handled by the plug-in function GenerateBubbleText. Why in this case the reservation of system flags and their announcement in SDK? In my understanding, this should affect the color of Bubble messages, for example.

@Marcus10110
Copy link
Contributor

Thanks for pointing this out, we desperately need to update the documentation for this.

I think when that comment was added, we did plan to do something with mFlags. However, I can confirm that we never implemented a feature that uses it. We also won't be adding this in the future. Many of our analyzers and 3rd party ones use mFlags for analyzer-specific information, so the software can't make any assumptions about what will be found there.

Unfortunately I don't know when we'll have the time - we're a tiny team at the moment, and we're focused mostly on real time view and other feature improvements. We are also working on a system for unit testing protocol analyzers though, which we will open source. (technically it's public at the moment, but it's still in the early stages)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants