| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously any presence of a function pointer parameter (which had a
void return type) in a function declaration would cause a crash,
because the type of the whole function pointer was misinterpreted
as just "void", and when type() was called on such a parameter a null
pointer was returned instead of an AbstractMetaType object, which when
accessed caused a crash.
Fix consists in checking that the parameters of a function do not have
a void type (unless there's only one parameter).
Some additionals checks are added when trying to access the signature
of a function.
Change-Id: I4baa477732ce1e7c8a494137f37f58bc0df38a07
Task-number: PYSIDE-455
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
|
|
|
|
|
|
|
|
|
|
| |
Don't remove the photon test; doing so is just asking to allow bugs to
creep in undetected. Fixing the test is much better.
This reverts commit 4614591a6cb88626541c03f10ec265f30668915b.
Change-Id: I68f0a8f43acb2ff8d4e00b01a1204e9e00a5c7f4
Reviewed-by: Christian Tismer <tismer@stackless.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The shiboken tests do not build, so we cannot test at all with VS2015.
One issue was a simple C++11 restriction, the other involves the
classes in photon.cpp / photon.h .
I could not find out how to fix that, so I removed the photon files,
in order to get a running test framework.
If somebody knows an easy patch instead of removing photon, please
let me know. Otherwise I would prefer pretty much to move on.
[ChangeLog][shiboken][PYSIDE-324]
Fix compilation of tests with VS2015
Change-Id: I6a8d64b18c5eba110f4c03dba63c4b21d6816918
Reviewed-by: Christian Tismer <tismer@stackless.com>
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
Fix omission of 'const' specifier when generating binding code for
containers whose value type is 'const', which would lead to binding code
that does not compile due to 'const' mismatch. Includes test case.
Change-Id: Iff99a16ee071bb255f78e86e2456e5206cc00cfb
Reviewed-by: John Cummings <jcummings2@users.sf.net>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add methods to various places in the hierarchy so that the generator can
access the topologically ordered class list (in addition to the
unordered class list). Modify CppGenerator::finishGeneration to use
this.
This fixes a failure to correctly create Python wrapper classes due to
subclass wrappers being initialized before their base class (with the
result that the looked-up type object for the base class is null,
causing the derived class to be created without referencing the base).
Also change one of the test cases to test that we correctly wrap a
typedef of a template class derived from a non-template base (which was
failing before this change for the aforementioned reason).
Change-Id: Ib4dc2626a41cb7bb905ff4a302c2613ea12d026b
Reviewed-by: John Cummings <jcummings2@users.sf.net>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add the global header and typesystem XML, and shiboken itself, as
dependencies to running Shiboken for generating the wrapped code used in
the unit tests. This gives a better example in case of users looking at
this code for how to do things in their own projects, and also makes
test writing easier by removing the need to do a clean build or touch
files to force the wrapping code to be recreated when modifying the test
code or the generator. Change command invocation to use shiboken as a
target name (CMake will recognize it as a target and use the correct
path automatically), and remove the now redundant dependency of the
wrapping libraries on the same. (This is superfluous both because using
shiboken as a target will cause CMake to add an implicit dependency on
it for users of the generated files, but also because it isn't really
needed; the generate command having it as a dependency covers needing it
built first, and is what we really care about as far as rebuilding
anything if it changes.)
Change-Id: If5cdf2e98602e500b49ba7b000e4e30e77e1a136
Reviewed-by: John Ehresman <jpe@wingware.com>
|
|
|
|
|
|
|
| |
Fixes bug PYSIDE-54
Change-Id: Ic5c341741170cc77e8ebb59c46c746211582ddeb
Reviewed-by: Marcelo Lira <marcelo.lira@openbossa.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Conflicts:
CMakeLists.txt
data/CMakeLists.txt
doc/CMakeLists.txt
doc/_templates/index.html
doc/_templates/layout.html
doc/_themes/pysidedocs/static/pysidedocs.css
doc/commandlineoptions.rst
doc/conf.py.in
doc/contents.rst
doc/images/bindinggen-development.png
doc/images/boostqtarch.png
tests/CMakeLists.txt
|
|
|
|
| |
Also added a couple of related unit tests.
|
| |
|
|
|
|
| |
This is only a compilation test, no Python test was provided.
|
| |
|
|
|
|
|
|
|
|
| |
This simulates a situation found in QtWebKit's QWebDatabase
and QWebSecurityOrigin classes.
Reviewed by Luciano Wolf <luciano.wolf@openbossa.org>
Reviewed by Renato Araújo <renato.filho@openbossa.org>
|
|
|
|
| |
This simulates the situation found in Qt's phonon module.
|
|
|
|
|
| |
The test is for an Object Type that didn't existed in Python and is
passed to it as C++ reference through a virtual method.
|
|
|
|
| |
It it inspired by some of PySide's QByteArray tests.
|
| |
|
|
|
|
| |
same name"
|
|
|
|
|
|
|
| |
This is related to bug #716: http://bugs.pyside.org/show_bug.cgi?id=716
Reviewed by Hugo Parente <hugo.lima@openbossa.org>
Reviewed by Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
| |
The classes Rect and RectF were added to help with the imitation.
|
|
|
|
|
|
|
|
|
|
|
| |
CPython calls nb_and with swapped arguments
for reverse operators.
This tests checks if the reverse operator
raises NotImplementedError instead of TypeError
when there isn't a valid reverse operator.
Raising TypeError would cause the operator to fail.
|
|
|
|
|
|
|
|
| |
Instead of the old ugly way of compiling the behaviour forever using
"#define".
Reviewed by Hugo Parente <hugo.lima@openbossa.org>
Reviewed by Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
|
|
| |
const-ness"
Reviewer: Marcelo Lira <marcelo.lira@openbossa.org>
Renato Araújo <renato.filho@openbossa.org>
|
|
|
|
|
| |
Reviewed by Hugo Parente <hugo.lima@openbossa.org>
Reviewed by Luciano Wolf <luciano.wolf@openbossa.org>
|
| |
|
|
|
|
|
| |
The version command line argument was previously answered by GeneratorRunner.
This commit also generally updates some CMake files.
|
|
|
|
| |
argument.
|
|
|
|
|
|
|
| |
PYTHON_INCLUDE_PATH.
Reviewer: Marcelo Lira <marcelo.lira@openbossa.org>
Lauro Moura <lauro.neto@openbossa.org>
|
|
|
|
|
|
|
| |
types?) returns True for empty instance; probably should be False"
Reviewer: Marcelo Lira <marcelo.lira@openbossa.org>
Renato Araújo <renato.filho@openbossa.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
conversions.
This test is inspired by bug #511 [1], and it checks the precedence of
an enum argument over a class that can be implicitly built from an
integer.
[1] http://bugs.openbossa.org/show_bug.cgi?id=511
Reviewed by Luciano Wolf <luciano.wolf@openbossa.org>
Reviewed by Renato Araújo <renato.filho@openbossa.org>
|
|
|
|
| |
parameter"
|
|
|
|
|
|
|
|
|
|
| |
Classes in namespaces marked not to be generated are always moved to
upper scopes, the current modification add this same behaviour to enums.
Tests were also added.
Reviewed by Lauro Moura <lauro.neto@openbossa.org>
Reviewed by Renato Araújo <renato.filho@openbossa.org>
|
|
|
|
|
|
|
|
|
|
|
| |
The ObjectModel test class was introduced to check if the transference
of ownership of objects returned from Python to C++ through a virtual
method is working properly.
Also updated the other test that uses the ObjectView class.
Reviewed by Lauro Moura <lauro.neto@openbossa.org>
Reviewed by Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
| |
Reviewer: Renato Araújo <renato.filho@openbossa.org>
Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
| |
Reviewed by Hugo Parente <hugo.lima@openbossa.org>
Reviewed by Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
|
|
| |
returns a reference"
Reviewer: Luciano Wolf <luciano.wolf@openbossa.org>
Marcelo Lira <marcelo.lira@openbossa.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Explaining with an example. Consider a function called "function" with
two signatures accepting a list of Point and a list of PointF,
respectively. Consider also that Point is implicitly convertible to
PointF.
void function(list<Point>&)
void function(list<PointF>&)
A list of Point should be checked before a list of PointF.
Reviewed by Hugo Parente <hugo.lima@openbossa.org>
Reviewed by Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
| |
Reviewer: Luciano Wolf <luciano.wolf@openbossa.org>
Marcelo Lira <marcelo.lira@openbossa.org>
|
|
|
|
|
| |
Reviewer: Luciano Wolf <luciano.wolf@openbossa.org>
Marcelo Lira <marcelo.lira@openbossa.org>
|
|
|
|
|
| |
Reviewer: Renato Araújo <renato.filho@openbossa.org>
Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
| |
Reviewers: Renato Araújo <renato.filho@openbossa.org>
Hugo Parente Lima <hugo.pl@gmail.com>
|
|
|
|
|
|
|
| |
files to allow multiples version installed simultaneous.
Reviewer: Hugo Parente Lima <hugo.pl@gmail.com>
Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
| |
Reviewer: Luciano Wolf <luciano.wolf@openbossa.org>
Hugo Parente Lima <hugo.pl@gmail.com>
|
|
|
|
|
| |
Reviewer: Hugo Parente Lima <hugo.pl@gmail.com>
Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
| |
Reviewer: Marcelo Lira <marcelo.lira@openbossa.org>
Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
| |
Reviewer: Marcelo Lira <marcelo.lira@openbossa.org>
Luciano Wolf <luciano.wolf@openbossa.org>
|
|
|
|
|
|
|
|
| |
calls
Author: Didier Raboud <didier@raboud.com>
Reviewers: Luciano Wolf <luciano.wolf@openbossa.org>,
Marcelo Lira <marcelo.lira@openbossa.org>
|
|
|
|
|
| |
Reviewer: Hugo Parente Lima <hugo.lima@openbossa.org>,
Marcelo Lira <marcelo.lira@openbossa.org>
|