| Age | Commit message (Collapse) | Author |
|
CommandDataContainer
std::move within a copy constructor (on a data member that isn't
mutable) will always result in a copy. Because of that, the behavior of
this copy constructor is identical to the one that would be generated
automatically by the compiler, so we can remove it.
|
|
video_core/memory_manager: Minor tidying
|
|
video_core: Remove a few unused variables and functions
|
|
CMakeLists: Specify /volatile:iso for MSVC
|
|
CMakeLists: Explicitly specify -Wall for the non-MSVC case
|
|
Rasterizer Cache: refactor flushing & optimize memory usage of surfaces
|
|
slot.
It seems instructions marked with the 'exit' flag will not cause an exit when executed within a delay slot.
This was hwtested by fincs.
|
|
This corrects cases where it was possible to write more entries into the
write buffer than were requested. Now, we check the size of the buffer
before actually writing into them.
We were also returning the wrong value for
GetAvailableLanguageCodeCount2(). This was previously returning 64, but
only 17 should have been returned. 64 entries is the size of the static
array used in MakeLanguageCode() within the service binary itself, but
isn't the actual total number of language codes present.
|
|
Corrects the typo in its name and marks the function as a const member
function, given it doesn't actually modify memory manager state.
|
|
Prevents implicit converting constructions of the memory manager.
|
|
Makes the class less surprising when it comes to forward declaring the
type, and also prevents inlining the destruction code of the class,
given it contains non-trivial types.
|
|
Corrects references to non-existent parameters and corrects typos.
|
|
These are able to be omitted from the declaration of functions, since
they don't do anything at the type system level. The definitions of the
functions can retain the use of const though, since they make the
variables immutable in the implementation of the function where they're
used.
|
|
Silences a -Wreorder warning.
|
|
|
|
DeclareInternalFlags()
|
|
function
This isn't used at all, so it can be removed.
|
|
UploadGLMipmapTexture()
This variable is unused entirely, so it can be removed.
|
|
Instead of retrieving the data from the std::variant instance, we can
just check if the variant contains that type of data.
This is essentially the same behavior, only it returns a bool indicating
whether or not the type in the variant is currently active, instead of
actually retrieving the data.
|
|
Silences a few compilation warnings.
|
|
By default, MSVC doesn't use standards-compliant volatile semantics.
This makes it behave in a standards-compliant manner, making
expectations more uniform across compilers.
|
|
service/audctl: Update documentation comments to be relative to 8.0.0
|
|
src/CMakeLists: Add /Zc:externConstexpr to the MSVC build flags
|
|
Fix Layered ASTC Textures
|
|
gl_shader_disk_cache: Skip stored shader variants instead of asserting
|
|
Port citra-emu/citra#4749: "web_service: Misc fixes"
|
|
yuzu/hotkeys: Remove unnecessary constructor
|
|
yuzu/compatdb: Remove unnecessary qualifiers
|
|
configure_dialog: Remove the Whats This? button from the dialog
|
|
core/memory: Remove unused FlushMode enum
|
|
Corrections and Implementation on GPU Engines
|
|
|
|
The behavior of the Hotkey constructor is already accomplished via in-class member
initializers, so the constructor is superfluous here.
|
|
For similar reasons to the previous change, we move this to a single
function, so we don't need to duplicate the conversion logic in several
places within main.cpp.
|
|
Specifies the conversions explicitly to avoid implicit conversions from
const char* to QString. This makes it easier to disable implicit QString
conversions in the future.
In this case, the implicit conversion was technically wrong as well. The
implicit conversion treats the input strings as ASCII characters. This
would result in an incorrect conversion being performed in the rare case
a branch name was created with a non-ASCII Unicode character, likely
resulting in junk being displayed.
|
|
Keeps the code consistent in regards to how the buttons are referred to.
|
|
groups
Over time our config values have grown quite numerous in size.
Unfortunately it also makes the single functions we have for loading and
saving values more error prone.
For example, we were loading the core settings twice when they only
should have been loaded once. In another section, a variable was
shadowing another variable used to load settings from a completely
different section.
Finally, in one other case, there was an extraneous endGroup() call used
that didn't need to be done. This was essentially dead code and also a
bug waiting to happen.
This separates the section loading code into its own separate functions.
This keeps variables only visible to the code that actually needs it,
and makes it much easier to visually see the end of each individual
configuration group. It also makes it much easier to visually catch bugs
during code review.
While we're at it, this also uses QStringLiteral instead of raw string
literals, which both avoids constructing a lot of QString instances, but
also makes it much easier to disable implicit ASCII to QString and
vice-versa in the future via setting QT_NO_CAST_FROM_ASCII and
QT_NO_CAST_TO_ASCII as compilation flags.
|
|
The C++ standard allows constexpr variables declared with the extern
keyword to have external linkage. Previously MSVC wasn't abiding by
this. This just makes the compiler more standards compliant during
builds.
Given we currently don't make use of anything that would break by this,
this is safe to enable.
|
|
Makes it much nicer to visually scan the options. This also starts the
flag descriptions from the same column for the same reason.
|
|
Recent changes to memory-related code resulted in this being unused, so
we can remove it.
|
|
There was a weird bug that could happen if the object died directly and
the cache address wasn't stored.
|
|
core/frontend/emu_window: Make GraphicsContext's destructor virtual
|
|
common/zstd_compression: Remove #pragma once directive from source file
|
|
The backend is not used until we decide to submit the testcase/telemetry, and creating it early prevents users from updating the credentials properly while the games are running.
|
|
This isn't used anywhere, so we can get rid of it.
|
|
Ensures that -Wall is always active as a compilation flag.
|
|
Makes use of src, so it's not considered unused.
|
|
Introduced in 72477731ed20c56a4d6f18a22f43224fab667cef. This is only
necessary within header files.
|
|
This class is used in a polymorphic context, so destruction of the
context will lead to undefined behavior if the destructor isn't virtual.
|
|
|