[haiku-commits] Re: r37017 - in haiku/trunk: build/jam src/apps/screenshot

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 06 Jun 2010 11:19:29 +0200

wpjvandermeer@xxxxxxxxx wrote:
> - Separate executables for the CLI (Screenshot) and GUI 
> (ScreenshotApp).
>     When "Screenshot" is launched it runs in the background, and 
> launches the
>     GUI (if not run with the -s or -c switch) before quiting itself. 
> When a new
>     screenshot is requested through the ScreenshotApp GUI interface,
>     ScreenshotApp launches Screenshot before quiting itself, after 
> which
>     Screenshot re-launches ScreenshotApp. This fixes ticket #4100.
>     Note that because of this change the deskbar entry will show 
> "ScreenshotApp"
>     and not "Screenshot" as might be expected. I am not sure if this 
> is a
>     problem.

Not by itself, but I think it should be renamed to "screenshot", and 
"Screenshot" - CLI utilities are usually lower case, and I find that 
"App" suffix rather superfluous.
If that is too similar, we could also name the CLI utility 
"screenshooter", for example, which would even contain an hint as to 
what it does in comparison with the GUI application :-)

Bye,
   Axel.


Other related posts: