Skip to content

return true sketch size using ESP.getSketchSize() #2154

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

Closed
wants to merge 2 commits into from

Conversation

sticilface
Copy link
Contributor

return true sketch size using ESP.getSketchSize()
#2153

add MD5 for current binary ESP.getSketchMD5()

@codecov-io
Copy link

codecov-io commented Jun 15, 2016

Current coverage is 27.61%

Merging #2154 into master will not change coverage

@@             master      #2154   diff @@
==========================================
  Files            20         20          
  Lines          3686       3686          
  Methods         337        337          
  Messages          0          0          
  Branches        678        678          
==========================================
  Hits           1018       1018          
  Misses         2490       2490          
  Partials        178        178          

Powered by Codecov. Last updated by 44d2722...420c18d

@sticilface
Copy link
Contributor Author

just realised that my md5 method is pants! it does work but remainder bit needs fiddling

@themindfactory
Copy link
Contributor

do you release the 512 byte buffer?

On Wed, Jun 15, 2016 at 11:45 AM, sticilface notifications@github.com
wrote:

just realised that my md5 method is pants! it does work but remainder bit
needs fiddling


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#2154 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AINdaUOOwRc09gXgEZYFLrcPtNSBkoe4ks5qMB4ugaJpZM4I2dKl
.

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

Successfully merging this pull request may close these issues.

3 participants