[E-devel] Provide sensible errors for edje utilities

2011-06-14 Thread Ulrich Eckhardt
Hi! If you start edje_player or edje_inspector with a path to a file that doesn't exist you get a bogus error message saying that the file doesn't contain any groups. The attached patch uses access() to check if the program can read the file, giving a sensible error message if not. I have chec

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-14 Thread Mike Blumenkrantz
On Wed, 15 Jun 2011 07:34:08 +0200 Ulrich Eckhardt wrote: > Hi! > > If you start edje_player or edje_inspector with a path to a file that doesn't > exist you get a bogus error message saying that the file doesn't contain any > groups. The attached patch uses access() to check if the program ca

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-14 Thread Vincent Torri
On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: > On Wed, 15 Jun 2011 07:34:08 +0200 > Ulrich Eckhardt wrote: > >> Hi! >> >> If you start edje_player or edje_inspector with a path to a file that doesn't >> exist you get a bogus error message saying that the file doesn't contain any >> groups. The

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-14 Thread Mike Blumenkrantz
On Wed, 15 Jun 2011 07:55:27 +0200 (CEST) Vincent Torri wrote: > > > On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: > > > On Wed, 15 Jun 2011 07:34:08 +0200 > > Ulrich Eckhardt wrote: > > > >> Hi! > >> > >> If you start edje_player or edje_inspector with a path to a file that > >> doesn't exis

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-14 Thread Vincent Torri
On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: > On Wed, 15 Jun 2011 07:55:27 +0200 (CEST) > Vincent Torri wrote: > >> >> >> On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: >> >>> On Wed, 15 Jun 2011 07:34:08 +0200 >>> Ulrich Eckhardt wrote: >>> Hi! If you start edje_player or edje

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-14 Thread Mike Blumenkrantz
On Wed, 15 Jun 2011 08:00:33 +0200 (CEST) Vincent Torri wrote: > > > On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: > > > On Wed, 15 Jun 2011 07:55:27 +0200 (CEST) > > Vincent Torri wrote: > > > >> > >> > >> On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: > >> > >>> On Wed, 15 Jun 2011 07:34:08

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-14 Thread Vincent Torri
On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: > On Wed, 15 Jun 2011 08:00:33 +0200 (CEST) > Vincent Torri wrote: > >> >> >> On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: >> >>> On Wed, 15 Jun 2011 07:55:27 +0200 (CEST) >>> Vincent Torri wrote: >>> On Wed, 15 Jun 2011, Mike Blume

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-15 Thread The Rasterman
On Wed, 15 Jun 2011 07:34:08 +0200 Ulrich Eckhardt said: well the error isnt bogus.. it's correct. it contains none.. that you can read :) but yes better error message could be good. it's an improvement and the code isnt bad. :) in svn! > Hi! > > If you start edje_player or edje_inspector with

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-15 Thread Ulrich Eckhardt
On Wednesday 15 June 2011 07:56:25 Mike Blumenkrantz wrote: > On Wed, 15 Jun 2011 07:55:27 +0200 (CEST) > Vincent Torri wrote: > > On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: > > > I like the idea of better error messages, but using access creates a > > > permissions loophole where the file coul

Re: [E-devel] Provide sensible errors for edje utilities

2011-06-15 Thread The Rasterman
On Wed, 15 Jun 2011 21:27:13 +0200 Ulrich Eckhardt said: > On Wednesday 15 June 2011 07:56:25 Mike Blumenkrantz wrote: > > On Wed, 15 Jun 2011 07:55:27 +0200 (CEST) > > Vincent Torri wrote: > > > On Wed, 15 Jun 2011, Mike Blumenkrantz wrote: > > > > I like the idea of better error messages, but