Code review comment for lp:~albaguirre/unity8/add-screenshotter

Revision history for this message
Alberto Aguirre (albaguirre) wrote :

> +QString ScreenShotter::generateName()
> +QString ScreenShotter::generateUniqueNum()
> + // First lets look for existing files using our numbering pattern
> instead of this complex numbering pattern, why not just use current date &
> time?

Ok I guess I can do that. I suppose I just wanted it to produce sequentially numbered files as most other products do when saving say camera snapshots.

« Back to merge proposal