News:

--

Main Menu

Flip filter

Started by Jan Gruuthuse, August 07, 2016, 03:11:20 PM

Previous topic - Next topic

Jan Gruuthuse

60937d7    [filter] Cosmetic rename filter (sl1pkn07)
Still some conflicting information: marked in red.
Horizontal <> Vertical flip.

eumagga0x2a

The following patch solves the identity crisis.

diff --git a/avidemux_plugins/ADM_videoFilters6/horizontalFlip/horizontalFlip.cpp b/avidemux_plugins/ADM_videoFilters6/horizontalFlip/horizontalFlip.cpp
index c7427c3..6fce41e 100644
--- a/avidemux_plugins/ADM_videoFilters6/horizontalFlip/horizontalFlip.cpp
+++ b/avidemux_plugins/ADM_videoFilters6/horizontalFlip/horizontalFlip.cpp
@@ -135,7 +135,7 @@ void horizontalFlipFilter::setCoupledConf(CONFcouple *couples)
const char *horizontalFlipFilter::getConfiguration(void)
{
     
-    return "Vertical flip.";
+    return "Horizontal flip.";
}
// Normally not needed :virtual FilterInfo  *getInfo(void)
//EOF

sl1pkn07

Quote from: eumagga0x2a on August 07, 2016, 05:30:08 PM
The following patch solves the identity crisis.

I laughed so hard

greetings XD

Jan Gruuthuse

Can these filters be joined? Instead of having Horizontal or Vertical Flip filter?

Available Filter: Flip
Where you could mark the wanted operation(s) in its configuration pop-up window
[ ] Horizontal Flip
[ ] Vertical Flip

and any of the 3 combinations would work.
Selecting both would flip video horizontal and vertical?
[v] Horizontal Flip
[v] Vertical Flip