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

RS crashes when getting empty detect query #169

Closed
jworch opened this issue Mar 28, 2019 · 2 comments
Closed

RS crashes when getting empty detect query #169

jworch opened this issue Mar 28, 2019 · 2 comments

Comments

@jworch
Copy link
Collaborator

jworch commented Mar 28, 2019

query: {"detect":""}

->

run: /u/ropha/git/robosherlock_ws/src/robosherlock/rapidjson_ros/include/rapidjson/include/rapidjson/document.h:1081: rapidjson::GenericValue<Encoding, Allocator>::MemberIterator rapidjson::GenericValue<Encoding, Allocator>::MemberBegin() [with Encoding = rapidjson::UTF8<>; Allocator = rapidjson::MemoryPoolAllocator<>; rapidjson::GenericValue<Encoding, Allocator>::MemberIterator = rapidjson::GenericMemberIterator<false, rapidjson::UTF8<>, rapidjson::MemoryPoolAllocator<> >]: Assertion IsObject() failed.

@bbferka
Copy link
Collaborator

bbferka commented Mar 29, 2019

@jworch dumb mistake, not checking in the code before. Is this with the latest devel? I'll fix it with the next PR.

@jworch
Copy link
Collaborator Author

jworch commented Mar 29, 2019

@bbferka Ignore that for the moment. That error occurred with the older code base we use in ropha. I haven't tested that with the current version yet.

@bbferka bbferka mentioned this issue Jul 2, 2019
bbferka added a commit that referenced this issue Jul 2, 2019
* changes to how RSProcessManage is constructed (no more init function)

* a bit of cleanup in the code

* fix #169 and #135
@bbferka bbferka closed this as completed Jul 2, 2019
bbferka added a commit that referenced this issue Jul 3, 2019
* changes to how RSProcessManage is constructed (no more init function)

* a bit of cleanup in the code

* fix #169 and #135
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