summaryrefslogtreecommitdiffstats
path: root/avmedia
Commit message (Collapse)AuthorAgeFilesLines
* tdf#97087 Give comprehensible, unique names to idlesMuhammet Kara2016-04-281-0/+1
| | | | | | | | | | Timers and idles should have programmer comprehensible, unique names Change-Id: Id0f2c0a77cd28c3ec5473e8432569739b58d2101 Reviewed-on: https://gerrit.libreoffice.org/24388 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* clang-tidy modernize-loop-convert in a*Noel Grandin2016-04-211-2/+1
| | | | Change-Id: If8eada464c6dc7350e50fc90eb27aa47bd90e198
* clang-tidy performance-unnecessary-value-param in variousNoel Grandin2016-04-143-3/+3
| | | | | | | Change-Id: I7168d44dab8e6a8e37bb7920d744ff32f5e52907 Reviewed-on: https://gerrit.libreoffice.org/24019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Avoid reserved identifierStephan Bergmann2016-04-121-3/+3
| | | | Change-Id: Ie881d77774ea05b5683850c113cf3f6b1d6f2ca7
* headless build: reset ENABLE_GLTF and use that setting as beforeOliver Specht2016-04-052-4/+0
| | | | | | | Change-Id: Iea54c59393c7c2955ec5fa321d9e9ea7b57b198a Reviewed-on: https://gerrit.libreoffice.org/23831 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
* tdf#94306 replace boost::noncopyable in ..Jochen Nitschke2016-04-044-8/+17
| | | | | | | | | | .. accessibility, avmedia, basctl, basic and bridges remove boost from module if not needed anymore Change-Id: I6177f8276766a0a7df1703e81bf1b448912df6e2 Reviewed-on: https://gerrit.libreoffice.org/23744 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* tdf#96505 Remove 0L occurences from avmedia and basegfx modulesslideon2016-03-311-4/+4
| | | | | | | | | No compiler warnings on my machine Change-Id: Ibd303c64f04f6f52272ac19c3069da2f14cb4e9a Reviewed-on: https://gerrit.libreoffice.org/23558 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* sfx2: make SfxDispatcher::Execute() less variadic ...Michael Stahl2016-03-311-1/+2
| | | | | | | | | | | ... and less overloaded; C++11 initializer_list should make undefined behavior as fixed in ba0a866b83f016fd460320e7d057dd107e019f19 less likely. Change-Id: I15cc0804451b6a4fcbfaa623f9a30db222207865 Reviewed-on: https://gerrit.libreoffice.org/23666 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Spurious closing parenthesisStephan Bergmann2016-03-311-1/+1
| | | | Change-Id: I0b723b758a4728816587a26d9b9b28cdc06f5dd1
* use SAL_N_ELEMENTS more widelyNoel Grandin2016-03-312-2/+2
| | | | | | | found using git grep -n 'sizeof.*/.*sizeof.*[0]' Change-Id: Icd4a6cc1ca8ec8ebd68e1701a02789c74cf0eb2a
* avmedia: fix varargs call for WNT 64-bitMichael Stahl2016-03-301-1/+1
| | | | | | | | | | SfxDispatcher::Execute(sal_uInt16 nSlot, SfxCallMode eCall, const SfxPoolItem* pArg1, ...) assigns the varargs to a "const SfxPoolItem *", so passing in "0L" is only correct if long is 64-bit, but on WNT 64-bit long is 32-bit, so use "nullptr" already. Change-Id: Iaf41efa795fd4439daab7ed7ec334d8ad8c476f3
* avmedia: the Module makefile already checks and disables thatMichael Stahl2016-03-291-2/+1
| | | | Change-Id: I4620a0358e06e56684954f208d19bd4350e397a5
* avmediaogl now seems to require ENABLE_GLTF for some reasonTor Lillqvist2016-03-281-1/+2
| | | | Change-Id: Id5b65b944ed06598238f5ab139c52d4de5ca1f84
* fix headless buildOliver Specht2016-03-254-4/+16
| | | | | | | | | | disables OpenGL and glew usage, lets --without-gui do what --without-x did before and disables X related test Change-Id: I680b47c9962a0d43c8ece593db0b82e347ceebdb Reviewed-on: https://gerrit.libreoffice.org/23474 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
* Resolves: tdf#96989 videos playback at maximum possible volumeCaolán McNamara2016-03-172-6/+23
| | | | | | | | | | On systems with flat-volumes then setting the volume directly on the playbin to 100% results in resetting the global volume to the maximum possible volume. We expect to set as % of the current system volume. Putting an intermediate volume object into the pipeline does the more expected thing. Change-Id: I911d6fffba0983e4fd7b455e820959a96115de34
* remove unnecessary parentheses in case labelsNoel Grandin2016-03-153-26/+26
| | | | Change-Id: I987bf68c767709080c67c8b183c036a4109c6caa
* fix for 10 sec hang with audio-only formats with gstreamer1Caolán McNamara2016-03-101-1/+2
| | | | | | | just do the same as we did for gstreamer 0.10 and set the size found condition once we get the message even if there is no size Change-Id: I826d76dd1097432e5bd33d4acc1a187b8e5068da
* -Werror,-Wunused-macrosStephan Bergmann2016-03-041-1/+0
| | | | | | | | | | | | | | | | | <https://msdn.microsoft.com/en-us/library/windows/desktop/ms647466%28v=vs.85%29.aspx> "About Strsafe.h": When you include Strsafe.h in your file, the older functions replaced by the Strsafe.h functions will be deprecated. Attempts to use these older functions will result in a compiler error telling you to use the newer functions. If you want to override this behavior, include the following statement before including Strsafe.h. #define STRSAFE_NO_DEPRECATE Appears to no (longer?) be relevant here, as reported by clang-cl. Change-Id: I94a576b71a2a34edd6009059fc5047ba59ec0f12
* Silence -Werror,-Wnon-virtual-dtorStephan Bergmann2016-03-041-0/+8
| | | | Change-Id: Ie346bca98a2da91e20ddb533ab19720f193291d4
* loplugin:unuseddefaultparams variousNoel Grandin2016-03-031-4/+2
| | | | | | | Change-Id: Ibf8489c957b307156689de4c7cb8440ddd4e4546 Reviewed-on: https://gerrit.libreoffice.org/22852 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:write only fieldsNoel Grandin2016-02-223-7/+4
| | | | Change-Id: I44f249a17d0a510ec63a488b656d57a1a392f821
* use consistent #define checks for the Windows platformNoel Grandin2016-02-173-3/+3
| | | | | | | | | | | | stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:cstylecastStephan Bergmann2016-02-161-1/+1
| | | | Change-Id: I17025698070d3140eb2d09b705020ee4e0ee97c1
* gtk3+wayland: play video via gtksink gstreamer elementCaolán McNamara2016-02-163-56/+122
| | | | Change-Id: Ib371fa06eda73962cbe94739e69a68b46c26e4bf
* vcl: move CommandEvent functions from header to cxx fileChris Sherlock2016-02-101-1/+1
| | | | | | | | | | | | | | | | Moving functions from commandevent.hxx to commandevent.cxx, this means they are no longer inline, but there was no clear reason as to why they were inline in the first place! I've also reordered the header to ensure that CommandEvent is the first fully declared class, the data classes get declared after this (hence the forward declarations). This makes reading this header much easier. One more small thing: a few if statements had braces in the primary if block, but not in the else block, which is IMO bad style and can possibly lead to errors. I've made the else block use braces. Change-Id: I79b010bd66d3c9f4a74d4fa470f62d0263c811f4
* Remove excess newlinesChris Sherlock2016-02-091-1/+0
| | | | | | | | | | | | | | | | | | | A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* Formatting changes across all modulesChris Sherlock2016-02-0936-354/+130
| | | | | | | | | | | | | | + Removed comment cruft + Tab formatting in number of files + Some commented out code removed + Tab characters replaced with spaces + Newline cleanup in quite a few files + Tweak header guard #endifs Change-Id: I3208ff2f047da890edcc49b73389aca22442f5fc Reviewed-on: https://gerrit.libreoffice.org/22221 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* vcl: bmpacc.hxx -> bitmapaccess.hxxChris Sherlock2016-02-063-3/+3
| | | | Change-Id: I4bb19d6103c4a6a902d86b62a857e3478493924c
* Silence more warnings, clang-cl's /W4 now also includes -WextraStephan Bergmann2016-01-261-1/+1
| | | | Change-Id: I20fa3b02570bdbc3ccd297401a7669b0fda5c624
* more Windows build fixesNoel Grandin2016-01-251-4/+4
| | | | | | after my OInterfaceContainer2 change Change-Id: I345c47fac366759f61e6336b39feb91210dd25bd
* InterfaceContainer2 with vector instead of SequenceNoel Grandin2016-01-251-4/+4
| | | | | | | | | | create an InterfaceContainer2 class to replace InterfaceContainer. It uses a std::vector instead of a Sequence for the mutable listener list, which provides far better performance. Switch all our internal use-sites to the new class. Change-Id: I6b56cfa511ded2395faa22e68fab3b2f16c3cb88
* loplugin:unusedfieldsNoel Grandin2016-01-183-8/+4
| | | | | | | Change-Id: I321b594f1190cdd997b4769161e1c02d31f46213 Reviewed-on: https://gerrit.libreoffice.org/21566 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:privatebase: Make derivation from noncopyable explicitly privateStephan Bergmann2016-01-113-4/+4
| | | | Change-Id: Ia410950353aec0106ea82d025ed3ee89c79ca127
* Fix typosAndrea Gelmini2016-01-101-2/+2
| | | | | | | Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
* Fix typosAndrea Gelmini2016-01-051-1/+1
| | | | | | | Change-Id: I3fba2c76c83381eb398c80947ef4849bccf7ab27 Reviewed-on: https://gerrit.libreoffice.org/21078 Reviewed-by: Ashod Nakashian <ashnakash@gmail.com> Tested-by: Ashod Nakashian <ashnakash@gmail.com>
* loplugin:unusedfieldsNoel Grandin2015-12-243-6/+3
| | | | | | | Change-Id: I979592adb978c3757a1e54615021ee424a2e02bf Reviewed-on: https://gerrit.libreoffice.org/20892 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unusedfields in avmediaNoel Grandin2015-12-233-6/+3
| | | | Change-Id: I725a80cf7b6f2b728e650d0f365d063931b17ea7
* loplugin::stringconcatStephan Bergmann2015-12-201-3/+3
| | | | Change-Id: I212293d369be8279d886379359fc90e930ad9719
* Fix mis-edit in d0652bfd2b87fad10c141e3bf3f5d0c0a8683cb5Tor Lillqvist2015-12-201-1/+1
| | | | Change-Id: I637544c19c407389041343c951e7cbb7129bc1a2
* Change a local DBG macro to SAL_INFOTor Lillqvist2015-12-201-40/+26
| | | | | | | | | | But actually, I am starting to lean more and more towards the opinion that very little of our debug output code is actually useful to keep in the sources permanently. When somebody wants to debug some part of the code, they will very likely want to add much more detailled tracing anyway, temporarily. Change-Id: I6a60a130e3b72ba3d6ad32a1b22a2a9f3ac66b27
* Cosmetic cleanupTor Lillqvist2015-12-201-13/+0
| | | | Change-Id: I68511f33c13fa8277219a170e10821f9d0ee9ecf
* Bin silly DBG macro used in two not that interesting casesTor Lillqvist2015-12-201-11/+0
| | | | | | | I did not bother changing this to SAL_INFO, no need to keep debug output like that in the code permanently. Change-Id: I59fe193e291cdd662e7cd7ecdb26d354aee60c80
* Make .mk files more portableBaptiste Daroussin2015-12-121-1/+1
| | | | | | | | | | Replace some -ldl by $(DLOPEN_LIBS) Add some FREEBSD tests Change-Id: Ic30fe29b938d877694db2927775f89de15c02d87 Reviewed-on: https://gerrit.libreoffice.org/20652 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Matúš Kukan <matus@libreoffice.org>
* loplugin:nullptr: More NULL -> nullptr automatic rewriteStephan Bergmann2015-12-103-13/+13
| | | | Change-Id: Ie83819e2bcdc5fa160b39296b005ca9a5ff74b1d
* Use Get/SetWindowLongPtr, for 64-bit buildStephan Bergmann2015-11-281-2/+2
| | | | Change-Id: I3041d93f98e491e4600da78a0d1f6100be1177a7
* -Werror,-WreorderStephan Bergmann2015-11-281-3/+3
| | | | Change-Id: I202fee919179c89101a7f2b8a802e14254d768e2
* Use Get/SetWindowLongPtr, for 64-bit buildStephan Bergmann2015-11-281-2/+2
| | | | Change-Id: I6d763369aa0c142ec0b0b9fb08aeb212eaae7f2f
* -Werror,-Wint-to-pointer-castStephan Bergmann2015-11-284-19/+27
| | | | Change-Id: Iec98a5a3df5dcfcd3fcf658bf27e9b5fe57ffc44
* other #includes should be AFTER the include-once #defineNoel Grandin2015-11-171-2/+2
| | | | | | | | | | | | found with the following script: git ls-files *.hxx | xargs perl -0777 -ne 'if (/include.* .* .* "; close ARGV }' Change-Id: I3afb0e81eb315c34bc0eb6bb4733dd1f0537d5c0
* Don't assume sal_Unicode is unsigned shortStephan Bergmann2015-11-174-4/+4
| | | | Change-Id: I7fcaa4e86f264182db7f4e46c2388402f801531a