On-line errors listing

Time: 20/05/01,10:08:03

Reply form

[JP, 8.-9.11.97]
nevim, jake chyby mam vracet z cFilter::load() (kdyz najdu spatny token nebo nastane EOF). Zatim jsem tam dal ERR_FILEBADFORMAT. V chybovych kodech je taky treba udelat poradek! Cim drive to udelate, tim mene se toho potom bude muset opravovat!

[JP, 10.11.97]
potrebuji rutinu, ktera omezi masku na dany obdelnik - to by se melo delat, kdyz nejsou stejne velke vstupy a na vystup chci masku okopirovat!

[JP, 15.11.97]
GIF, BMP, atd. by pri save mely spocitat, kolik barev se v obrazku vyskytuje a ulozit nejmensi vetsi mocninu dvojky (aby se po nacteni 2-barevneho GIFu neulozil 256-barevny!). Prip. bude nutne zpermutovat paletu v cXImage! Tyka se jen formatu, ktere umeji palety kratsi nez 256 (urcite GIF)..

[JP, 15.11.97]
v TGAopen je ne zcela spravny prikaz: "if ( fread(&res1,2,1,file)+fread(&res2,2,1,file)!=2 ) goto fail;"
Myslim, ze se v jazyce C(++) nezarucuje poradi vyhodnocovani souctu! Dejte to do dvou logickych vyrazu, u logickych spojek je poradi zaruceno...

[JP, 15.11.97]
i v TGA se alokuje velke pole pomoci new!

[MH, 19/11/97]
GIF v gif.cpp includuje Gifgloba.h misto gifgloba.h a odstranit #define NULL a #define ASSERT (nebo zavrit do #ifndef #endif) v hlavickovych souborech

[JP, 19/11/97]
prepocitavat "alpha" v konvolucnich (a jinych) filtrech? Zatim u AVERAGE prepocitavam a u DIFFERENCE propaguji z 1. vstupu...

[JP, 19/11/97]
vsichni clenove tymu budou ve svych updatech uvadet soubor "changes.txt", ve kterem napisou:

  - aktualni datum
  - datum posledniho (predchoziho) update/distribuce - tj. PROTI CEMU jsou brany zmeny!
  - seznam vsech zmenenych souboru (ktere spadaji pod jejich kompetenci)
  - seznam vsech smazanych/prejmenovanych souboru (ktere spadaji pod jejich kompetenci)
  - seznam vsech zmen v cizich souborech (tim nemyslim navrhy zmen nebo chybova hlaseni - ta patri
    jinam! - ale zmeny v rutinach, ktere puvodne napsali oni a pak byly presunuty do jineho modulu)

Dale navrhuji, zby nikdo jiny krome koordinatora NEDAVAL na trh nove distribuce. Vsichni ostatni (vcetne pana Chrzanowskeho) prece mohou vyrabet "update" sve casti kodu a koordinator (pan Halva) to pak da dohromady. Radsi si stahnu posledni distribuci a pridam k ni nekolik i netrivialnich update, nez budu zase resit podobny neporadek, ktery se stal 17.11.

[JP, 19/11/97]
mely by se jeste urcite udelat nasledujici I/O formaty: SUN rasterfile, SGI (aby se dala nacitat data vygenerovana programem 'gscan' v nasem grafickem koutku)

[DC, 19.11.97]
TGA pri nacitani otevira soubor R/W - pokud je soubor read-only, pak to spadne

[JP, 21.11.97]
gifglobal.h: cela definice "#ifdef _DEBUG" by jeste mela byt uzavrena do "#ifndef ASSERT"

[JP, 21.11.97]
do vasich update archivu (do souboru changes.txt) piste jeste zmeny v projektu - ktere soubory se maji pridat do makefile/projektu a ktere se maji naopak odstranit!

[JP, 21.11.97]
cXImage::setBounds(): ...color barva okraje. Pokud je obrázek v formátu IF_PALETTED použije se (v tomto pořadí):

1. Pokud v paletě existuje stejná barva použije se ta.
2. Pokud je v paletě volné místo color je vložen do palety.
3. Použije se "nejbližší" barva - [RFC jakou použít metriku ??]

Navrhuji pouzit vazenou Euklidovskou metriku - jako vahove koeficienty pouzit grayWeights[] z meho souboru "jpglobal" (v soucasne verzi jsou jeste v "jpdistance", ale presunul jsem je)

