summaryrefslogtreecommitdiff
path: root/src/core (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #806 from lioncash/friendGravatar bunnei2018-07-246-48/+15
|\ | | | | friend: Deduplicate interfaces
| * friend: Add friend:m, friend:s, and friend:v servicesGravatar Lioncash2018-07-241-0/+3
| | | | | | | | | | Given we already have friend:a and friend:u, we should add the remaining services as well.
| * friend/interface: Add missing CreateDaemonSuspendSessionService() to the ↵Gravatar Lioncash2018-07-241-0/+1
| | | | | | | | function handler table
| * friend: Deduplicate interfacesGravatar Lioncash2018-07-246-48/+11
| |
* | Merge pull request #805 from lioncash/signGravatar bunnei2018-07-241-4/+7
|\ \ | | | | | | svc: Resolve sign comparison warnings in WaitSynchronization()
| * | svc: Resolve sign comparison warnings in WaitSynchronization()Gravatar Lioncash2018-07-241-4/+7
| |/ | | | | | | | | The loop's induction variable was signed, but we were comparing against an unsigned variable.
* / deconstructed_rom_directory: Remove unused FindRomFS() functionGravatar Lioncash2018-07-241-29/+0
|/
* Merge pull request #798 from lioncash/constGravatar bunnei2018-07-242-3/+3
|\ | | | | arm_dynarmic: Make MakeJit() a const member function
| * arm_dynarmic: Make MakeJit() a const member functionGravatar Lioncash2018-07-232-3/+3
| | | | | | | | | | This functions doesn't modify instance state, so it can be a made a const member function.
* | Merge pull request #797 from lioncash/explicitGravatar bunnei2018-07-245-5/+5
|\ \ | | | | | | core: Make converting constructors explicit where applicable
| * | core: Make converting constructors explicit where applicableGravatar Lioncash2018-07-235-5/+5
| |/ | | | | | | | | Avoids unwanted implicit conversions. Thankfully, given the large amount of cleanup in past PRs, only this tiny amount is left over to cover.
* | Merge pull request #795 from lioncash/declGravatar bunnei2018-07-241-3/+0
|\ \ | | | | | | apm/interface: Remove redundant declaration of InstallInterfaces()
| * | apm/interface: Remove redundant declaration of InstallInterfaces()Gravatar Lioncash2018-07-231-3/+0
| |/ | | | | | | This is already declared in apm/apm.h
* | Merge pull request #794 from lioncash/refGravatar bunnei2018-07-241-1/+1
|\ \ | | | | | | mutex: Pass SharedPtr to GetHighestPriorityMutexWaitingThread() by reference
| * | mutex: Pass SharedPtr to GetHighestPriorityMutexWaitingThread() by referenceGravatar Lioncash2018-07-231-1/+1
| | | | | | | | | | | | | | | The pointed to thread's members are simply observed in this case, so we don't need to copy it here.
* | | Merge pull request #793 from lioncash/privGravatar bunnei2018-07-232-17/+19
|\ \ \ | | | | | | | | ipc_helpers: Make member variables of ResponseBuilder private
| * | | hle_ipc: Make constructors explicit where applicableGravatar Lioncash2018-07-232-12/+13
| | | |
| * | | ipc_helpers: Make member variables of ResponseBuilder privateGravatar Lioncash2018-07-231-5/+6
| |/ / | | | | | | | | | These aren't used externally at all, so they can be made private.
* | | Merge pull request #785 from lioncash/fsGravatar bunnei2018-07-231-3/+3
|\ \ \ | |_|/ |/| | partition_filesystem: Use std::move where applicable
| * | partition_filesystem: Use std::move where applicableGravatar Lioncash2018-07-231-3/+3
| | | | | | | | | | | | | | | Avoids copying a std::string instance and avoids unnecessary atomic reference count incrementing and decrementing.
* | | VFS Regression and Accuracy Fixes (#776)Gravatar Zach Hilman2018-07-233-36/+62
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Regression and Mode Fixes * Review Fixes * string_view correction * Add operator& for FileSys::Mode * Return std::string from SanitizePath * Farming Simulator Fix * Use != With mode operator&
* | Merge pull request #786 from lioncash/exclusiveGravatar bunnei2018-07-233-22/+18
|\ \ | | | | | | exclusive_monitor: Use consistent type alias for u64
| * | exclusive_monitor: Use consistent type alias for u64Gravatar Lioncash2018-07-233-22/+18
| |/ | | | | | | | | Uses the same type aliases we use for virtual addresses, and converts one lingering usage of std::array<uint64_t, 2> to u128 for consistency.
* | Merge pull request #784 from lioncash/loaderGravatar bunnei2018-07-231-1/+1
|\ \ | | | | | | loader: Minor cleanup
| * | loader: Remove unnecessary constructor call in IdentifyFile()Gravatar Lioncash2018-07-231-1/+1
| |/ | | | | | | | | | | RealVfsFile inherits from VfsFile, the instance from std::make_shared is already compatible with the function argument type, making the copy constructor call unnecessary.
* | Merge pull request #783 from lioncash/linkerGravatar bunnei2018-07-232-7/+4
|\ \ | | | | | | linker: Remove unused parameter from WriteRelocations()
| * | linker: Remove unused parameter from WriteRelocations()Gravatar Lioncash2018-07-232-7/+4
| |/ | | | | | | | | is_jump_relocation is never used within the function, so we can just remove it.
* | Merge pull request #782 from lioncash/fileGravatar bunnei2018-07-232-14/+33
|\ \ | | | | | | loader/nro: Minor changes
| * | nro: Replace inclusion with a forward declarationGravatar Lioncash2018-07-232-1/+8
| | | | | | | | | | | | | | | It's sufficient to use a forward declaration instead of a direct inclusion here.
| * | nro: Make bracing consistentGravatar Lioncash2018-07-231-10/+24
| | | | | | | | | | | | | | | Makes the code more uniform, and also braces cases where the body of an unbraced conditional travels more than one line.
| * | nro: Make constructor explicitGravatar Lioncash2018-07-231-1/+1
| | | | | | | | | | | | | | | Makes it consistent with the other Apploader constructors, and prevents implicit conversions.
| * | nro: Remove unused forward declarationGravatar Lioncash2018-07-231-2/+0
| |/ | | | | | | This isn't used anywhere in the header.
* | Merge pull request #780 from lioncash/moveGravatar bunnei2018-07-231-11/+22
|\ \ | | | | | | vi: Minor changes
| * | vi: Add std::is_trivially_copyable checks to Read and Write functionsGravatar Lioncash2018-07-231-2/+13
| | | | | | | | | | | | | | | | | | It's undefined behavior to memcpy an object that isn't considered trivially copyable, so put a compile-time check in to make sure this doesn't occur.
| * | vi: std::move std::vector in constructors where applicableGravatar Lioncash2018-07-231-9/+9
| | | | | | | | | | | | | | | | | | | | | Allows avoiding unnecessary copies of the vector depending on the calling code. While we're at it, remove a redundant no-parameter base constructor call
* | | Merge pull request #779 from lioncash/sharedGravatar bunnei2018-07-238-263/+0
|\ \ \ | |_|/ |/| | hle: Remove unused config_mem and shared_page source files
| * | hle: Remove config_mem.h/.cppGravatar Lioncash2018-07-236-102/+0
| | | | | | | | | | | | | | | This is just an unused hold-over from citra, so we can get rid of this to trim off an exposed global, among other things.
| * | hle: Remove shared_page.h/.cppGravatar Lioncash2018-07-236-161/+0
| |/ | | | | | | This is a holdover from citra that's essentially unused.
* | Merge pull request #695 from DarkLordZach/nro-assetGravatar bunnei2018-07-235-1/+215
|\ \ | | | | | | NRO Assets and NACP File Format
| * | NRO Assets and NACP file formatGravatar Zach Hilman2018-07-235-1/+215
| | | | | | | | | | | | | | | | | | Cleanup Review fixes
* | | set: Add missing log call in GetAvailableLanguageCodeCount()Gravatar Lioncash2018-07-231-0/+2
| |/ |/| | | | | Forgot to include this in 22f448b6327044076959e338811ee576f3dcf093
* | Merge pull request #777 from lioncash/langGravatar bunnei2018-07-232-23/+31
|\ \ | |/ |/| set: Amend return value of GetAvailableLanguageCodes()
| * set: Implement GetAvailableLanguageCodeCount()Gravatar Lioncash2018-07-232-21/+29
| | | | | | | | This just returns the size of the language code buffer.
| * set: Correct return code size of value in GetAvailableLanguageCodes()Gravatar Lioncash2018-07-231-2/+2
| | | | | | | | The return code should be 32-bit in size.
* | Merge pull request #774 from Subv/atomic_signalGravatar bunnei2018-07-221-7/+31
|\ \ | |/ |/| Kernel/SVC: Perform atomic accesses in SignalProcessWideKey as per the real kernel.
| * Kernel/SVC: Perform atomic accesses in SignalProcessWideKey as per the real ↵Gravatar Subv2018-07-221-7/+31
| | | | | | | | kernel.
* | Merge pull request #768 from lioncash/string-viewGravatar bunnei2018-07-228-93/+158
|\ \ | |/ |/| file_util, vfs: Use std::string_view where applicable
| * vfs: Correct file_p variable usage within InterpretAsDirectory()Gravatar Lioncash2018-07-221-2/+5
| | | | | | | | | | | | | | | | | | ReplaceFileWithSubdirectory() takes a VirtualFile and a VirtualDir, but it was being passed a string as one of its arguments. The only reason this never caused issues is because this template isn't instantiated anywhere yet. This corrects an issue before it occurs.
| * file_util, vfs: Use std::string_view where applicableGravatar Lioncash2018-07-228-91/+153
| | | | | | | | | | Avoids unnecessary construction of std::string instances where applicable.
* | Implement exclusive monitorGravatar MerryMage2018-07-229-13/+160
|/