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
TIMOB-6187: Android: Re-enable Fastdev for V8 / Rhino #713
Conversation
into KrollAssetHelper.readAsset, moved some dependencies into kroll-common. TIMOB-6187
…kroll-v8.jar TIMOB-6187
… timob-6187 Conflicts: android/titanium/src/java/org/appcelerator/titanium/util/TiDownloadManager.java
Pull Accepted: |
|
||
totalCount += count; | ||
if (out != null) { | ||
out.write(buffer, 0, count); |
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.
At this point, you could technically be writing beyond byteCount
. Example: total stream is 1030 bytes, bufferSize
is 1024 bytes and byteCount
(the desired number of bytes) is 1028. The second iteration of this loop would grab 1025-1030 and put it in out
, even though only up to 1028 was desired.
from KrollStreamHelper.pumpCount. TIMOB-6187
Code review + functional test accepted |
… timob-6187 Conflicts: android/runtime/common/src/java/org/appcelerator/kroll/KrollApplication.java
readDeployData(deployData); | ||
|
||
if (enabled && fastDevSocket != null) | ||
{ |
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.
Move curly onto the same line as the "if" statement.
Code reviewed and accepted PENDING resolution of comments for TiFastDev::102 and KrollStreamHelper::65 |
Code reviewed. accepted |
TIMOB-6187: Android: Re-enable Fastdev for V8 / Rhino
http://jira.appcelerator.org/browse/TIMOB-6187
Testing instructions are on the JIRA issue