[Bug 560240] Review Request: libxls - Library for parsing Excel (XLS) files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=560240

--- Comment #8 from Chen Lei <supercyper@xxxxxxx> 2010-02-01 10:55:17 EST ---
By the way, most warnings are useful only in debug mode. The global variable
xls_debug is 0 by default.
xls.c:123: warning: format '%ld' expects type 'long int', but argument 2 has
type 'DWORD'
xls.c:151: warning: format '%ld' expects type 'long int', but argument 2 has
type 'DWORD'
xls.c:176: warning: format '%ld' expects type 'long int', but argument 2 has
type 'DWORD'
xls.c:224: warning: format '%li' expects type 'long int', but argument 3 has
type 'DWORD'
xls.c:252: warning: format '% 4ld' expects type 'long int', but argument 2 has 

  122     if (xls_debug) {
  123      printf("xls_appendSST %ld\n", size);
  124     }

  150   if (xls_debug) {
  151          printf("ln=%ld\n", ln);
  152   }

xlstool.c:447: warning: zero-length gnu_printf format string
  446     case 0x201:  //BLANK
  447         sprintf(ret,"");
  448         break;
  449     case 0x0BE:  //MULBLANK
  450         sprintf(ret,"");
  451         break;
> xlstool.c:456: warning: format '%.*s' expects type 'char *', but argument 4 has
> type 'WORD *'
  452     case 0x0204: //LABEL (xlslib generates these)
  453   lPtr = (WORD *)cell->l;
  454   len = *lPtr++;
  455   if(pWB->is5ver) {
  456    sprintf(ret,"%.*s", len, lPtr);
  457    //printf("Found BIFF5 string of len=%d \"%s\"\n", len, ret);
  458   } else
  459   if((*(char *)lPtr & 0x01) == 0) {
  460    sprintf(ret,"%.*s", len, (char *)lPtr + 1); // 1 is the format
  461    //printf("Found BIFF8/ASCII string of len=%d \"%s\"\n", len, ret);

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]