You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@poi.apache.org by bu...@apache.org on 2003/12/28 15:05:38 UTC

DO NOT REPLY [Bug 25781] New: - SprmUtils.addSprm gets wrong instruction type

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25781>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25781

SprmUtils.addSprm gets wrong instruction type

           Summary: SprmUtils.addSprm gets wrong instruction type
           Product: POI
           Version: 2.0-dev
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: HDF
        AssignedTo: poi-dev@jakarta.apache.org
        ReportedBy: simon.niederberger@want.ch


>From how I understand the code in SprmUtils.addSprm, the (instruction) type is 
derived from the highest three bits of the passed instruction. However, the 
bits are not right-shifted after the bit-and.

    int type = instruction & 0xe000;
    
    // SIN, ADD BEG
    // I guess the code is meant to read the top 3 bits and pack them in an 
int 0-7
    type >>= 13;
    // SIN, ADD END

(If this isn't bug report doesn't meet your standards, my apologies - just 
submitting my first one ...)

Cheers
Simon

---------------------------------------------------------------------
To unsubscribe, e-mail: poi-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: poi-dev-help@jakarta.apache.org