[JP, 21.11.97]
do seznamu filtru (a I/O formatu) nepiste implementacni detaily (nebo popisy nejakych pomocnych trid!). Jenom filtry, jak je muzu pouzit ve workspace. Je to urceno pro uzivatele filtru, takze je dulezita jakakoliv semanticka informace (CO presne to dela), ale ne uz JAK jste to udelali..

Ty informace interniho charakteru budou taky potreba (do programatorske dokumentace), kdyz tak pro to zalozte jine soubory (a jiny adresar).

[JP, 26.11.97]
nefunguje mi save TGA ve formatu RGBA16 (uplne cerny obrazek)

[JP, 26.11.97]
nefunguje mi save PCX ve formatu RGBA16 (vytvori soubor s nulovou delkou)

[JP, 26.11.97]
nefunguje mi save BMP ve formatu RGBA16 (chyba ochrany pameti na u_bmp.cpp:426 - neni to dodelane?)

[JP, 26.11.97]
alpha == 0xFF00 by se mela chapat jako ekvivalentni s 0xFFFF (vznikne to konverzi z BYTE na WORD) Nebo prevadet "word = ((WORD)byte) * 257" ? (misto "word = ((WORD)byte) << 8")

[Kristik, 26.11.97]
SAVE_PCX zapisuje do casti pameti, do ktere mu nic neni.
==========
[JP, 26.11.97] Neslo by to popsat presneji? Jaky format, kde (na ktere radce kodu se to deje)..

[JP, 28.11.97]
Pro vsechny (gr_tga.cpp i jinde): includujte "is_pch.h" na zacatku! Nebo alespon dodrzujte poradi headeru z "is_pch.h". Na mnoha mistech se includuje "define.h" az za nejakymi vasimi headery a to pak nejde prelozit!

Koho to napadlo davat #include "define.h" na posledni misto za vsechny svoje headery? Je to v *mnoha* souborech!

[JP, 28.11.97]
pro vsechny: vite o tom, ze do parametru funkce lze napsat "const"? Skoro mi pripada, ze ne :) Obzvlast u struktur, objektu a stringu (kdyz to nemuzu predat hondnotou, ale odkazem) je to uzitecna vec - kazdy pak dokaze odlisit vstupni a vystupni parametry, vas to ochrani proti nekterym chybam zamena promennych..

[David Cernoch, 30.11.97]
sunicon.cpp:42 je tam test if( fclose( ...) == NULL)... - spravne: if( fclose( ...) != 0)...!!!

[David Cernoch, 30.11.97]
tkostky.cpp includovat is_pch.h

[MH, 1.12.97]
Maskbox nefunguje (zhrouti se) pokud je obdelnik vetsi nez obrazek.

[MH, 1.12.97]

