summaryrefslogtreecommitdiff
path: root/avmedia
AgeCommit message (Collapse)AuthorFilesLines
2016-03-04-Werror,-Wunused-macrosStephan Bergmann1-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
2016-03-04Silence -Werror,-Wnon-virtual-dtorStephan Bergmann1-0/+8
Change-Id: Ie346bca98a2da91e20ddb533ab19720f193291d4
2016-03-03loplugin:unuseddefaultparams variousNoel Grandin1-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>
2016-02-22loplugin:write only fieldsNoel Grandin3-7/+4
Change-Id: I44f249a17d0a510ec63a488b656d57a1a392f821
2016-02-17use consistent #define checks for the Windows platformNoel Grandin3-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>
2016-02-16loplugin:cstylecastStephan Bergmann1-1/+1
Change-Id: I17025698070d3140eb2d09b705020ee4e0ee97c1
2016-02-16gtk3+wayland: play video via gtksink gstreamer elementCaolán McNamara3-56/+122
Change-Id: Ib371fa06eda73962cbe94739e69a68b46c26e4bf
2016-02-10vcl: move CommandEvent functions from header to cxx fileChris Sherlock1-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
2016-02-09Remove excess newlinesChris Sherlock1-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>
2016-02-09Formatting changes across all modulesChris Sherlock36-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>
2016-02-06vcl: bmpacc.hxx -> bitmapaccess.hxxChris Sherlock3-3/+3
Change-Id: I4bb19d6103c4a6a902d86b62a857e3478493924c
2016-01-26Silence more warnings, clang-cl's /W4 now also includes -WextraStephan Bergmann1-1/+1
Change-Id: I20fa3b02570bdbc3ccd297401a7669b0fda5c624
2016-01-25more Windows build fixesNoel Grandin1-4/+4
after my OInterfaceContainer2 change Change-Id: I345c47fac366759f61e6336b39feb91210dd25bd
2016-01-25InterfaceContainer2 with vector instead of SequenceNoel Grandin1-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
2016-01-18loplugin:unusedfieldsNoel Grandin3-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>
2016-01-11loplugin:privatebase: Make derivation from noncopyable explicitly privateStephan Bergmann3-4/+4
Change-Id: Ia410950353aec0106ea82d025ed3ee89c79ca127
2016-01-10Fix typosAndrea Gelmini1-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>
2016-01-05Fix typosAndrea Gelmini1-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>
2015-12-24loplugin:unusedfieldsNoel Grandin3-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>
2015-12-23loplugin:unusedfields in avmediaNoel Grandin3-6/+3
Change-Id: I725a80cf7b6f2b728e650d0f365d063931b17ea7
2015-12-20loplugin::stringconcatStephan Bergmann1-3/+3
Change-Id: I212293d369be8279d886379359fc90e930ad9719
2015-12-20Fix mis-edit in d0652bfd2b87fad10c141e3bf3f5d0c0a8683cb5Tor Lillqvist1-1/+1
Change-Id: I637544c19c407389041343c951e7cbb7129bc1a2
2015-12-20Change a local DBG macro to SAL_INFOTor Lillqvist1-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
2015-12-20Cosmetic cleanupTor Lillqvist1-13/+0
Change-Id: I68511f33c13fa8277219a170e10821f9d0ee9ecf
2015-12-20Bin silly DBG macro used in two not that interesting casesTor Lillqvist1-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
2015-12-12Make .mk files more portableBaptiste Daroussin1-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>
2015-12-10loplugin:nullptr: More NULL -> nullptr automatic rewriteStephan Bergmann3-13/+13
Change-Id: Ie83819e2bcdc5fa160b39296b005ca9a5ff74b1d
2015-11-28Use Get/SetWindowLongPtr, for 64-bit buildStephan Bergmann1-2/+2
Change-Id: I3041d93f98e491e4600da78a0d1f6100be1177a7
2015-11-28-Werror,-WreorderStephan Bergmann1-3/+3
Change-Id: I202fee919179c89101a7f2b8a802e14254d768e2
2015-11-28Use Get/SetWindowLongPtr, for 64-bit buildStephan Bergmann1-2/+2
Change-Id: I6d763369aa0c142ec0b0b9fb08aeb212eaae7f2f
2015-11-28-Werror,-Wint-to-pointer-castStephan Bergmann4-19/+27
Change-Id: Iec98a5a3df5dcfcd3fcf658bf27e9b5fe57ffc44
2015-11-17other #includes should be AFTER the include-once #defineNoel Grandin1-2/+2
found with the following script: git ls-files *.hxx | xargs perl -0777 -ne 'if (/include.* .* .* "; close ARGV }' Change-Id: I3afb0e81eb315c34bc0eb6bb4733dd1f0537d5c0
2015-11-17Don't assume sal_Unicode is unsigned shortStephan Bergmann4-4/+4
Change-Id: I7fcaa4e86f264182db7f4e46c2388402f801531a
2015-11-16use initialiser list for Sequence<OUString>Noel Grandin4-9/+4
Change-Id: Ia5e47261d1fc6fac2d046656c05a1c5eedb07e02 Reviewed-on: https://gerrit.libreoffice.org/19978 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser for Sequence<OUString>Noel Grandin11-22/+11
using variations of: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<OUString> (\w+)\(1\)\; \s*OUString\* pArray.*; .*\[0\]\s*=\s*(\S+)\;/Sequence<OUString> \1 { \2 };/g" Change-Id: I03c64334ff30ee14dce0d17b67f5122a3893bbe3 Reviewed-on: https://gerrit.libreoffice.org/19971 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser for Sequence<OUString>Noel Grandin1-2/+1
replaced using: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<\s*OUString\s*> (\w+)\(\s*1\s*\); .*\[0\] = (\S+);/Sequence<OUString> \1 { \2 };/g" Change-Id: I20ad0489da887a9712982531c3b127339bb8b3b9 Reviewed-on: https://gerrit.libreoffice.org/19969 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-15use initialiser syntax for Sequence<OUString>Noel Grandin9-18/+9
replaced using the script: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence< OUString > (\w+)\(1\); .*\[0\] = (\S+);/Sequence< OUString > \1 { \2 };/g" Change-Id: I23688a91562051a8eed11fc2a85599545c285c34 Reviewed-on: https://gerrit.libreoffice.org/19967 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-12Bin mesa_headers completelyTor Lillqvist2-2/+0
We don't need them in a MSVC or OS X compilation either. Change-Id: I00181fe0a047df09bbdfcce34c07eb2ebc45a2da
2015-11-12Avoid mesa_headers on X11, also link with -lGL only where necesssaryTor Lillqvist2-2/+2
The GLEW headers are enough, and what we actually use in these places. In addition to handling GL extension things in its dynamic fashion, GLEW headers also have declarations for standard, non-extension, OpenGL API, including xgl and wgl ones. Most likely we don't need mesa_headers on Windows or OS X either, and can drop them completely. Change-Id: Ic0d8d6238c862f8fe4a74e99e95344dcbf540980
2015-11-115th step to remove tools/rtti.hxxOliver Specht1-1/+1
tools/rtti.hxx removed completed the interface of some Sdr.* Items and removed pseudo items Change-Id: I0cdcd01494be35b97a27d5985aa908affa96048a Reviewed-on: https://gerrit.libreoffice.org/19837 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
2015-11-10loplugin:nullptr (automatic rewrite; Mac-specific code)Stephan Bergmann6-13/+13
Change-Id: Ib50c9415e1ddd0e6cabc1b737a39a7d101014da9
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann22-68/+68
Change-Id: Ib3f24439ffda42d217300dcc0ca979964ade360a
2015-11-09new loplugin: oncevarNoel Grandin2-5/+2
Change-Id: If57390510dde4d166be3141b9f658a7453755d3f Reviewed-on: https://gerrit.libreoffice.org/19815 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-06loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann1-1/+1
Change-Id: I154cad87f3015783726ea8f533f79bdbc3efc0a0
2015-11-04loplugin:stringconstantStephan Bergmann1-1/+1
Change-Id: I182d3b11e7b216f5bdd4d0a850f6a21a100425ef
2015-11-04yyyyyNoel Grandin1-1/+1
Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
2015-10-30use uno::Reference::set method instead of assignmentNoel Grandin15-18/+18
Change-Id: I58410209f32f988f258a588364e0b037c2790211
2015-10-25Replace boost::bind by lambdas in avmedia tree.Mario J. Rugiero2-4/+2
Change-Id: I0072ba9972a6c4d1565b1d3826202ff11f0c335d Reviewed-on: https://gerrit.libreoffice.org/19582 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-21Improve awkward dialog titleAdolfo Jayme Barrientos1-1/+1
Please never use the word “dialog” in a dialog’s title. It’s obvious. This string has led to confused translations. Change-Id: Id55d45a48630e5f3d3f01d4eee87240f9907a803
2015-10-14avmedia/quicktime: replace OSL_TRACE with SAL_INFODouglas Mencken3-18/+27
Change-Id: Ic001787cbb0b08242f74331c54a597103b8d2648 Reviewed-on: https://gerrit.libreoffice.org/19351 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>