shot - only access e client if not null

fix evry comparison macro to work as expected

missing parens = !operating as intended in many cases

fix return value in backlight helper exe on non-freebsd systems

ref D2337

fix nocomp_override conditional which would inaccurately toggle nocomp

hooray gcc 5.0

L10N: Update Finnish translations

tempget: Add lm75(4) temperature source for non-x86 FreeBSD

Summary:
Another suggestion from Dmitry Luhtionov. I am told this sensor (on the
i2c bus) is more likely to be present on non-x86 systems (ARM, etc).

Reviewers: zmike, q66

Reviewed By: q66

Subscribers: cedric, seoz

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

backlight: Use basic ACPI sysctl knob on FreeBSD (kernel)

Summary:
Unconditionally build the suid-helper _backlight program; conditionalize
behavior on Eeze, FreeBSD (doing nothing on neither). Add logic to set
the FreeBSD sysctl in a similar manner to udev devices on Linux.

Add _bl_sys_find/_get helpers for FreeBSD that check for and consult the
video ACPI sysctl.

Test Plan:
Seems to work okay on my laptop (with EINA_CPU_FAKE=1 to workaround a threadq
race that I believe is unrelated -- T2287).

Reviewers: zmike, q66

Reviewed By: q66

Subscribers: cedric, seoz

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

Generate *all* music control files in src tree.

This fixes once and for all out of tree builds.

allow client icon change events in ibar client prop handler

fix T2284

set E_Exec_Instance desktop during client idler if none present

ref T2284

check correct number of params passed to internal/desk cfd

fix T2221

only set shot window layer if taking window shot

fix T2282

fix desktop editor entry widget parenting

fix T2283

do not attempt to re-center fullscreen/maximized internal windows

fix T2279

modules/bluez4: fix duplicated symbol

Summary:
Global variable should be declared as extern to avoid symbol duplication.
Fix compiling of ebluez4 module on OS X.

Test Plan:
Since Enlightenment cannot (yet) be fully compiled on OS X, I cannot test
the real-life impact of this change.

Reviewers: zmike, raster

Subscribers: cedric

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

toggle comp_hidden and grab input while client color editor is active

Tiling: don't tile centered windows.

This broke with the change to elm (so within this release cycle). Tiling
wasn't properly detecting windows who become centered after creation as
there was no e event for that. This is now fixed, and a blanket solution
for similar gotchas has been implemented (otherwise known as a hack).

Thanks to Mike for helping out.

tempget: Add more temperature sources for FreeBSD

Summary:
Not all motherboards have 'hw.acpi.thermal.tz0.temperature' sensor.
Fallback to 'dev.cpu.0.temperature' (provided by coretemp(4) or
amdtemp(4)), or 'dev.aibs.0.temp.0' (aibs(4)) (per suggestion of Dmitry
Luhtionov).

Test Plan: Ran `src/modules/temperature/tempget` manually with some debugging printfs and various sensors commented out. I can only test tz0 and dev.cpu (coretemp(4)) on my system.

Reviewers: zmike, q66

Reviewed By: q66

Subscribers: seoz

Projects: #enlightenment-git

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

e-comp-wl: Remove unused variable

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

MEDIUM: modules/shot: switch to https://

This is needed since www.enlightenment.org is now using https:// only due to
login forms

build: Remove generated eldbus files during normal clean and not only maintainer-clean

These generated files should be cleaned up during a normal run of make clean.

make connman agent dialog resizable and autosized

fix T2279

Module Music control: Don't build if disabled.

The whole point of supporting disabling modules is to actually
be able to disable them. :).

@fix.

add xkb options scroller as subobj to toolbook widget

ensure chain of e_widgets for cfd changed policy

fix T2146

modules/battery: fix compiling on OS X

Summary:
Battery module relies on Apple's CoreFoundation (CF), but CF's CFLAGS
were not provided to this module.

Test Plan: Enlightenment now compiles on OS X.

Reviewers: zmike, raster

Subscribers: cedric

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

remove cslider widget

no longer used

remove csel widget

no longer used

redo color dialog to use elm colorselector

this is waaaaaaay less functional and good-looking, but at least it doesn't
crash

make entry widget call changed callbacks on "changed" again

this caused infinite loops before, but now I'm removing the things that looped.

fixes a lot of config dialogs

ref T2271

don't redo min size hints on e_dialog show

these are guaranteed to be wrong because elm won't provide accurate size
hints without some arm twisting

force dialogs to recalc content sizes before setting min size

fix entry sizing in file props dialog

ref T2271

set xkb dialog as changed after adding a new layout

ref T2271

set desktop editor appselector dialog to be resizable

ref T2271

match client dialogs to client's layer

a small annoyance for literally over a decade has been that window
dialogs would sometimes start behind their owners. now they don't.

raise deskmirror windows on startup

these get added bottom -> top on creation, so ensure that they are raised

always lower e_layout children upon packing

fixes issues where a new child added would appear above other child objects
if the new child was meant to be stacked below other children

mostly noticeable in deskmirror

remove layer setting from deskmirror

smart child objects can't have layers. apparently.

Mixer Pulse: Fix trivial use-after-free.

Summary: Don't use memory after freeing it.

Fixes T2274

@fix

Reviewers: raster, zmike

Subscribers: abyomi0, zmike

Projects: #enlightenment-git

Maniphest Tasks: T2274

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

Commit message updates by: Tom Hacohen

add options to randr2 to ignore acpi and hotplug randr events.

don't crash on wl shutdown

devilhorns? more like crasherhorns imo

e_comp: Only need 'type' variable if we are building with HAVE_WAYLAND

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

build: Use EFL_WITH_BIN instead of E homegrown one.

We already have the efl_binary.m4 macro available with EFL_WITH_BIN which
basically is the successor of this macro.

build: Allow to define path for eldbus-codegen binary

Since the geolocation and music-control changes we need eldbus-codegen to
generate some files. Not all setups have this in their normal $PATH so let
allow these setups to set the correct path during configure. This allows our
Jenkins setup as well as others to work again.

remove all debug output from mixer module

fix T2274

only call e_comp_wl_shutdown() for wayland compositors

prepend geolocation build dir to cppflags

elocation headers conflict with these, so ensure that our headers get picked up first

 #glimaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

music-control: Mimic explicit file deps fix from geolocation.

This was already firxed for geolocation in
7a1163a1adb942f49380e848c809129c2c0b9c89 but I forgot to integrate it in
my last patch. Thanks to Scimmia for pointing it out.

everything: Actual return something in non wayland only-mode

While making it work for wayland only the return got missing.

everything: win variable is only used inside ifdef.

We only use this in not wayland only mode so ifdef the variable as well.

music-control: Generate eldbus files during compile

Better have up to date eldbus files generated from the xml instead of
keeping the generated files around.

 Bugfix: e dialog: don’t crash when using the escape key to close.

 Commit 99589eb5af346add9fe912a768ed006416cb2220 introduced this bug as the callback creation wasn’t changed to suit the new prototype and stil passed a window object.

remove e_color_class

move colorclass config to use elm infra, redo cc dialog to use new editor

this will allow all colorclasses present in the current theme to be edited
instead of only the hardcoded ones in the module. it will also require
completely new translations, for which translators will need to read the edc
files of the default theme (cleverly located in another repo) and provide
translations to the _translate() callback in the theme module

the editor currently lacks indicators for active/type on the colorclass
list, but this can be added in time. meanwhile, a large amount of code is
no longer duplicated or maintained in this repo

wl-drm: If we end up using software_drm, then override elm preferences (ala X compositor)

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

wl-drm: Hook into ecore_evas_resize callback to update comp canvas

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

wl-drm: Remove output event handler when we shutdown

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

wl-drm: Remove need for extra E_Comp variable

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

fix wl randr error message string

make application color editor follow client geometry on move/resize

add explicit file deps for geolocation eldbus generated files

some setups have stricter dep chain following during make which causes
build errors if the chain does not explicitly have the generated files
as a dep of the files for which they are included

add colorclass editor to app menu

block ignored clients from lost client list

e - lost windows menu is back. no windows menu does not handle it.

