[Bug-xnee] [bugs #8324] --retype-file option for cnee does not set mode

[Henrik Sandklef]

This mail is an automated notification from the bugs tracker
 of the project: Xnee.

/**************************************************************************/
[bugs #8324] Latest Modifications:

Changes by: 
                Henrik Sandklef <address@hidden>
'Date: 
                Sun 04/18/04 at 20:49 (Europe/Stockholm)

            What     | Removed                   | Added
---------------------------------------------------------------------------
          Resolution | None                      | Fixed
         Assigned to | None                      | hesa
              Status | Open                      | Closed







/**************************************************************************/
[bugs #8324] Full Item Snapshot:

URL: <http://savannah.nongnu.org/bugs/?func=detailitem&item_id=8324>
Project: Xnee
Submitted by: Joseph Miele
On: Sat 03/27/04 at 08:07

Category:  cnee
Severity:  5 - Average
Item Group:  Error report
Resolution:  Fixed
Assigned to:  hesa
Status:  Closed


Summary:  --retype-file option for cnee does not set mode

Original Submission:  Problem: Option "--retype-file" fails to set the mode, 
causing the option to fail.

Version affected: CVS

Steps to reproduce the bug:
1. Create text file /tmp/test.xnee. The contents of the file do not matter.
2. Enter the following command:
cnee --verbose --retype-file /tmp/test.xnee

One of the lines of the verbose output will be: "No mode specified... leaving".

Cause: Missing call to xnee_set_retyper in file parse.c (directory cnee/src)

Here is the corrected code segment for parse.c:

      else if(xnee_check(argv[i], "--retype-file", "-rtf" ))
        {
          xnee_set_retyper(xd); /* this is the fix */
          if (++i >= argc)
            {
              xnee_usage(stderr);
              xnee_close_down(xd);
              exit(XNEE_WRONG_PARAMS);
            }
          if ( xnee_set_rt_name (xd, argv[i]) != XNEE_OK)
            {
              xnee_print_error ("Unable to open retype filen");
              xnee_verbose ((xd, "Could not open retype filen"));
              xnee_verbose ((xd, "... leavingn"));
              xnee_close_down(xd);
              exit(XNEE_WRONG_PARAMS);
            }
          continue;

        }

Notice the new line designated as the fix. Adding this line solved the problem 
for me. Running cnee as indicated in step 2 works correctly with this change in 
place.

Please review this code change. If you find the fix acceptable, please update 
CVS.

Thank you for your time.

Follow-up Comments
------------------


-------------------------------------------------------
Date: Tue 03/30/04 at 19:33         By: hesa
Yes, this is the way to do it. Will include it.....














For detailed info, follow this link:
<http://savannah.nongnu.org/bugs/?func=detailitem&item_id=8324>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/