fmatrix.cpp(26): Memory leak; 7 bytes allocated by operator new in D:\PROJECTS\IS\filters\fmatrix.cpp at line 26, HANDLE: 0x026ACA00M
fmatrix.cpp(29): Memory leak; 4 bytes allocated by operator new in D:\PROJECTS\IS\filters\fmatrix.cpp at line 29, HANDLE: 0x026ACA58M
jpwindow.cpp(48): Memory leak; 16 bytes allocated by operator new in D:\PROJECTS\IS\filters\jpwindow.cpp at line 48, HANDLE: 0x026AEFA0M
jpglobal.cpp(67): Memory leak; 7 bytes allocated by operator new in D:\PROJECTS\IS\filters\jpglobal.cpp at line 67, HANDLE: 0x026AF000M
fseprgb.cpp(24): Memory leak; 20 bytes allocated by operator new in D:\PROJECTS\IS\filters\fseprgb.cpp at line 24, HANDLE: 0x027B7120M
fseprgb.cpp(26): Memory leak; 6 bytes allocated by operator new in D:\PROJECTS\IS\filters\fseprgb.cpp at line 26, HANDLE: 0x027B7180M
fseprgb.cpp(28): Memory leak; 12 bytes allocated by operator new in D:\PROJECTS\IS\filters\fseprgb.cpp at line 28, HANDLE: 0x027B71D8M
fseprgb.cpp(30): Memory leak; 14 bytes allocated by operator new in D:\PROJECTS\IS\filters\fseprgb.cpp at line 30, HANDLE: 0x027B7230M
fseprgb.cpp(32): Memory leak; 13 bytes allocated by operator new in D:\PROJECTS\IS\filters\fseprgb.cpp at line 32, HANDLE: 0x027B7290M
alphfilt.cpp(738): Memory leak; 208 bytes allocated by operator new in D:\PROJECTS\IS\filters\alphfilt.cpp at line 738, HANDLE: 0x027C80E0M
alphfilt.cpp(740): Memory leak; 7 bytes allocated by operator new in D:\PROJECTS\IS\filters\alphfilt.cpp at line 740, HANDLE: 0x027C8200M
alphfilt.cpp(743): Memory leak; 8 bytes allocated by operator new in D:\PROJECTS\IS\filters\alphfilt.cpp at line 743, HANDLE: 0x027C8258M
support.cpp(50): Memory leak; 10 bytes allocated by operator new in D:\PROJECTS\IS\kernel\support.cpp at line 50, HANDLE: 0x027C9D48M
gif87.cpp(669): Memory leak; 1196 bytes allocated by operator new in D:\PROJECTS\IS\io\gif87.cpp at line 669, HANDLE: 0x027CE100M
gif87.cpp(472): Memory leak; 136 bytes allocated by operator new in D:\PROJECTS\IS\io\gif87.cpp at line 472, HANDLE: 0x027CF648M
lzw.cpp(41): Memory leak; 24576 bytes allocated by operator new in D:\PROJECTS\IS\io\lzw.cpp at line 41, HANDLE: 0x027D0770M
gif.cpp(139): Memory leak; 768 bytes allocated by operator new in D:\PROJECTS\IS\io\gif.cpp at line 139, HANDLE: 0x027DB3D8M
bitstrea.cpp(107): Memory leak; 65536 bytes allocated by realloc in D:\PROJECTS\IS\io\bitstrea.cpp at line 107, HANDLE: 0x027DB7C0M
gif.cpp(80): Memory leak; 125860 bytes allocated by operator new in D:\PROJECTS\IS\io\gif.cpp at line 80, HANDLE: 0x02846720M
gif.cpp(123): Memory leak; 125861 bytes allocated by operator new in D:\PROJECTS\IS\io\gif.cpp at line 123, HANDLE: 0x02886EB8M

[David Cernoch, 4.12.97]
bsplin_f.cpp funkce guiDrawLine() by mela vzdy byt zavrena mezi #ifdef GUI ... #endif

[JP, 4.12.97]
linkovani plugin DLL s KERNEL.LIB by nemuselo byt na zavadu, protoze se tim velikost vysledne DLL nezvetsi. Zustane jen mensi neprijemnost - nutnost dodavat s distribucemi i ten .LIB soubor a (asi?) binarni neprenostitelnost uzivatelskych pluginu (s novou verzi IS se budou muset vsechny pluginy znova slinkovat ?? nevim jiste - nekteri odbornici tvrdi, ze to neni potreba ??).

[JP, 4.12.97]
navrhuji automatickou registraci pluginu - budeme mit nejaky adresar, ktery se pri startu IS projde a vsechny DLL v nem nalezene se zkusi spustit a zaregistrovat. K tomu ale musi byt dohodnuta vhodna konvence - napr. jakou funkci museji DLL exportovat (aby mohl kernel vyzkouset, jedna-li se o IS plugin - a nejlepsi by bylo, kdyby se take dalo poznat, pro kterou verzi IS byl modul vyroben [zpetna kompatibilita])

[JP, 6.12.97]
"gr_io.h" (a asi i jinde): "is_pch.h" se ma includovat na prvnim miste! A potom se uz nemusi includovat <strdio.h> a <string.h>

[JP, 6.12.97]
kdo a kde pouziva cIOFilter::suffix? A kde presne se ma nastavovat? Ve formatu PNM se pripona urcuje podle formatu souboru (PBM/PGM/PPM) - jak to mam udelat?

[JP, 6.12.97]
imageio.cpp:187 - tady by stacilo "buf->putKey("FORMAT_SPEC");", nebo se v budoucnu pouzije i ta hodnota?

[JP, 6.12.97]
v kucharce pro I/O filtry chybi poznamka, ze ve WSF souboru musi byt parametry filtru oddeleny klicovym slovem "FORMAT_SPEC"

[JP, 6.12.97]
cWorkspace::load() (radka 764): asi by tam melo byt "delete [] strpom;"

[JP, 6.12.97]
cFilter::loadFilter(): nekolik promennych pouzivate jako boolovske priznaky, ale mate je deklarovane jako "int" a dosazujete do nich cisla. Nebylo by lepsi pouzit typ BOOL a konstanty TRUE/FALSE? Mnohem lepe by se ty zdrojaky cetly (kdyz uz tam musim hledat chyby). A co znamenaji ty promenne (is_input, is_output, binputim, bif_inp_image, boutputim, ..)?

[JP, 6.12.97]
cWorkspace::load: je tam spousta hlaseni (chvalim), ALE ten jazyk!!! Ja neumim dobre anglicky, ale tohle mi trha usi (nebo oci :-)

[David Cernoch, 8.12.97]
JPEG v save je nekolikrat (pro kazdy format jednou :-) tento radek PIX(image,0,cinfo->next_scanline-1). Je to vcelku dobre - az na tu -1 ...

[JP, 12.12.97]
existuje nejaka free knihovna pro vystupy na rastrove tiskarny? Ja bych nekde mel mit modul pro tisk na HP laserovych tiskarnach (jazyk PCL) a Epson 9-24 pins (ESC). Jestli ovsem existuje nejaka univerzalni knihovna, bylo by lepsi pouzit ji.

[JP, 12.12.97]
do hlavnich programu dat interaktivni lupu (pohybuji se mysi a prekresluje se mi zvetseny obrazek). Dalsi funkce lupy: zvetseni kontrastu na maximum v kreslenem okenku (treba i dynamicky - nejdriv to rychle nakreslim beze zmeny kontrastu a po chvili to prepocitam)

[JP, 12.12.97]
proc se v cKernel::getFullPath() alokuje 64KB pameti na jeden string???

[JP, 12.12.97]
nekdo by mel napsat memory-manager pro jpeglib (pro velke kusy pameti by se mel volat nas mman)

[JP, 12.12.97]
cImageLoad by mel pocitat s konfiguracni promennou PATH (obrazek zadany pouze jmenem bez adresare by se mel hledat vsude). Pro save() by tam ale porad melo zustat to puvodni (kratke) jmeno, aby ulozeni workspace neznamenalo pripsani cest ke jmenum souboru.. Ja jsem si to docasne upravil v "cImageLoad::load():114":

  file_name = kernel->getFullPath(item->Value);

A co cImageSave? Kam se ma vysledny obrazek ukladat? Nebo tam musi byt absolutni cesta?

[JP, 16.12.97]
WIN_GUI: kdyz nahraji do pameti pouze obrazek, prida se mi do prazdneho workspace. Kdyz potom na nej udelam "delete", spadne cely program.

[JP, 16.12.97]
WIN_GUI: hlavicka okenka s default workspace - ma tam byt "WorkSpace" (bez toho maleho "s" uprostred)

[JP, 16.12.97]
WIN_GUI: "input filter" se nechova rozumne - musi nabizet jako default jmeno souboru nactene z WSF!

[JP, 16.12.97]
WIN_GUI: zmenseni true-color obrazku v hi-color grafickem rezimu vypada mizerne (do modre barvy se primicha fialova, atd.). To zmensovani si asi budete muset delat sam!

[JP, 16.12.97]
WIN_GUI: co je prikaz "Fx" (button na tool-baru)?

[JP, 16.12.97]
WIN_GUI: sedy obrazek (sarah.pgm) se zobrazuje se spatnymi barvami a navic vzhuru nohama (pardon - otoceny o 180 stupnu).

[JP, 17.12.97]
proc ma konstruktor "dialog::dialog()" parametr "gravity", kdyz se (podle kucharky) do nej stejne jeste musi vlozit frame jako kontejner pro vsechny prvky? Ted uz jsem to pochopil - napiste to trochu jinak (aby clovek pro pochopeni nekterych veci nemusel cist cely navod - az v bodu 8. je receno, ze tech ramecku tam muze byt vice..)

[JP, 17.12.97]
WIN_GUI - INPUT_LINE: okenko se kresli prilis male. Zadam, ze chci videt 3 znaky retezce a on mi ukaze tak 1.2 znaku.

[JP, 17.12.97]
WIN_GUI - podivejte se na dialog u filtru PAINTER. Libi se vam rozmisteni prvku?
Proc je levy ramecek tak vysoky (prava cast dialogu to take nepotrebuje)?
Proc staticky text napsany do ramecku prekresli cast toho ramecku?

[JP, 17.12.97]
GUI obecne: proc funkce AddFrame() nema take parametr "align" (jako AddItem)?
Mohl bych si tim napr. rici, ze v nejakem sloupecku chci prvni dva ramecky mit nahore a posledni ramecek (obsahujici napr. tlacitka Ok a Cancel) zase uplne nahore! WIN_GUI mi da prvni nahoru, druhy doprostred a treti dolu (a jeste se musim postarat, aby v tom tretim ramecku byla tlacitka zatlacena dolu..).
Musi se ramecky vzdycky roztahovat na max. velikost? Nebylo by lepsi definovat, ze ramecek zabira tolik mista, kolik potrebuji jeho vlozene prvky?

