Comment 5 for bug 1061157

I do not know, although I would guess so. If you really want to be sure, one can modify line 366 from:
   units = value_unquoted2.substr(le-2, 2);
into something like (not tested):
   units = value_unquoted2.substr(le-2, 2).lowercase();

Note that your variant of the length fix of comment 81 of the omibus patch (Bug #988601) still leaves a memory leak. In general it is best to use c++ string classes (Glib::ustring for unicode strings or alternatively std::string for strings where you are sure you never need unicode characters) which are more flexible and prevent memory leaks associated to string allocation and deallocation.