[Bug 10661] regedit imports wrong binary data from .reg files ( affects FAR Manager) fix proposed

wine-bugs at winehq.org wine-bugs at winehq.org
Mon Dec 3 23:41:07 CST 2007


http://bugs.winehq.org/show_bug.cgi?id=10661





--- Comment #2 from Dmitry Timoshkov <dmitry at codeweavers.com>  2007-12-03 23:41:07 ---
(In reply to comment #0)
> http://source.winehq.org/source/programs/regedit/regproc.c reads (line 570):
> 570                 if ((c = fgetc (in)) == EOF || c != ' ' ||
> 571                         (c = fgetc (in)) == EOF || c != ' ')
> 572                     fprintf(stderr,"%s: ERROR - invalid continuation.\n",
> 573                             getAppName());
> It should be:
> do { c=fgetc(in); } while(c==' '||c=='\t');
> if(c==EOF){fprintf(stderr,"%s: ERROR - invalid continuation.\n",
> getAppName());}
> ungetc(c,in);

It should be pretty easy to rewrite the fix to avoid the ungetc() at the end,
and use the same whitespace formatting rules as the existing code does.


-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.



More information about the wine-bugs mailing list