The optimization they provide is not worth it IMO.
Fixes bugs #10898, #10899, #10940.
return false;
// read value
- bool legalChar = true;
while (ifs && !isSpace(ch) &&
- delimChars.find(ch) == docstring::npos &&
- (legalChar = (illegalChars.find(ch) == docstring::npos))
- )
+ delimChars.find(ch) == docstring::npos &&
+ illegalChars.find(ch) == docstring::npos)
{
if (chCase == makeLowerCase)
val += lowercase(ch);
ifs.get(ch);
}
- if (!legalChar) {
+ if (illegalChars.find(ch) != docstring::npos) {
ifs.putback(ch);
return false;
}
else
name = master.B_(tclass.floats().getType(type).name());
docstring counter = from_utf8(type);
- if ((is_subfloat_ = cnts.isSubfloat())) {
+ is_subfloat_ = cnts.isSubfloat();
+ if (is_subfloat_) {
// only standard captions allowed in subfloats
type_ = "Standard";
counter = "sub-" + from_utf8(type);
{
vector<string> options = split_options(opts);
add_package(name, options);
- char const * const * where = 0;
if (is_known(name, known_xetex_packages)) {
xetex = true;
else if (name == "subfig")
; // ignore this FIXME: Use the package separator mechanism instead
- else if ((where = is_known(name, known_languages)))
+ else if (char const * const * where = is_known(name, known_languages))
h_language = known_coded_languages[where - known_languages];
else if (name == "natbib") {