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
(Partial) TIMOB-9543: BlackBerry: Implement BufferStream #133
(Partial) TIMOB-9543: BlackBerry: Implement BufferStream #133
Conversation
[Issues Fixed] (Partial) TIMOB-9543: BlackBerry: Implement BufferStream (Partial) TIMOB-10257: BlackBerry: Implement Titanium.IOStream [Added] TiBufferStreamObject.cpp TiBufferStreamObject.h TiIOStreamObject.cpp TiIOStreamObject.h TiStreamObject.cpp TiStreamObject.h [Changes] TiTitaniumObject.cpp - function onCreateStaticMembers - add TiBufferStreamObject and TiStreamObject bootstrap.js - add temporary code to show how to add a function to the API using JavaScript [Tests] Test 1: Test new JavaScript function var bs = Ti.Stream.createStream(); bs.demoFunc(); Expected: Should print the following: [DEBUG]:demoFunc is being run
} | ||
|
||
Handle<Value> TiStreamObject::_createStream(void* /*userContext*/, TiObject* /*caller*/, const Arguments& args) | ||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
according to documentation , creatStream should take one and only one argument, do we need to check it here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be done in a later PR.
Is it what we need to make anvil run? (don't we need to implement functions, properties for buffersstream and iostream in this jira task?) |
Reviewed |
This is just a partial patch. the functions, etc will come later in another PR. |
(Partial) TIMOB-10257: BlackBerry: Implement Titanium.IOStream updated apidoc fixed onCreateStaticMembers calls in TiBufferStreamObject and TiIOStreamObject
Updated patch for apidoc and onCreateStaticMembers |
Approved |
Conflicts: blackberry/tibb/TiTitaniumObject.cpp
(Partial) TIMOB-9543: BlackBerry: Implement BufferStream
Reviewer: Harut