aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Remove unneeded CharArray:sPierre Ossman2023-02-043-22/+18
| | | | | Avoid complicating things by moving things in to a second buffer here as there is no need for it.
* Use StringParameters directlyPierre Ossman2023-02-0420-75/+40
| | | | | We don't need to make extra copies of the string in most cases, so let's simplify the code and access the string directly when we can.
* Get rid of TCHAR magicPierre Ossman2023-02-0468-700/+453
| | | | | We never use Windows' "UNICODE" mode anyway, so let's get rid of this complexity.
* Return std::vector instead of dynamic allocationPierre Ossman2023-02-048-40/+35
| | | | | This makes memory management more clear and robust when using these helper functions.
* Avoid duplicating configuration settingPierre Ossman2023-02-041-16/+7
| | | | | Let the string helpers call the more fancy setParam(). This makes sure we can avoid duplicating things.
* Make sure length is reset on assignmentPierre Ossman2023-02-041-1/+3
| | | | | Otherwise the old length is preserved, which will result in NULL pointer dereferencing if the parameter is ever accessed.
* Clean up BinaryParameter typingPierre Ossman2023-02-043-16/+21
| | | | | This is explicitly a byte sequence, so let's try to keep a consistent typing.
* Move hex conversion helpers to utilPierre Ossman2023-02-049-88/+109
| | | | | These are used here and there so let's make them more general rather than hiding them in the stream classes.
* Use std::vector for basic data arraysPierre Ossman2023-02-0113-212/+150
| | | | | Avoid our own custom types in favour of what's already included with C++.
* Use stdint typesPierre Ossman2023-02-01151-1086/+1114
| | | | | Avoid having our own custom stuff and instead use the modern, standard types, for familiarity.
* Use templates for optimized codePierre Ossman2023-02-0131-2062/+1582
| | | | | Avoid preprocessor magic and instead rely on templating to generate optimized functions for performance critical code.
* Fix CRLF line endingsPierre Ossman2023-01-175-442/+442
| | | | Everything else uses LF line endings, so fix up the few stray ones.
* Merge branch 'libtool' of https://github.com/CendioOssman/tigervncPierre Ossman2023-01-102-25/+79
|\
| * Create .la files at the end of the cmake runPierre Ossman2023-01-102-11/+34
| | | | | | | | | | We might not have all the necessary information, e.g. all targets might not exist yet, until we're done going through all CMakeLists.
| * Bail on missing librariesPierre Ossman2023-01-101-2/+2
| | | | | | | | They may be crucial, so refuse to continue if this happens.
| * Fall back to dynamic libraries in .la filesPierre Ossman2023-01-101-3/+7
| | | | | | | | | | This is what the linker does, so we should do the same for correct behaviour.
| * Respect library search pathsPierre Ossman2023-01-101-1/+7
| | | | | | | | | | Things might be in odd places, so make sure we respect where we're told to look for libraries.
| * Include other targets in .la filesPierre Ossman2023-01-101-2/+5
| | | | | | | | | | | | We have internal dependencies that should be respected, as otherwise it will be up to the user of the .la file to figure out the correct order of the libraries.
| * Handle alternative library suffixesPierre Ossman2023-01-101-1/+9
| | | | | | | | | | I think this only affects macOS at the moment where they also have .tbd files for linking.
| * Make sure frameworks are included in .la filesPierre Ossman2023-01-101-1/+9
| | | | | | | | | | We don't really use the .la files for macOS at the moment, but let's try to be prepared.
| * Remove unnecessary regexp \\Pierre Ossman2023-01-101-3/+3
| | | | | | | | | | This serves no useful purpose as we have no reason to suspect there will be a dangerous first character in these variables.
| * Fix static library check regexpPierre Ossman2023-01-101-1/+1
| | | | | | | | | | Not sure how the old one ever worked as it incorrectly just tried to match last character and nothing else.
| * Get target libraries from INTERFACE_LINK_LIBRARIESPierre Ossman2023-01-101-4/+6
| | | | | | | | | | <target>_LIB_DEPENDS is an internal variable that contains lots of other weird stuff.
| * Use correct FATAL_ERROR CMake optionPierre Ossman2023-01-101-1/+1
|/
* Merge branch 'wextra' of https://github.com/CendioOssman/tigervncPierre Ossman2023-01-05125-584/+388
|\
| * Use __attribute__:s directlyPierre Ossman2023-01-059-65/+34
| | | | | | | | | | | | These are supported by gcc and clang, and I don't think MSVC works well with our code anyway, so let's keep things simple and use these directly.
| * Enable more warnings by defaultPierre Ossman2023-01-051-2/+2
| | | | | | | | | | | | | | | | These are also useful warnings that tend to find real bugs, so let's enable then. The downside is they require us to mark certain things so the compiler knows if they were on purpose or not.
| * Work around -Wcast-function-typePierre Ossman2023-01-052-10/+10
| | | | | | | | | | That warning doesn't play well with Windows' GetProcAddress(), so add some extra casting to work around it.
| * Remove old mingw compatibility codePierre Ossman2023-01-054-223/+0
| | | | | | | | | | This isn't needed with modern build environments, and can even conflict with them.
| * Don't partially init monitor informationPierre Ossman2023-01-051-1/+1
| | | | | | | | gcc will complain, and we will fill out all fields later anyway.
| * Fix incorrect typesPierre Ossman2023-01-054-8/+8
| | | | | | | | | | These types caused an incorrect signed/unsigned behaviour, so let's make sure we use the appropriate type.
| * Fix order of qualifiersPierre Ossman2023-01-051-1/+1
| | | | | | | | gcc can get upset if they aren't in this order.
| * Protect key variables from setjmp()Pierre Ossman2023-01-052-10/+13
| | | | | | | | | | If we don't mark these as volatile then they might get reset on longjmp() and the code will misbehave.
| * Remove useless const specifiersPierre Ossman2023-01-052-4/+4
| | | | | | | | | | These are just values, so setting const on them has no effect as the caller will get a copy.
| * Remove useless checksPierre Ossman2023-01-052-4/+1
| | | | | | | | These are all truisms because of the valid range of the types.
| * Explicitly mark switch case fall throughPierre Ossman2023-01-056-0/+11
| | | | | | | | | | This allows us to separate accidental fall through from explicit ones, which allows us to turn on such checks in the compiler.
| * Fix RSA-AES state machinesPierre Ossman2023-01-052-48/+44
| | | | | | | | | | | | | | | | | | | | If there isn't enough data, then the client reading the hash will fall down and try to read the subtype instead, which isn't correct. Invert the logic so we get a more consistent way through where we only break out when there is insufficient data. Do the same for the server code, for consistency.
| * Add missing breaks to Windows key grab codePierre Ossman2023-01-041-0/+2
| | | | | | | | | | These were not meant to fall through. Likely didn't have any adverse effects, but let's fix it anyway.
| * Explicitly mark unused parametersPierre Ossman2023-01-0481-163/+207
| | | | | | | | | | This allows us to separate accidentally unused, from explicitly unused parameters, which allows us to turn on such checks in the compiler.
| * Respect given server namePierre Ossman2023-01-041-1/+1
| | | | | | | | | | No practical difference currently, but let's respect what this API was supposed to do.
| * Don't include unused file path parameterPierre Ossman2023-01-041-3/+3
| |
| * Don't include palette to full color methodsPierre Ossman2023-01-044-9/+7
| | | | | | | | They have no need for it, so let's simplify things.
| * Remove unused flags to decoder contextsPierre Ossman2023-01-046-6/+13
| | | | | | | | Everything in flags is already handled at a higher level.
| * Stop supplying flags to clipboard peek handlerPierre Ossman2023-01-0410-10/+10
| | | | | | | | The flags should always be empty anyway.
| * Mark system include paths correctlyPierre Ossman2023-01-048-16/+16
| | | | | | | | | | This makes sure the compiler doesn't complain about problems in those files.
| * Always enable -WvlaPierre Ossman2023-01-041-4/+4
| | | | | | | | | | It's only the -Werror part we want to be conditional on being debug builds.
* | Update Russian translationYuri Kozlov2023-01-051-235/+276
|/
* Remove debugging strdup()Pierre Ossman2023-01-021-1/+1
| | | | | This was not meant to be included in the previous commit. The static storage for the font string is more than enough.
* Keep font name in permanent memoryPierre Ossman2023-01-021-5/+10
| | | | | Fl::set_font() doesn't keep its own copy, so we need to make sure the font string doesn't get freed or overwritten at a later point.
* Use Rocky Linux for RHEL buildsPierre Ossman2022-12-305-4/+4
| | | | | CentOS Stream isn't fully compatible with RHEL, and there are no useful RHEL images available, so that leaves us with one of the new RHEL forks.
generated by cgit v1.2.3 (git 2.39.1) at 2024-12-23 06:23:42 +0000