Re: jdgordon: r17690 - in trunk/apps: . gui

2008-06-05 Thread Dave Chapman
[EMAIL PROTECTED] wrote: Date: 2008-06-05 10:20:39 +0200 (Thu, 05 Jun 2008) New Revision: 17690 Log Message: make %V a little simpler. only the x and y values have to be specified now (i.e %V|0|0|| ) default values are as follows: width - lcd width - the x value height - lcd height - y

Re: jdgordon: r17690 - in trunk/apps: . gui

2008-06-05 Thread Thomas Martitz
Dave, you have some good points there. I agree with most, especially the part with the happily accepting invalid syntax, that's a no-go IMHO. I also think that for e.g. the height/width the code complexity is not really worth it. But I might add, I actually like at least the optional colors

Re: jdgordon: r17690 - in trunk/apps: . gui

2008-06-05 Thread Jonathan Gordon
2008/6/6 Dave Chapman [EMAIL PROTECTED]: IMO it's debatable whether this makes things simpler, it's adding code complexity, and just saves WPS authors a little typing (which they do once). this allows the same .wps to be used on more targets than just one Given the heavy use of bitmaps (and

Re: jdgordon: r17690 - in trunk/apps: . gui

2008-06-05 Thread Jonathan Gordon
2008/6/6 Jonathan Gordon [EMAIL PROTECTED]: Yes, thats correct.. again, 2 options... only accept missing values if they are completly missing, or add some printf warnings Actually, I'd rather force a - to say the param was purposly left out