Fix linux build and startup #400
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- added vst2sdk to header search path - use standard AU type 'aumi'
Fix macOS build
This patch circumvents program aborts when - The window is too small so that the text width may get negative - uiImageSlider is fully initialised, so that the slider range contains invalid data
…name. When a Lua chank has a name, after the compilation an extended type of the variable with the chunk's name is stored in the CtrlrLuaMethod object. Errors are defined as additional types. This complies to the fact that a Lua chunk can contain everything, not only a single functions. The function isValid() changed its semantics from “running fine, so far” to “no probable error has been detected, so far“. In this sense a named Lua chunk that does not define a variable with the chunk's name is considered to probably contain an error. The function isValid() is mainly used to inform the Ctrlr user/programmer that a certain chunk is problematic in some way, which meens colouring its name red if isValid() returns false. In order to invalid calls to the Lua variable a function isCallable() has been introduced. It returns true iff the Lua chunk defines a function with the chunk's name that compiles fine and did not raise any Lua error so far. So, isCallable() covers most of the cases where isValid() had been called until this patch. In order to avoid unnecessary confusion “json4lua“ has been renamed to “json” and “debugger.lua” has been renamed to “debugger”.
This helps to find out to which panel an error message belongs to. Especially on startup, when many panels are loaded at once.
A thread should be running either as detached thread or as joinable thread. JUCE blurres this nomenclature with their own words. In general creating and deleting a system thread is expensive. So it should be deleted when it is save (i.e. not in realtime functions). This patch solves several issues with the thread: - the thread should exit if it gets signalled by setting a special variable. This variable is handled by JUCE. - The main program should signal the thread to end its action. This is done by stopThread() by setting the status variable and waking up the thread. - When the thread ends before the main program allows it to end, it must wait indefinitely. Otherwise an undefined state with a race condition is created, which may cause additional trouble.
This allows the compiler to warn about unimplemented values.
* using menu of the file editor * from the method tree context menu
The previous patches were developed against Lua 5.2
- Update luausb (now located in libs) - Don't build included libusb (fixes RomanKubiak#321) - Link against system provided libusb on LinuxMakefile TODO: Windows, MacOS and Code::Blocks
- use system lua library - use separate luausb library and system provided libusb - drop linking against udev - Fix ALSA initialisation in JUCE - Set global include path for project files - new libraries in libs instead of Source/Misc - ready to make new libraries into git subprojects
They somehow don't work with run-cmake
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Finally, I managed to compile a version of Ctrlr that
TODO: