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

fix build warning of using c++ when build multirom.cpp #28

Open
wants to merge 3 commits into
base: android-6.0
Choose a base branch
from

Conversation

sndnvaps
Copy link

fix build warning of using c++ when build multirom.cpp

  1. Test MS_RELATIME if we has define in android source
  2. Fix build error when TW_USE_TOOLBOX := true
  3. Fix build issuse of f2fs when build with MoKee Open Source
  4. Fix build warning of twrp-functions.cpp

  with TW_USE_TOOLBOX := true in BoardConfig.mk

Signed-off-by: sndnvaps <sndnvaps@gmail.com>
 test MS_RELATIME if we has define in android source
 move MR_UPDATE_SCRIPT_PATH && MR_UPDATE_SCRIPT_NAME from multirom.cpp to multirom.h
 fix build warning of twrp-functions.cpp

Signed-off-by: sndnvaps <sndnvaps@gmail.com>
@sndnvaps sndnvaps changed the title Patch set fix build warning of using c++ when build multirom.cpp Feb 23, 2016
@@ -177,8 +182,8 @@ bool MultiROM::setRomsPath(std::string loc)
return false;
}

m_curr_roms_path = "/mnt/multirom-"TARGET_DEVICE"/";
mkdir("/mnt/multirom-"TARGET_DEVICE"/", 0777);
m_curr_roms_path = "/mnt/multirom-" + (string)TARGET_DEVICE + "/";
Copy link
Owner

Choose a reason for hiding this comment

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

Don't cast it to string, just put a space in between the quotatuion marks and TARGET_DEVICE define.

Fix build warning when using c++ compiler

Signed-off-by: sndnvaps <sndnvaps@gmail.com>
@sndnvaps
Copy link
Author

@Tasssadar i have fix the problem you just promote.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants