Merge pull request #511 from lioncash/warnings

Fix a few warnings.
This commit is contained in:
bunnei 2015-01-30 13:57:08 -05:00
commit c8c3a94b20
3 changed files with 4 additions and 4 deletions

View File

@ -373,7 +373,7 @@ void SetAppCpuTimeLimit(Service::Interface* self) {
u32 percent = cmd_buff[2]; u32 percent = cmd_buff[2];
if (value != 1) { if (value != 1) {
LOG_ERROR(Service_APT, "This value must be one!", value); LOG_ERROR(Service_APT, "This value should be one, but is actually %u!", value);
} }
cmd_buff[1] = RESULT_SUCCESS.raw; // No error cmd_buff[1] = RESULT_SUCCESS.raw; // No error
@ -395,7 +395,7 @@ void GetAppCpuTimeLimit(Service::Interface* self) {
u32 value = cmd_buff[1]; u32 value = cmd_buff[1];
if (value != 1) { if (value != 1) {
LOG_ERROR(Service_APT, "This value must be one!", value); LOG_ERROR(Service_APT, "This value should be one, but is actually %u!", value);
} }
// TODO(purpasmart96): This is incorrect, I'm pretty sure the percentage should // TODO(purpasmart96): This is incorrect, I'm pretty sure the percentage should

View File

@ -89,7 +89,7 @@ public:
class File : public Kernel::Session { class File : public Kernel::Session {
public: public:
File(std::unique_ptr<FileSys::FileBackend>&& backend, const FileSys::Path& path) File(std::unique_ptr<FileSys::FileBackend>&& backend, const FileSys::Path& path)
: path(path), backend(std::move(backend)), priority(0) { : path(path), priority(0), backend(std::move(backend)) {
} }
std::string GetName() const override { return "Path: " + path.DebugStr(); } std::string GetName() const override { return "Path: " + path.DebugStr(); }

View File

@ -150,7 +150,7 @@ ResultStatus LoadFile(const std::string& filename) {
// IdentifyFile could know identify file type... // IdentifyFile could know identify file type...
case FileType::Unknown: case FileType::Unknown:
{ {
LOG_CRITICAL(Loader, "File %s is of unknown type."); LOG_CRITICAL(Loader, "File %s is of unknown type.", filename.c_str());
return ResultStatus::ErrorInvalidFormat; return ResultStatus::ErrorInvalidFormat;
} }
} }