+e_win_evas_win_get()

comp-wl: Remove unused variable

Signed-off-by: Chris Michael <cp.michael@samsung.com>

comp-wl: Add missing function prototype

Since we call this function before it's actually defined in the
source, we'll just add a small function prototype at the top

Signed-off-by: Chris Michael <cp.michael@samsung.com>

wl-desktop-shell: Fix issue where internal elm windows were not
borderless

This fixes the issue of elementary windows having 2 borders, however
it does expose a new issue where those same internal elm windows now
have a black shadow border around them.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

comp_wl: Fix issue with multiple clients being created

This fixes an issue where internal elementary dialogs would end up
creating 2 pixmaps and 2 clients for each window. This happened due to
the pixmap id not matching the elm window id.

This patch also handles reparenting internal elm windows if needed.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

fix crash when activating menu that is not in range

Summary: add the condition check code if the pressed key is not in range of menu items

Test Plan:
(1) Left-click on the desktop to bring up the menu.
(2) Press down the number key to activate the menu item.
(3) When you press down the number key that is not in range, you will see the segmentation error.

below is the crash log which I encoutered.
-------------------------------------------------------------------
Program received signal SIGSEGV, Segmentation fault.
_e_menu_item_activate_nth (n=n@entry=5) at src/bin/e_menu.c:2318
2318       _e_menu_item_ensure_onscreen(mi);

Reviewers: zmike, gwanglim, raster

Reviewed By: raster

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1845

e zone - add stow/unstow apis

don't change shade.x/y during unshade or after shade

fix T1930

check focus after X nocomp end

fix T1768

fix issue where wayland modules were always creating a new compositor

Signed-off-by: Chris Michael <cp.michael@samsung.com>

wl-desktop-shell: Remove use of deprecated e_comp_get function

Signed-off-by: Chris Michael <cp.michael@samsung.com>

remove use of deprecated e_comp_get function

Signed-off-by: Chris Michael <cp.michael@samsung.com>

add E_VERSION_MAJOR define for external module usage

this should simplify writing modules with conditional code for different versions of e

unused param--

remove some e_comp_get() usage in core comp/client files

disable client stack debugging

this should have been turned off for e19 release but I forgot

remove e_comp_list(), deprecate all related functions for pending removal

there is only one E_Comp which can now be accessed by the e_comp global.

if you're editing a file with some uses of these deprecated functions, replace their usages with appropriate references to this variable

pass -Wno-deprecated-declarations to ignore these warnings during build

enforce no multihead in X compositor

this hasn't worked since E17, may was well stop trying

wl_drm: Add event handler for session activation signal.

Summary:
Compositor need to change the compositing mode to offscreen or render state depend on session activation state.
this patch is for it.

Test Plan:
(1) Run enlightenment on X.
(2) Change another VT.
(3) Run enlightenment on wayland and drm.
(4) Change the VT to enlightenment which is ran (1).
enlightenment on wayland will not render during session deactivation state.

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1816

Tiling: don't try to change the border of borderless windows.

@fix

add support for new netwm csd X atoms

nothing uses these yet so there's no way to test them.

fix T1963

completely block border changes on MWM borderless windows

ref T1963

seriously.

don't set gravity on client windows during X move/resize

fix T1940

remove e_box from POTFILES.in

seriously why is a file with no translatable text in here

update copyright year

Summary: update copyright year

Reviewers: zmike, bu5hm4n

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1836

add compile warning note for elm hack

set elm log level to CRI

current elm usage triggers infinite (harmless) widget-parenting errors. to avoid filling hard disks with log files, shut this up for now

don't recalc winlist on client add event if client is not added to winlist

e_box -> elm_box conversion

the next installment in the exciting adventure game Elementary Wars!

happy new year from #teamborker

comp pending resizes should not require both w+h to change

don't show mixer popup during nocomp

fix T1270

use frame geometry in win resize trap to get accurate sizes

ref T1905

add back notification merging

this was removed without warning or explanation in r82118:

    e: Change notification module to use internal notification API
    Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

currently broken by T1954 for lines longer than 36 characters

config: Fix segfault

We can pass here NULL, win does not have a specific saved e_comp.
So we pass NULL to e_comp_get and get the current active one.

This fixes T1937.

Revert "all dialogs are now resizable"

This reverts commit 9157ca8ca739e3b62442fef9d73367d381ae7306.

this also makes all dialogs that have content in a toolbar that goes
off window at min size resizable - this is actually what you wanted ..
bgut this previous "all dialogs are resizable" thing was uber-lazy.

Conflicts:
src/modules/bluez4/e_mod_main.c
src/modules/conf_bindings/e_int_config_signalbindings.c
src/modules/conf_theme/e_int_config_theme.c
src/modules/mixer/conf_gadget.c
src/modules/shot/e_mod_main.c

call e_powersave_init Before we call e_screensaver_init because the
init of e_screensaver sets up an event handler for the
powersave_update event (which would be 0 unless powersave is init
first).

Thanks to dt9 for the report ! ;)

Signed-off-by: Chris Michael <cp.michael@samsung.com>