fix lack of lost windows submenu. no - windows menu does not handle
it. lost windows lists windows that are LOST - out of screen bounds.
this happens a lot with some apps asking to be placed out of bounds
for whatever silly reasons they have. you literally cannot get these
windows back without this menu. bring it back!

@fix

geolocation module build: now with fewer generated files!

also fix distcheck of geolocation edj file by not compiling it

fix geolocation compile

modules/geolocation: Add source file for artwork.

In case someone wants to play around with it. Given my little art skills I would
be happy if anyone wants to enhance this. :)

modules/geolocation: Ignore now generated edj in git

modules/geolocation: Add module theme and icons.

This is likely to change a bit until it stabilize so we keep the theme
with the module for now.

module/geolocation: No need to init and shutdown eldbus as E handles that

Another thing spotted by the review from zmike.

modules/geolocation: No need to set popup to NULL here.

This is already handled by E_FREE_FUNC. Thanks to zmike for spotting this
during review.

e_module: Whitelist geolocation module

modules/geoloation: Do not stop the client for now as we cannot enable it again

There is a problem with GeoClue2 version 2.0.0 that does not allow to start a
client again after stopping it. Need to investigate newer versions about this.
For now keep it running until it times out.

modules/geolocation: Fix naming for the whole module to be geolocation

It still ahd some bits from the original name (geoclue2). I tried squeeze
as much renaming as possible in the earlier commits but these done here
would have broken the code so I do the rest all in one go here.

modules/geolocation: Add popup with location information to gadget

modules/geolocation: Add communication to geoClue2 to receive current location.

This basically is the GeoClue2 client functionality a normal application also
uses. We request the current location and subscribe to updates on it.

Only activated when the users does a left click on the gadget so we do not
run always run it in the background. The received information will be displayed
in a popup in a following commit.

modules/geolocation: Keep the UI indicator in off state at start

modules: Add new geolocation module to handle location access requests

This implements an GeoClue2 agents which gets called whenever an application
tries to access the location resource. It allows the user to allow or deny
this resource to the application.

Update catalan translation

make only critical notifications display in presentation mode

ref T2212

move wl E_Client creation to same place as E_Pixmap creation

ensure that surface manipulation functions will succeed even with no shell surface

remove useless wl comp code for shape exports

fix e_remote bg commands

decrement e_comp->new_clients counter when unsetting new_client flag on wl cursor clients

mobile policy desk set hook needs to ignore new_client clients

desk set happens during client init, so these will always be new clients

ref D2242

Updating spanish translation

fix debug message for wl_output_bind

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

wl-drm: Add more debugging during output hotplug events

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

add debugging when output gets bound, and fix formatting

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

wayland compositor: remove _e_comp_wl_client_focus()

Summary:
We now have a subset of that functionality in e_comp_wl_input.c, so use
that function instead.  The missing bits are moved into the one remaining
caller.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

remove E_Comp from gadman and systray structs

wayland compositor: Send keyboard enter events on get keyboard

Summary:
Now when a client gets the keyboard global it will immediately be sent
a keyboard enter event if appropriate.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

lokker: update caps-lock hint even on "checking" state

Summary:
This fixes the inconsistency in the "Caps Lock is On" hint, if you press
Caps-Lock when lokker is busy with checking the password
(LOKKER_STATE_CHECKING).

Test Plan:
  1. Lock the screen
  2. Enter invalid password (and press Enter)
  3. During password check, press Caps-Lock
The "Caps-Lock is On" should be consistent with the actual Caps-Lock
state.

@fix

Reviewers: zmike

Subscribers: cedric

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

wayland compositor: Remove focus_update from compositor client data

Summary:
The only use for this was sending keyboard enter events on first commit
after a focus during app startup, and this has proven to be unreliable.

NOTE: Focus before an app requests the keyboard global is now broken and
will be fixed in a following patch.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

move comp fake layer init into canvas init for non-X compositors

move E_Comp->root window setting into e_comp_wl_init() for wl compositors

remove e_util_head_exec()

remove E_Remember->prop.head

head is always 0

remove E_Comp->num

num is always 0

fix straggler container text in mousebindings config