Rev 173 | Go to most recent revision | Details | Compare with Previous | Last modification | View Log | RSS feed
Rev | Author | Line No. | Line |
---|---|---|---|
129 | dmarschall | 1 | |
2 | Wishlist/ToDo by Daniel Marschall <www.daniel-marschall.de> |
||
3 | =========================================================== |
||
4 | |||
5 | Important/Bugs |
||
6 | -------------- |
||
7 | |||
8 | * In ui_mac.c, there does not seem to be a limit for the expressions. So, is it possible |
||
130 | dmarschall | 9 | to write an expression longer than 1023 bytes, which would lead to a crash? |
129 | dmarschall | 10 | |
11 | * Test with 64 Bit Mac, and publish 1.7 (when can we leave the beta state?) |
||
135 | dmarschall | 12 | Windows 64 is already tested. We just need an updated Makefile. |
160 | dmarschall | 13 | |
14 | * For Mac, please look at the (*) marked entries in version.h and verify the bugs/fixes |
||
15 | and/or apply the changes on Mac. |
||
141 | dmarschall | 16 | |
157 | dmarschall | 17 | |
129 | dmarschall | 18 | Minor priority stuff or ideas |
19 | ----------------------------- |
||
20 | |||
167 | dmarschall | 21 | * persistent_savestate (enabled with GIMP/PSPI) should somehow delete the temporary AFS file |
22 | at each restart of GIMP. Otherwise, the user would always see the previous session when they re-open GIMP. |
||
23 | |||
160 | dmarschall | 24 | * Create and evaluate more testcases, and either fix differences between FilterFactory and |
25 | FilterFoundry, or document them in "Filter Factory Compatibility.txt" |
||
134 | dmarschall | 26 | |
129 | dmarschall | 27 | * Should an expression like "----r" be forbidden? (Multiple negate signs) On the other hand, FilterFactory |
28 | didn't forbit them either. |
||
29 | |||
30 | * CMYK mode is possible (although a bit misleading to have r=c, g=m, b=y, a=k), but then it is impossible to |
||
31 | control the alpha channel. |
||
32 | |||
33 | * Other Telegraphics Filters have the same problem in VERSIONINFO where the translation |
||
34 | strings "04900000" don't fit together "080904B0" (fixed in Filter Foundry) |
||
35 | |||
36 | * I have found following in the source code: Do we need to do something here? |
||
37 | strcpy(gdata->parm.formula[i],expr[i] ? expr[i] : "bug! see builddlgitem"); |
||
38 | |||
39 | * Add pow() or "**" operator? |
||
40 | |||
166 | dmarschall | 41 | * Search for "TODO", "FIXME" and "Codereview" in the code |
129 | dmarschall | 42 | |
141 | dmarschall | 43 | * Evaluate the file "Filter Factory Compatibility.txt" and think about if we should |
44 | change some of our functions to become 100% compatible with Filter Factory. |
||
45 | |||
46 | * There is no warning if a formula contains a number that exceeds 32 bits. |
||
129 | dmarschall | 47 | |
152 | dmarschall | 48 | * Check, if all combinations of imports work: |
49 | |||
50 | Category: Premiere Ext. Sig. Testcase file(s) WinImport MacImport |
||
51 | ----------------------------------------------------------------------------------------------------------- |
||
52 | Premiere Settings File .pff n/a premiere*.pff Works TODO: Check |
||
53 | Import Premiere for Windows 16bit .prm 0x208C premiere40.prm Works TODO: Check |
||
54 | Import Premiere for Windows 32bit .prm 0x208C premiere50*.prm Works TODO: Check |
||
55 | Import Premiere for MacOS ??? ??? ??? ??? ??? |
||
56 | |||
57 | Category: FilterFactory Ext. Sig. Testcase file(s) WinImport MacImport |
||
58 | ----------------------------------------------------------------------------------------------------------- |
||
59 | FilterFactory Settings File .afs n/a factory30.afs Works TODO: Check |
||
60 | Import FilterFactory for Windows .8bf 0x2086 factory30.8bf Works TODO: Check |
||
61 | Import FilterFactory for MacOS .bin 0x1C86 mac_filterfactory.bin Works TODO: Check |
||
62 | |||
63 | Category: FilterFoundry Ext. Sig. Testcase file(s) WinImport MacImport |
||
64 | ----------------------------------------------------------------------------------------------------------- |
||
65 | Import FilterFoundry 1.6 for Windows .8bf 0x1C86 foundry16.8bf Works TODO: Check |
||
66 | Import FilterFoundry 1.7 for Windows .8bf 0x2086 (Wait for 1.7 release) |
||
154 | dmarschall | 67 | Import FilterFoundry 1.6 for Mac ??? ??? ??? ??? ??? |
68 | Import FilterFoundry 1.7 for Mac ??? ??? (Wait for 1.7 release) |
||
150 | dmarschall | 69 | |
70 | * Minor bug: Testcase rst_3.afs applied to a 1000x1000 canvas: |
||
71 | When the preview is zoomed in to 29% or 59%, and the preview is panned, the |
||
72 | bars change during panning. It does not look "smooth" like in 100%, 50%, or 25% zoom. |
||
154 | dmarschall | 73 | The problem is that the offset of the preview area is always different, and if the zoom |
74 | level is not a multiple of two, you will always "pick" other bars. |
||
156 | dmarschall | 75 | |
178 | dmarschall | 76 | * In regards "Actions/Scripting": |
77 | fixaete(): Replace the cTl0...cTl7 descriptions with the names of the slider! |
||
141 | dmarschall | 78 | |
157 | dmarschall | 79 | * Fast (double) click in [+]/[-] scroll buttons is not accepted as 2 clicks / zoom-requests |
156 | dmarschall | 80 | |
159 | dmarschall | 81 | * Support more colors modes and 16bit. Why is Lab color not accepted, although doesSupportLABColor is set? |
82 | |||
83 | * Add Unicode support? (Does PiPL support Unicode at all?) |
||
157 | dmarschall | 84 | |
159 | dmarschall | 85 | * Build filter: Individualize Manifest resource (Attention: htmlentities) |
157 | dmarschall | 86 | |
159 | dmarschall | 87 | * Picture with 78x63 in grayscale mode: In preview on the left side there is a black bar |
88 | |||
164 | dmarschall | 89 | * With the change in 1.7 in re "preview evaluates whole picture if state changing functions rst,put,rnd are |
90 | used": Is there anything we can do to make it more performant when the user changes sliders? |
||
91 | Especially with large pictures, the process becomes a bit laggy! |
||
166 | dmarschall | 92 | |
93 | * Currently, we use WIN_ENV and MAC_ENV to decide which Endianess we use. However, if we want to support |
||
94 | Linux (with GIMP?), then we should use other compiler defines. |
||
95 | Check, what is necessary to support Linux? |
||
171 | dmarschall | 96 | |
97 | * Check why the plugin crashes Premiere 5 (called in image filter mode; obviously not as transition filter) |
||
159 | dmarschall | 98 | |
164 | dmarschall | 99 | |
129 | dmarschall | 100 | Design/UI Tweaks |
101 | ---------------- |
||
102 | |||
140 | dmarschall | 103 | * The exclamation mark symbols should have a tooltip ("hint") when the mouse hovers over it. (Also for Mac) |
129 | dmarschall | 104 | Not sure how easy it is to do that with WIN32 API |
105 | |||
106 | * Should it be possible to zoom more than 100%? |
||
107 | |||
108 | * The vertical scrollbar should auto-hide when the expression formula is short. (Also for Mac) |
||
109 | Unfortunately, this task is very hard in WIN32 API. |
||
110 | |||
111 | * Should there be more space for expressions, like in Filter Factory? |
||
130 | dmarschall | 112 | |
113 | * Standalone filter: Map labels should be centered between two sliders, like in Filter Factory |
||
159 | dmarschall | 114 | |
115 | * Sliders are extremely tiny! |
||
116 | |||
117 | * simplealert() is only "appmodal" and has no hWnd. Due to this, the window is not blinking when |
||
118 | the user tries to click on a parent window. (Is this a Windows bug?) |
||
163 | dmarschall | 119 | |
120 | * When a standalone filter is built, maybe the user should get a notification that it was successful. |
||
121 | Something like a simplealert(), but with "information" as message type, not "error" |
||
164 | dmarschall | 122 | |
166 | dmarschall | 123 | * Support BigDocument |
164 | dmarschall | 124 | |
166 | dmarschall | 125 | |
164 | dmarschall | 126 | Things to check/correct for other Telegraphics plugins (fixed in FilterFoundry) |
127 | ------------------------------------------------------ |
||
128 | |||
169 | dmarschall | 129 | * The lengths of the PIPL TLV structure must include everything, including string paddings, |
130 | with 1 zero padding guaranteed. See annotations in "GIMP Incompatibilities". |
||
164 | dmarschall | 131 | |
132 | * The language/country code in VarFileInfo must be a code that exists in StringFileInfo. |
||
166 | dmarschall | 133 | |
134 | * 64 Bit Windows builds |
||
135 | |||
136 | * XP Manifests (+ activation contexts) |