e_actions: Fix to parse syntax correctly in key binding settings

Summary:
Correct the way to parse syntax in key bindings for launching application according to syntax guide
and pop an error dialog when a space is detected while activating the action.

Example:
Guide is given like "exe:xterm" with "No whitespace" between param's name and contents.
However, existing way to parse syntax should require "ONE whitespace" between name and contents.
This modification will parse syntax correctly.

@fix

Reviewers: seoz, zmike, Hermet

Subscribers: raster, cippp, cedric

Differential Revision: https://phab.enlightenment.org/D1699

e - randr include - move to master includes (only if building for x)

randr: improve code readability in lid update

Be sure we have active non-lids before deactivating lids.

randr: fix bad copy paste error

systray: revert change which shouldn't go in

randr: add error print

Revert "randr: always call _e_randr_output_active_set"

This reverts commit 7f14d81e6a4afe7217a7ed4d6dac06e4fbf3a983.

Broke LID open/close for me, note to self, more testing before commit.

randr: TODO++ and error print

randr: don't try to enable crtc with no outputs

randr: always call _e_randr_output_active_set

_e_randr_output_active_set should handle if we set active on an active
output

randr: handle (x, y) > 0

x and y can be larger than 0, so we must set a high start value.

randr: remove weird code

If we do E_Randr_Output *out2 = _e_randr_output_find(output->xid), then
output should be equal to out2, else we have several output variables per
output xid which should be fixed correctly.

randr: cleanup try 2

This does not change any behaviour, just moves code:

1. We only try to locate a crtc for an output in _e_randr_output_active_set

And we only do this when we set active.

2. We only search for a crtc in _e_randr_output_crtc_find

This includes looping known crtcs without querying X if it matches.

Please don't revert, but help cleaning. As the current code does not
work for me...

Revert "Randr: Fix null dereference."
This reverts commit 4aef218d28ed66e80d882f052c704d92d11c3c81.

Revert "randr: remember crtcid for fast lookup on reconnect"
This reverts commit 55b4ad41d19f1e4573203bf4296356d8f991e809.

Revert "randr: further cleanup"
This reverts commit 3e02824663a2d6b54a2a9ba6d5d9ab760c42aff2.

these make nvidia randr resolution changing work again. sorry seb.
even though cleanups might be nice... this code is FIDDLY and every
driver seems to work differently, so take it easy and be very careful
- make no assumptions. i know it works on my intel desktop and nvidia
deskop atm. once i get a vga cable i'll fix up intel laptop with
external display that has problems. i can try radeon too later. but
right now - this stuff needs to work. being clean is far less
important than working. :)

Revert "randr: Don't track active outputs"

This reverts commit 01f43e3618d07917a15345e2063b9efd866e4b7d.

this broke nvidia randr - i cant tunr my screens on or off anymore.

po: Remove file from now disabled access module to pass distcheck

Randr: Fix null dereference.

CID1181926

randr: Don't track active outputs

Remove tracking of active outputs to reduce code complexity.

randr: TODO++

randr: remember crtcid for fast lookup on reconnect

When an output is set inactive, it is disconnected from a crtc. Now we
remember the id of the last crtc it was connected to, so we get a faster
lookup on reconnect.

e randr - restore edp checks

the dp checks were looking for edp stuff. but due to naming they appeard as
eDP or maybe edp or EDP - thus were simplified to dp. put more explicit
checks there.

randr: Don't match DP as LID

On my intel laptop DP are my DisplayPort connections, not LID.

randr: further cleanup

Primarly two things removed:
1. crtc->outputs should only contain outputs which should be enabled
So looping them twice in e_randr_apply is definitly wrong.

2. _e_randr_crtc_find should find a crtc for an output
So looping our crtcs again in _active_set is definitly wrong. If
_e_randr_crtc_find does not find us a crtc, it should be fixed there.

e randr - make it work on nvidia - lots of "wrong" things fixed

i've spent a while testing randr on my nvidia box and drviers. it now
works - i can turn screens on and off. i couldn't before. e_randr was
trying to be too smart for its own good. it wasnt matching up outputs
to crtcs, was thinking outputs active when they were not, etc. etc.
etc. - reality is e_randr could do witha  lot of simplification, but
at this stage i'm just going to hammer it into shape to work across as
many drivers and setups as i can test against.

only use elm win title for internal windows if it has been set

e_win: Fix Dialog Titles

We should not set the title to "E" in the trap function of show.
If we do this we will override the icccm title and the netwm name, this
mean e_comp_x will set E as title of the window if the call to
elm_win_title_set was BEFORE the window was shown.

Now the name which was set before will be inserted.

Register the new window into out evas.

Summary:
Otherwise the mouse up event is emitted to the evas but the evas does
not work with it, because it does not know the window it is from. This
leads to a evas_event_down_count_get bigger than 1 which leads to a
missing mouse_out event. With this missing mouse_out event _post_up_handle
in evas_event.c is not called anymore, this means all the click events
are getting emitted to the object which was last clicked.

This fixes T1894.

