Ignore:
Timestamp:
2017-12-12T16:47:34+13:00 (6 years ago)
Author:
ak19
Message:

Marking all the uses of sysread() with a comment saying they're a candidate to use FileUtils::readUTF8File() in future, if thinking about each case beforehand has confirmed that the contents will indeed be UTF8.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • main/trunk/greenstone2/bin/script/replace_srcdoc_with_html.pl

    r24475 r32096  
    187187    }
    188188
     189    # The following file reading section is a candidate to use FileUtils::readUTF8File()
     190    # in place of calling sysread() directly. But only if we can reason we'd be working with UTF8
    189191    # Now we know we have no file name collisions. We 'move' the html file by copying its
    190192    # contents over and ensuring that these contents are utf8. If we don't do this, PDFs
Note: See TracChangeset for help on using the changeset viewer.