Opened 7 years ago
Last modified 6 years ago
#13952 new bug
Make Icon-O-Matic usable from command line so that it can convert icons from IOM to HVIF/SVG
Reported by: | owenca | Owned by: | stippi |
---|---|---|---|
Priority: | normal | Milestone: | Unscheduled |
Component: | Applications/Icon-O-Matic | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
Usage:
Icon-O-Matic -[h|s] -o output_file IOM_file
It returns 0 if successful or 1 otherwise.
Attachments (3)
Change History (10)
by , 7 years ago
Attachment: | 0001-Icon-O-Matic-convert-IOM-to-HVIF-SVG-from-command-li.patch added |
---|
follow-up: 3 comment:1 by , 7 years ago
follow-up: 4 comment:2 by , 7 years ago
Hmm, I'd rather avoid using -h as it's traditionally used for help.
Also, you don't need to check for NULL before calling delete.
I don't see any reason to remove member init from the constructor, it's not like it creates any window.
comment:3 by , 7 years ago
Replying to jim saxton:
Might as well add -R for RDef and -S for HVIF Source Code
Done. (I wasn't sure if there was a special reason that the GCI task didn't ask for these two formats.)
comment:4 by , 7 years ago
Replying to François Revol:
Hmm, I'd rather avoid using -h as it's traditionally used for help.
Changed to -H
. (I had the same concern, but after trying -h with mv/cp, I thought it was not the case anymore.)
Also, you don't need to check for NULL before calling delete.
Fixed. (I got it mixed up with double free in C.)
I don't see any reason to remove member init from the constructor, it's not like it creates any window.
The _Init() function creates two file panels and calls _RestoreSettings(). I wanted to bypass it for the command-line usage to make it run faster and to avoid any possible side effects unknown to me. If it's better to have the same constructor for both usage, I can put it back in.
by , 7 years ago
Attachment: | 0001-Icon-O-Matic-convert-IOM-to-HVIF-RDef-Source-SVG-fro.patch added |
---|
Replacing the previous patch.
by , 7 years ago
Attachment: | 0001-Icon-O-Matic-convert-IOM-to-HVIF-RDef-SVG-source-fro.patch added |
---|
Replacing previous patches.
comment:6 by , 7 years ago
Please submit patches to Gerrit instead of attaching them to the bugtracker.
comment:7 by , 6 years ago
Component: | - General → Applications/Icon-O-Matic |
---|---|
Owner: | changed from | to
Might as well add -R for RDef and -S for HVIF Source Code