Test Plan: Drag and Drop a icon somewhere and click somewhere after that, the evas will not get stuck anymore

Reviewers: raster

Subscribers: cedric, abyomi0, zmike, raster

Maniphest Tasks: T1894

Differential Revision: https://phab.enlightenment.org/D1771

store hide_logo state for desklock backgrounds

fix T1714

apply client size based on correct remember value, only apply if value != 0

ref T999

remembering size on a window should also remember maximize state

greatly reduce FAILURE_MAX

ref T1820

remove access module from build

this hasn't been updated in over 1.5 years and has caused me to waste time repeatedly fixing build-related issues. in addition, it uses symbols which are no longer even available in efl.

if nobody updates/maintains this, it will be removed for e20 release

src/modules/access/e_mod_main.c: In function '_cover_new':
src/modules/access/e_mod_main.c:765:4: warning: implicit declaration of function 'ecore_x_input_multi_select' [-Wimplicit-function-declaration]

reduce shape recalc for clients which cannot currently receive input

redo shaped client checks to use shape flags instead of rect pointer, fix shapeless clients

according to the shape extension protocol, the number of rectangles returned should be checked to determine a client's shape. if 0 is returned, the client has no shape, meaning that it either should not be drawn or should have no input region. this improves behavior with various client window types such as tooltips

ref T1820

add failure count for client rendering, drop clients after FAILURE_MAX tries

this traps clients which are showing and hiding themselves extremely quickly and prevents them from crashing us when we try to draw them

fix T1820

fix X crash when trying to apply desktop window profile to client without desk

ensure clients are visible before triggering shape/render queue

everything: terminate bc instead of quit

On some systemd SIGQUIT causes the logger to save a coredump.
Quiting bc here does not need save a backtrace here, the termination is
not a crash its a "closing" of the program.

attempt to run eeze_scanner on failure to connect, not successful connect

ref T1612

only attempt to start eeze_scanner a couple times before giving up

ref T1612

store eeze_scanner connection poller to avoid creating an infinite number

ref T1612

don't exit e_fm when eeze init fails, send RASTER mode instead

print more descriptive error messages when scanner connection fails, also don't exit() here

ref T1612

AUTHORS: Add Marcel to authors file.

AUTHORS: Add Luis to authors file.

Thanks for your contributions, let's hope they are the first of many. :)

backlight: remove code duplication

Move repeated blocks of code to a static function

backlight: visual feedback when adjusting value through keybindings

The backlight module has an action to show the value of the backlight brightness
by showing the gadget's popup. Added a parameter to this action to adjust the
brightness at the same time.

ibar: Fix "additional action"s and seperator placement.

Summary: This moves the additional actions to the top of the menu and the seperator right underneath.

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1694

e_start_main: Close fd only if we actually succeeded opening it.

In the error case fd would be -1 here. Closing needs to be handled in the
cases where fd != -1.

CID: 1256492

e-start: Fix 'control reaches end of non-void function'

Summary: Seems one of the Daniels did not check their compiler output
;) _e_ptrace_attach function should return an integer value...

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

fix indentation

remove unused variable

e_start_main: refactor (reduce scope depth, separate ifdefs, cleanup)

The main reason for this change is to clean it up for proper non-Linux
platform support, but also to make it cleaner in general.

fix efm dnd crash

fix T1880

e randr - fix warning (signed vs unsigned cmp)

cpufreq: teach cpuinfo_{min,max}_freq as available frequencies

Summary:
The intel_pstate scaling driver exposes the `scaling_cur_freq` since
kernel 3.17 [*], it would be fine that the min and max frequencies
are known even without the `scaling_available_frequencie`.

This commit teaches to use the cpuinfo_{min,max}_freq as fallback in
case the `scaling_available_frequencies` is not available within
intel pstate.

[*] https://lkml.org/lkml/2014/11/11/1060

Signed-off-by: Takeshi Banse <takebi@laafc.net>
Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1686

e - randr - stop saving config using XID - this is utterly WRONG

so e was storing randr config using XIDs to match outputs etc. this is
all kinds of wrong. XIDs are NOT STATIC. they change from xserver to
xserver and from run to run. they MAY be the same. they may not. so
this was just broken.

use output name + edid as a big "string" (name.edid) as a way ofr
identifying config for a specific combination of output plus monitor
and to find/identify the corrent output+monitor to apply it to (of
course missing edid gets replaced with ??? and missing output name is
??? too - i have never seen a missing output name so you get this at
least).

so this FIXES "restore" of screen mode on login for starters. this
does nothing to "fix" the screen setup dialog in any way. there are
separate issues there.

this also breaks e_randr config compat so i bumped epoch so your old
config is rejected. i don't see a sensible way of porting the config
forward.

apply pointer warp effects immediately when instant warping

fix T1462

detab e_start_main.c; spank spank spank

clean up E ptrace detection and usage

remove e_mod_xembed.c from POTFILES.in

ensure string safety on dbus message in connman agent

fix T1812

force shape queue on notification delete

fix T1801

remove xembeds from systray module