Skip to content
This repository has been archived by the owner on May 25, 2019. It is now read-only.

Refactoring/toxencryptsave #13

Merged
merged 5 commits into from
Jun 12, 2016
Merged

Refactoring/toxencryptsave #13

merged 5 commits into from
Jun 12, 2016

Conversation

kpp
Copy link
Owner

@kpp kpp commented Feb 9, 2016

No description provided.

@kpp kpp force-pushed the refactoring/toxencryptsave branch 2 times, most recently from ffcedf6 to eadbfd1 Compare February 10, 2016 01:04
@kpp kpp force-pushed the refactoring/toxencryptsave branch 12 times, most recently from 09181f3 to 4375691 Compare February 10, 2016 17:11
// hardcoded constants
// if they change, userdata will be lost

#define TOXENCRYPTSAVE_MAGIC_NUMBER "toxEsave"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NUMBER?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@kpp kpp force-pushed the refactoring/toxencryptsave branch from 4375691 to 98e7507 Compare February 14, 2016 22:44
virtual void SetUp() {
pplength = s_pass_len;
passphrase = new uint8_t[ pplength ];
memmove(passphrase, s_pass, pplength);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better use std::copy

@coveralls
Copy link

coveralls commented May 19, 2016

Coverage Status

Coverage increased (+0.9%) to 95.531% when pulling 09181f3 on refactoring/toxencryptsave into 26988cf on develop.

@kpp kpp merged commit af5cd01 into develop Jun 12, 2016
@kpp kpp deleted the refactoring/toxencryptsave branch June 12, 2016 10:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants