Ignore:
Timestamp:
06/03/10 17:51:06 (10 years ago)
Author:
curtis
Message:

Clarify and expound on some TODOs.

Location:
trunk/components/bio-formats/src/loci/formats
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/components/bio-formats/src/loci/formats/ChannelFiller.java

    r6448 r6473  
    158158    if (passthrough()) return reader.openBytes(no, buf, x, y, w, h); 
    159159 
     160    // TODO: The pixel type should change to match the available color table. 
     161    // That is, even if the indices are uint8, if the color table is 16-bit, 
     162    // The pixel type should change to uint16. Similarly, if the indices are 
     163    // uint16 but we are filling with an 8-bit color table, the pixel type 
     164    // should change to uint8. 
     165 
     166    // TODO: This logic below is opaque and could use some comments. 
     167 
    160168    byte[] pix = reader.openBytes(no, x, y, w, h); 
    161169    if (getPixelType() == FormatTools.UINT8) { 
  • trunk/components/bio-formats/src/loci/formats/CoreMetadata.java

    r6445 r6473  
    3737  // -- Fields -- 
    3838 
    39   // TODO 
    40   // 
    41   // We may want to consider refactoring the FormatReader getter methods 
     39  // TODO: We may want to consider refactoring the FormatReader getter methods 
    4240  // that populate missing CoreMetadata fields on the fly 
    4341  // (getChannelDimLengths, getChannelDimTypes, getThumbSizeX, getThumbSizeY) 
Note: See TracChangeset for help on using the changeset viewer.