Merge pull request #83 from ardovm/master

Jan Breuer

2016-10-17 15671f57ea852bc1df5c0d2a358b714eeb097074

 libscpi/src/expression.c

@@ -193,7 +193,7 @@
    size_t i = 0;
    while (scpiLex_DecimalNumericProgramData(state, &param)) {
        if (i < length) {
            SCPI_ParamToInt(context, &param, &values[i]);
            SCPI_ParamToInt32(context, &param, &values[i]);
        }

        if (scpiLex_SpecificCharacter(state, &param, '!')) {
@@ -207,7 +207,7 @@
    if (i == 0) {
        return SCPI_EXPR_NO_MORE;
    } else {
        // there was at least one number followed by !, but after ! was not another number
        /* there was at least one number followed by !, but after ! was not another number */
        return SCPI_EXPR_ERROR;
    }
}
@@ -245,6 +245,8 @@
            *dimensions = fromDimensions;
            return SCPI_EXPR_OK;
        }
    } else if (err == SCPI_EXPR_NO_MORE) {
        err = SCPI_EXPR_ERROR;
    }

    return err;
@@ -281,7 +283,7 @@
    lex.pos = lex.buffer;
    lex.len = param->len - 2;

    // detect channel list expression
    /* detect channel list expression */
    if (!scpiLex_SpecificCharacter(&lex, &token, '@')) {
        SCPI_ErrorPush(context, SCPI_ERROR_EXPRESSION_PARSING_ERROR);
        return SCPI_EXPR_ERROR;
@@ -303,5 +305,11 @@
    if (res == SCPI_EXPR_ERROR) {
        SCPI_ErrorPush(context, SCPI_ERROR_EXPRESSION_PARSING_ERROR);
    }
    if (res == SCPI_EXPR_NO_MORE) {
        if (!scpiLex_IsEos(&lex)) {
            res = SCPI_EXPR_ERROR;
            SCPI_ErrorPush(context, SCPI_ERROR_EXPRESSION_PARSING_ERROR);
        }
    }
    return res;
}