[JP, 17.12.97]
cFilter - jak jsem pochopil, novy clen "description" je zcela v rezii filtru (pokud je na heapu, musi si ho clovek sam uvolnit). Souhlasim - je nejaky duvod, proc by se takhle nemohly chovat i "name" a "id"? Misto alokace malych kousku pameti na heapu by si kazdy ty stringy udelal globalni staticke (= sdilene!).

Dale navrhuji, aby "cFilter::cFilter()" nastavil i "description" na NULL (pro ty zapomnetlive).

[JP, 17.12.97]
WIN_GUI - nefunguje mi dialog v "cFDistance::run()". Nez jsem tam pridal posledni 4 ramecky (v kazdem z nich je jeden STEXT a jedna INPUT_LINE), tak to chodilo (viz jp971217). Nesmi se spolu michat ramecky a prvky? (ja tam mam STEXT a pak 4 ramecky)

[JP, 29.12.97]
hodil by se mi dalsi typ borderu: BT_CYCLIC - okopiruje se do nej opacny konec obrazku (jako kdyby se ten obrazek cyklicky opakoval - a la tapeta)

[JP, 12.1.98]
CImsApp::Add_Image(): co ma znamenat

  if (new_doc==NULL) return FALSE; else return TRUE;
  return 0;

[JP, 12.1.98]
GUI dialogy: proc stale nejsou jednotne hlavicky?? Tak by se nemohlo stat, ze "AddFrame()" ma jinou hlavicku v WIN32 a XWIN.

[trinkewitz, 9.2.98]
PCX - load nefunguje pro lichou sirku obrazku (obrazek dodam)

[David Cernoch, 9.2.98]
GIF v gif87.h se redefinuje NULL!!! PORAD!!!

[David Cernoch, 9.2.98]
XBM, XPM index a rindex koliduje s X-ovymi promennymi. A MAX_LINE koliduje mezi dvema vasemi headery!!!

[David Cernoch, 9.2.98]
k promenne is_Abort -- melo by se jasne rici, co ma filtr vratit kdyz je jeho update zrusen -- neco jako ERR_USERABORT. Az se domluvime dam to do kucharky.

[MH, 9.2.98]
WinGUI stale pada pri pridavani filtru ... (o tom ze pri otvirani workspace se nestane vubec nic ani nemluvim :((((( ) dialog::Dosad_List_of_Values(HWND__ * 0x000009e0) line 44 + 3 bytes dialog::Run(char * 0x00000000) line 3413 cMyGraf::InsertItem(TYPE FILTR, tagPOINT {x=39 y=64}, int 0) line 154 + 13 byte CImsWksView::OnGrafPridejFiltr() line 128 + 23 bytes DispatchCmdMsg(CCmdTarget * 0x00fab810 {CImsWksView hWnd=0x000402e0}, unsigned CCmdTarget::OnCmdMsg(unsigned int 32777, int 0, void * 0x00000000, AFX_CMDHANDL CView::OnCmdMsg(unsigned int 32777, int 0, void * 0x00000000, AFX_CMDHANDLERINF CWnd::OnCommand(unsigned int 32777, long 0) line 2107 + 23 bytes CWnd::OnWndMsg(unsigned int 273, unsigned int 32777, long 0, long * 0x0012fd74) CWnd::WindowProc(unsigned int 273, unsigned int 32777, long 0) line 1617 + 30 b AfxCallWndProc(CWnd * 0x00fab810 {CImsWksView hWnd=0x000402e0}, HWND__ * 0x0004 AfxWndProc(HWND__ * 0x000402e0, unsigned int 273, unsigned int 32777, long 0) l AfxWndProcBase(HWND__ * 0x000402e0, unsigned int 273, unsigned int 32777, long DispatchMessageWorker@8 + 135 bytes DispatchMessageA@4 + 11 bytes CWinThread::PumpMessage() line 750 CWinThread::Run() line 434 + 11 bytes CWinApp::Run() line 603 + 8 bytes AfxWinMain(HINSTANCE__ * 0x00400000, HINSTANCE__ * 0x00000000, char * 0x00140e5 WinMain(HINSTANCE__ * 0x00400000, HINSTANCE__ * 0x00000000, char * 0x00140e50, WinMainCRTStartup() line 330 + 57 bytes BaseProcessStart@4 + 64 bytes