summaryrefslogtreecommitdiff
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* naming: Add unreachable return to !HAVE_PCRE pathAlistair Delva2020-08-131-0/+1
| | | | | | | | | | | | | Android builds all host tools with -Werror=no-return, which generates a false positive in name_regexmatch_value() if HAVE_PCRE is not present. Fix this by adding a return code to the !HAVE_PCRE path. This return will not be reached but will suppress the compiler warning. If/when SWIG can require C++11 compilers, a better fix would be to make SWIG_exit() [[noreturn]]. Closes #1860
* Merge branch 'directors-comparison-operators'William S Fulton2020-06-141-4/+0
|\ | | | | | | | | | | * directors-comparison-operators: Use %rename Fix wrapping of virtual comparison operators with directors
| * Fix wrapping of virtual comparison operators with directorsZackery Spytz2020-06-051-4/+0
| | | | | | | | Closes #1642.
* | Merge branch 'python-doxygen-quotes'William S Fulton2020-06-071-3/+27
|\ \ | | | | | | | | | | | | | | | * python-doxygen-quotes: Fix generated Python code for Doxygen comments with triple quotes Fix generated Python code for Doxygen comments ending with quote
| * | Fix generated Python code for Doxygen comments with triple quotesVadim Zeitlin2020-03-041-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | In addition to the changes in the previous commit, also avoid syntax errors in the generated Python docstrings by splitting them into several parts if there are 3 quotes in a row in the input, as it's impossible to have them inside triple-quoted strings, generally speaking (i.e. if there are occurrences of both """ and ''' inside the string).
| * | Fix generated Python code for Doxygen comments ending with quoteVadim Zeitlin2020-03-041-3/+17
| | | | | | | | | | | | | | | | | | Single-line Doxygen comments ending with a double quote resulted in syntactically-invalid Python docstrings in the output, so use triple single quotes as delimiters in this case to avoid it.
* | | escape phony targets as wellNoah Stegmaier2020-06-041-1/+1
| | |
* | | escape target nameNoah Stegmaier2020-06-041-1/+1
| | |
* | | escape path of generated fileNoah Stegmaier2020-06-041-1/+1
| | |
* | | escape filepathsNoah Stegmaier2020-06-043-1/+15
| |/ |/|
* | Merge branch 'r-script-fixes'William S Fulton2020-06-021-2/+2
|\ \ | | | | | | | | | | | | | | | * r-script-fixes: Fix further missing semicolons in R code. fixed some missing semicolons. Each one caused a compile error in generated code
| * | Fix further missing semicolons in R code.William S Fulton2020-06-021-1/+1
| | | | | | | | | | | | Fixes errors in R code when using -small as this option causes lines to be joined.
| * | fixed some missing semicolons. Each one caused a compile error in generated codekhoran2020-03-251-1/+1
| |/
* | octave.cxx: fix exception raising for newer Octave versionsKarl Wette2020-05-311-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | - Since (at least) Octave 5.1.0, the Octave error() function now raises a C++ exception, which if uncaught immediately exits a SWIG wrapper function, bypassing any cleanup code that may appear after a "fail:" label. - This patch adds a "try { ... } catch(...) { }" block around the contents of SWIG wrapper functions to first execute the cleanup code before rethrowing any exception raised. - It is backward compatible with earlier versions of Octave where error() does not raise an exception, which will still branch to the "fail:" block to execute cleanup code if an error is encountered.
* | octave.cxx: add missing return statement before "fail:" labelKarl Wette2020-05-311-0/+1
| |
* | octave.cxx: this belongs in the code (as opposed to definition) sectionKarl Wette2020-05-311-1/+1
| |
* | octave.cxx: remote whitespacesKarl Wette2020-05-311-2/+2
| |
* | octave.cxx: replace Printf() with Append() for consistencyKarl Wette2020-05-311-1/+1
| |
* | Merge pull request #1726 from vadz/unsigned-charWilliam S Fulton2020-05-301-2/+3
|\ \ | | | | | | Fix reading options files on platforms with unsigned char
| * | Fix reading options files on platforms with unsigned charVadim Zeitlin2020-02-161-2/+3
| |/ | | | | | | | | | | | | | | This fixes EOF detection on platforms where char is unsigned, as comparing it with EOF could never return true there. Thanks gcc for the warning "comparison is always true due to limited range of data type [-Wtype-limits]".
* | Merge branch 'DOH_POOL_SIZE-increase'William S Fulton2020-05-281-1/+1
|\ \ | | | | | | | | | | | | * DOH_POOL_SIZE-increase: Increase DOH_POOL_SIZE
| * | Increase DOH_POOL_SIZEZackery Spytz2020-04-181-1/+1
| |/ | | | | | | | | | | Increase DOH_POOL_SIZE to 2^22. Addresses GH-1775.
* | Fix for missing space after \endlink in doxygenJohn McFarland2020-05-021-0/+4
|/ | | | | | | | | | | | | | | | The "endlink" command is processed in processWordCommands, which by default skips space occuring after the command, which is intended for removing leading space from a command argument. For "end" commands, we don't want to do this. Note that certain end commands such as "endcode" aren't processed by processWordCommands (believe addCommandUnique ends up handling them). Update usage of \link in doxygen_translate_all_tags.i to test handling of space after \endlink. Tweaking some of the usage in doxygen_misc_constructs.i to remove what seems to be an extra space from the input (otherwise we would need to add an extra space to the expected output).
* Recognize C++ conversion operators with trailing '= 0' as abstractThomas REITMAYR2020-02-131-1/+16
| | | | | This fix is done for all supported variants of user-defined conversion operators and fixes swig#1723.
* Merge branch 'autodoc-doxygen-fix'William S Fulton2020-01-311-4/+11
|\ | | | | | | | | * autodoc-doxygen-fix: Fix crash in Python backend when using empty docstrings
| * Fix crash in Python backend when using empty docstringsVadim Zeitlin2020-01-301-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | Due to confusion in build_combined_docstring(), we could call DohDelete() on the "feature:docstring" string, which resulted in a crash when trying to use it later. Fix this and simplify the code at the same time by ensuring that we always use a copy of "feature:docstring" if it's not empty or don't use it at all if it's empty -- like this we don't have to check for its length each time before using it. Closes #1648.
* | Fix R memory leak on exceptionArnaud Barré2020-01-301-4/+19
| | | | | | | | | | | | | | | | | | | | There is a possible memory leak in case the SWIG_exception_fail macro is called. The problem is related to its definition that call the function Rf_warning. This function (as well as Rf_error) involves a longjmp over C++ destructors on the stack. Thus, all the objects allocated on the heap are not freed. Closes #914
* | Remove redundant code from r.cxxWilliam S Fulton2020-01-301-115/+2
| |
* | Fix display of template classnames in generated R codeWilliam S Fulton2020-01-301-4/+5
| |
* | Merge branch 'RMemberListTrialSimplify2019'William S Fulton2020-01-301-100/+149
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | * RMemberListTrialSimplify2019: ENH R abstract_access_runme ENH R accessor processing test Removed some remaining commented sections moved registration routine and use swig_name_get calling Swig_name_setget Used Swig_name_register so that Swig_name_wrapper produces the correct name without a separate replace call. Removed last instance of using Strcmp to check for a set/get method. Replaced with check for flag. Alternative version of using memberlist processing. This clarifies the logic within OutputMemberReferenceMethod by filtering the lists into classes, rather than doing it internally. Code isn't any shorter. commenting out unused code first pass at removing string comparisons for set/get methods trial changing member list processing
| * Removed some remaining commented sectionsRichard Beare2019-08-301-74/+2
| |
| * moved registration routine and use swig_name_getRichard Beare2019-04-101-7/+2
| |
| * calling Swig_name_setgetRichard Beare2019-04-101-2/+5
| |
| * Used Swig_name_register so that Swig_name_wrapper producesRichard Beare2019-04-101-1/+4
| | | | | | | | the correct name without a separate replace call.
| * Removed last instance of using Strcmp to check for a set/getRichard Beare2019-04-101-5/+1
| | | | | | | | method. Replaced with check for flag.
| * Alternative version of using memberlist processing. This clarifies theRichard Beare2019-04-101-21/+59
| | | | | | | | | | logic within OutputMemberReferenceMethod by filtering the lists into classes, rather than doing it internally. Code isn't any shorter.
| * commenting out unused codeRichard Beare2019-04-101-0/+10
| |
| * first pass at removing string comparisons for set/get methodsRichard Beare2019-04-101-36/+57
| |
| * trial changing member list processingRichard Beare2019-04-101-16/+71
| |
* | Add some comments about shared_ptr upcast codeWilliam S Fulton2020-01-244-0/+11
| |
* | Refactor upcasts codeWilliam S Fulton2020-01-233-79/+86
| | | | | | | | | | | | In preparation for possible improvement in usage of SwigType_typedef_resolve_all - a SwigType* should be used, not a readable name in a String*.
* | shared_ptr upcast fixes for DWilliam S Fulton2020-01-231-5/+6
| | | | | | | | Same changes that were applied to Java/C# in ab7f526805b86726a3c23c853e0ab19458f2c7d9
* | Merge branch 'shared-ptr-template-upcast'William S Fulton2020-01-172-10/+14
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * shared-ptr-template-upcast: comments Applying shared_ptr template upcast fix to CSharp, adding CSharp test, and cleanup Adding test case demonstrating issue where SWIG does not generate a correctly typed, upcasted shared_ptr for a template instantiation deriving from a base class WIP - Use the non-encoded type string for upcasting a shared_ptr of a derived type to a shared_ptr of the base type comments Applying shared_ptr template upcast fix to CSharp, adding CSharp test, and cleanup Adding test case demonstrating issue where SWIG does not generate a correctly typed, upcasted shared_ptr for a template instantiation deriving from a base class WIP - Use the non-encoded type string for upcasting a shared_ptr of a derived type to a shared_ptr of the base type
| * | Applying shared_ptr template upcast fix to CSharp, adding CSharp test, and ↵etse2019-05-062-17/+6
| | | | | | | | | | | | cleanup
| * | WIP - Use the non-encoded type string for upcasting a shared_ptr of a ↵Eric Tse2019-05-061-5/+20
| | | | | | | | | | | | derived type to a shared_ptr of the base type
* | | Fix segfault parsing varargs with -doxygenWilliam S Fulton2020-01-161-6/+6
| | | | | | | | | | | | Closes #1643
* | | Merge branch 'Issue-1643'William S Fulton2020-01-161-5/+4
|\ \ \ | | | | | | | | | | | | | | | | * Issue-1643: Fix pydoc null pointer dereference with missing arg type
| * | | Fix pydoc null pointer dereference with missing arg typeJohn McFarland2019-10-271-5/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Processing doxygen @param comments for a parameter whose name did not appear in the function declaration would cause a segfault due to a null pointer dereference. Adding test cases for both variadic function (no specified arguments) and @param comment that references an argument that is not named in the function prototype. Both of these cases previously segfaulted.
* | | | Memory leak fix handling empty /// doxygen commentsWilliam S Fulton2020-01-161-3/+5
| | | |
* | | | Merge branch 'Issue-1632'William S Fulton2020-01-161-0/+20
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Issue-1632: Minor workaround in doxygen_basic_translate_style3 test Add new test doxygen_basic_translate_style3.i Fix for newline handling in doxygen "///" style comments
View Lorry Controller Status