Actions
Forum Thread Details
  • Replies 3 replies
  • Subscribers 177 subscribers
  • Views 343 views
  • Users 0 members are here
Related

Bug in ULP strtol?

autodeskguest

ULP -      output("c:
eagle_debug.txt", "wt") {

          printf("Starting a run of ULP %s - ", argv[0]);

          int t = time();

          printf("It is now %02d:%02d:%02d\n",

          t2hour(t), t2minute(t), t2second(t));

}

 

     int n = 16;

     for (int i = 0; i <= n; i++) {

 

     string outputs;

     int outputu;

 

     sprintf(outputs,"%2.2u",i);

 

                output("c:
eagle_debug.txt", "at") {

                           printf("i as string = %s - ", outputs);

                           outputu = strtol(outputs);

                           printf("i as strtol = %u\n", outputu);

                           printf("\n");

                    }          

 

 

     }

 

 

 

Output

 

Starting a run of ULP C:/Program

Files/EAGLE-UserFiles/ulp/test_strtol.ulp - It is now 09:26:16

i as string = 00 - i as strtol = 0

 

i as string = 01 - i as strtol = 1

 

i as string = 02 - i as strtol = 2

 

i as string = 03 - i as strtol = 3

 

i as string = 04 - i as strtol = 4

 

i as string = 05 - i as strtol = 5

 

i as string = 06 - i as strtol = 6

 

i as string = 07 - i as strtol = 7

 

i as string = 08 - i as strtol = 0

 

i as string = 09 - i as strtol = 0

 

i as string = 10 - i as strtol = 10

 

i as string = 11 - i as strtol = 11

 

i as string = 12 - i as strtol = 12

 

i as string = 13 - i as strtol = 13

 

i as string = 14 - i as strtol = 14

 

i as string = 15 - i as strtol = 15

 

i as string = 16 - i as strtol = 16

 

 

Glenn

 

Parents

  • Hello,

     

    Glenn Jones wrote:

    i as string = 07 - i as strtol = 7

     

    i as string = 08 - i as strtol = 0

     

    i as string = 09 - i as strtol = 0

     

    i as string = 10 - i as strtol = 10

     

    This is not a bug. Numbers with leading zeros are interpreted as octal

    numbers and 08 and 09 are not valid octal numbers. Since strtol is not

    able to convert the string to a number it returns 0.

    Search the help for 'strtol()' and 'integer constants' for details.

     

    To avoid leading zeros in the output and have the numbers padded with

    spaces you could use

       sprintf(outputs,"% 2u",i);

    instead. The added space acts as padding character. I stripped the .2

    from the format specifier because the precision specifier only makes

    sense for real numbers.

     

     

    Regards,

    Christoph

     

Reply

  • Hello,

     

    Glenn Jones wrote:

    i as string = 07 - i as strtol = 7

     

    i as string = 08 - i as strtol = 0

     

    i as string = 09 - i as strtol = 0

     

    i as string = 10 - i as strtol = 10

     

    This is not a bug. Numbers with leading zeros are interpreted as octal

    numbers and 08 and 09 are not valid octal numbers. Since strtol is not

    able to convert the string to a number it returns 0.

    Search the help for 'strtol()' and 'integer constants' for details.

     

    To avoid leading zeros in the output and have the numbers padded with

    spaces you could use

       sprintf(outputs,"% 2u",i);

    instead. The added space acts as padding character. I stripped the .2

    from the format specifier because the precision specifier only makes

    sense for real numbers.

     

     

    Regards,

    Christoph

     

Children
  • in reply to autodeskguest

    Christoph Wurzinger wrote:

    Hello,

     

    Glenn Jones wrote:

    i as string = 07 - i as strtol = 7

     

    i as string = 08 - i as strtol = 0

     

    i as string = 09 - i as strtol = 0

     

    i as string = 10 - i as strtol = 10

     

    This is not a bug. Numbers with leading zeros are interpreted as octal

    numbers and 08 and 09 are not valid octal numbers. Since strtol is not

    able to convert the string to a number it returns 0.

    Search the help for 'strtol()' and 'integer constants' for details.

     

    To avoid leading zeros in the output and have the numbers padded with

    spaces you could use

      sprintf(outputs,"% 2u",i);

    instead. The added space acts as padding character. I stripped the .2

    from the format specifier because the precision specifier only makes

    sense for real numbers.

     

     

    Regards,

    Christoph

     

    Hi Christoph,

     

    I was parsing input that has leading zero's and converting to integer so

    I could increase the value by one and make a string out of it again.

     

    So I will strip off leading zeros, increment and then print back to a

    zero leading string. The leading zero is not really required but i would

    rather not modify the data as supplied more then necessary.

     

    And you are correct what I wanted was sprintf(outputs,"%02u",i);

     

    Thanks for the explanation.

     

    Glenn