summaryrefslogtreecommitdiff
path: root/src/core (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | patch_manager: Adds check for disabled cheats to prevent them from being ↵Gravatar Morph2019-11-281-5/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | enabled (#3178) * Adds check for disabled cheats to prevent them from being added to the CheatList * Address feedback
* | | | Merge pull request #3170 from lioncash/enumGravatar bunnei2019-11-272-3/+3
|\ \ \ \ | |_|/ / |/| | | file_sys/directory: Make EntryType an enum class
| * | | file_sys/directory: Make EntryType an enum classGravatar Lioncash2019-11-272-3/+3
| |/ / | | | | | | | | | | | | This can trivially be an enum class rather than a regular enum, making it more strongly typed.
* / / core_timing: Use better reference tracking for EventType. (#3159)Gravatar bunnei2019-11-2614-82/+71
|/ / | | | | | | | | | | | | * core_timing: Use better reference tracking for EventType. - Moves ownership of the event to the caller, ensuring we don't fire events for destroyed objects. - Removes need for unique names - we won't be using this for save states anyways.
* | kernel: Fix reference management for client/server session.Gravatar bunnei2019-11-253-20/+18
| | | | | | | | - Fixes shutdown crash and crash in Pokemon SwSh.
* | Merge pull request #3094 from lioncash/tablesGravatar bunnei2019-11-2433-7/+192
|\ \ | | | | | | service: Update function tables
| * | service: Update function tablesGravatar Lioncash2019-11-1233-7/+192
| |/ | | | | | | | | | | Keeps the function tables up to date. Updated based off information from Switchbrew.
* | kernel: Replace usage of boost::intrusive_ptr with std::shared_ptr for ↵Gravatar bunnei2019-11-2470-364/+365
| | | | | | | | | | | | | | | | kernel objects. (#3154) * kernel: Replace usage of boost::intrusive_ptr with std::shared_ptr for kernel objects. - See https://github.com/citra-emu/citra/pull/4710 for details.
* | Update svc.cppGravatar bunnei2019-11-231-0/+1
| |
* | svc: GetSystemTick should return cntpct_el0, not core ticks.Gravatar bunnei2019-11-231-1/+3
| |
* | Merge pull request #3114 from FernandoS27/cond-varGravatar bunnei2019-11-235-22/+74
|\ \ | | | | | | Kernel: Correct behavior of Condition Variables to be more similar to real hardware.
| * | Kernel: Optimize condition variable threads management.Gravatar Fernando Sahmkow2019-11-214-24/+21
| | |
| * | Kernel: Correct SignalProcessWideKeyGravatar Fernando Sahmkow2019-11-211-6/+2
| | | | | | | | | | | | When the target is 0, all threads must be processed.
| * | Kernel: Correct behavior of Condition Variables to be more similar to real ↵Gravatar Fernando Sahmkow2019-11-215-15/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | hardware. This commit ensures cond var threads act exactly as they do in the real console. The original implementation uses an RBTree and the behavior of cond var threads is that at the same priority level they act like a FIFO.
* | | Merge pull request #3130 from FernandoS27/cancel-syncGravatar bunnei2019-11-233-2/+19
|\ \ \ | | | | | | | | Kernel: Correct Cancel Synchronization.
| * | | Kernel: Correct Cancel Synchronization.Gravatar Fernando Sahmkow2019-11-163-2/+19
| | | | | | | | | | | | | | | | | | | | | | | | This commit corrects the behavior of cancel synchronization when the thread is running/ready and ensures the next wait is cancelled as it's suppose to.
* | | | Merge pull request #3112 from lioncash/skipGravatar bunnei2019-11-211-8/+16
|\ \ \ \ | | | | | | | | | | service/am: Remove unnecessary Skip calls
| * | | | service/am: Remove unnecessary Skip callsGravatar Lioncash2019-11-141-8/+16
| |/ / / | | | | | | | | | | | | | | | | We can simplify these by wrapping the necessary members in structs and then simply reading out the whole struct.
* | | | Merge pull request #3111 from lioncash/queryGravatar bunnei2019-11-212-5/+14
|\ \ \ \ | |_|/ / |/| | | am: Stub QueryApplicationPlayStatistics
| * | | am: Stub QueryApplicationPlayStatisticsGravatar Lioncash2019-11-142-5/+14
| |/ / | | | | | | | | | | | | | | | | | | | | | Maintains implementation parity between QueryApplicationPlayStatistics and QueryApplicationPlayStatisticsByUid. These function the same behaviorally underneath the hood, with the only difference being that one allows specifying a UID.
* | | Merge pull request #3091 from lioncash/core-conversionGravatar bunnei2019-11-1535-170/+182
|\ \ \ | |/ / |/| | core: Make most implicit type conversion warnings errors on MSVC
| * | externals: Update httplibGravatar Lioncash2019-11-121-1/+1
| | | | | | | | | | | | | | | | | | | | | Since the introduction of this library, numerous improvements have been made. Notably, many of the warnings we would get by simply including the library header have now been fixed. This makes it much easier to make conversion warning an error.
| * | service: Resolve sign conversion errorsGravatar Lioncash2019-11-1215-58/+55
| | | | | | | | | | | | | | | These are fairly trivial to resolve and most of the changes entail using RESULT_UNKNOWN over ResultCode(-1).
| * | perf_stats: Resolve implicit int to double conversion errorGravatar Lioncash2019-11-121-1/+1
| | | | | | | | | | | | | | | We simply need to turn the literal argument to std::accumulate into a double, rather than an int.
| * | loader; Resolve sign conversion/truncation errorsGravatar Lioncash2019-11-123-6/+6
| | |
| * | gdbstub: Resolve sign conversion errorsGravatar Lioncash2019-11-121-1/+2
| | |
| * | kernel: Resolve sign conversion warningsGravatar Lioncash2019-11-124-72/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | Uncovered a bug within Thread's SetCoreAndAffinityMask() where an unsigned variable (ideal_core) was being compared against "< 0", which would always be a false condition. We can also get rid of an unused function (GetNextProcessorId) which contained a sign mismatch warning.
| * | file_sys: Resolve sign conversion warningsGravatar Lioncash2019-11-124-12/+10
| | | | | | | | | | | | Resolves a few trivial sign conversion/mismatch errors.
| * | result: Add default error code for the ResultCode(-1) caseGravatar Lioncash2019-11-121-1/+9
| | | | | | | | | | | | | | | Will be used to reduce the overall duplication of the same magic value all over the codebase in following changes.
| * | crypto: Resolve sign-conversion warningsGravatar Lioncash2019-11-122-11/+12
| | |
| * | result: Resolve sign-coversion warningsGravatar Lioncash2019-11-121-1/+1
| | | | | | | | | | | | | | | The constructor was implicitly using signed->unsigned conversions to produce 0xFFFFFFFF. We can just specify this explicitly with UINT32_MAX.
| * | arm_unicorn: Resolve sign conversion warningsGravatar Lioncash2019-11-123-8/+10
| | | | | | | | | | | | | | | While we're at it, this also resolves a type truncation warning as well, given the code was truncating from a 64-bit value to a 32-bit one.
| * | CMakeLists: Make most implicit type conversion warnings errors on MSVCGravatar Lioncash2019-11-121-0/+17
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Quite frequently there have been cases where code has been merged into the core that produces warning. In order to prevent this from occurring, we can make the compiler flag these cases and allow our CI to flag down any code that would generate these warnings. This is beneficial given silent conversions from signed/unsigned can result in logic bugs. This forces one writing changes to be explicit about when signedness conversions are desirable, rather than leaving it up to readers' interpretation. Currently the codebase isn't in a state where it will build successfully with this change applied, but this will be addressed in subsequent follow-up changes. This set of changes will focus on making it build properly with these changes for MSVC as a starting point for basic coverage.
* | Merge pull request #3089 from SciresM/play_statisticsGravatar bunnei2019-11-142-0/+10
|\ \ | | | | | | Implement stub for IApplicationFunctions::QueryApplicationPlayStatisticsByUid
| * | Implement stub for QueryApplicationPlayStatisticsByUidGravatar Michael Scire2019-11-112-0/+10
| |/
* | Merge pull request #3093 from lioncash/mbedtlsGravatar bunnei2019-11-147-12/+12
|\ \ | | | | | | core: Migrate off deprecated mbedtls functions
| * | core: Migrate off deprecated mbedtls functionsGravatar Lioncash2019-11-127-12/+12
| |/ | | | | | | | | These functions are marked for deprecation and it's recommended that the *_ret variants be used instead.
* | Merge pull request #3092 from lioncash/utilGravatar bunnei2019-11-141-11/+15
|\ \ | | | | | | key_manager: Make use of IOFile in WriteKeyToFile()
| * | key_manager: Make use of IOFile in WriteKeyToFile()Gravatar Lioncash2019-11-121-11/+15
| |/ | | | | | | | | | | | | | | | | This properly handles unicode-based paths on Windows, while opening a raw stream doesn't out-of-the-box. Prevents file creation from potentially failing on Windows PCs that make use of unicode characters in their save paths (e.g. writing to a user's AppData folder, where the user has a name with non-ASCII characters).
* / xts_archive: Remove redundant std::string constructorGravatar Lioncash2019-11-131-2/+1
|/ | | | | We can just call the .data() member of path instead of constructing a completely new string.
* Merge pull request #3062 from bunnei/event-improveGravatar bunnei2019-11-0623-87/+53
|\ | | | | kernel: Improve events
| * kernel: readable_event: Signal only once.Gravatar bunnei2019-11-031-2/+4
| |
| * kernel: events: Remove ResetType::Automatic.Gravatar bunnei2019-11-0323-84/+48
| | | | | | | | | | | | | | | | - This does not actually seem to exist in the real kernel - games reset these automatically. # Conflicts: # src/core/hle/service/am/applets/applets.cpp # src/core/hle/service/filesystem/fsp_srv.cpp
| * kernel: readable_event: Initialize members.Gravatar bunnei2019-11-031-1/+1
| |
* | Merge pull request #2859 from Morph1984/hidGravatar David2019-11-062-92/+126
|\ \ | | | | | | hid: Stub SetNpadJoyAssignmentModeSingle and GetNpadHandheldActivationMode
| * | hid: Stub SetNpadJoyAssignmentModeSingle and reorganize service commandsGravatar Morph2019-10-072-92/+126
| | |
* | | common_func: Use std::array for INSERT_PADDING_* macros.Gravatar bunnei2019-11-035-38/+39
| | | | | | | | | | | | - Zero initialization here is useful for determinism.
* | | core/am: Stub InitializeApplicationCopyrightFrameBuffer, ↵Gravatar FearlessTobi2019-11-032-3/+31
| |/ |/| | | | | | | | | SetApplicationCopyrightImage and SetApplicationCopyrightVisibility These commands require Screenshots to be implemented anyway, so they are safe to stub for now.
* | Merge pull request #3038 from lioncash/docsGravatar Rodrigo Locatti2019-10-302-91/+73
|\ \ | | | | | | kernel/scheduler: Minor changes
| * | scheduler: Mark parameter of AskForReselectionOrMarkRedundant() as constGravatar Lioncash2019-10-272-5/+5
| | | | | | | | | | | | This is only compared against, so it can be made const.
HIC SVNT DRACONES