Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix stack overflow in FilterExpression.ValidForProperties #3946

Merged
merged 16 commits into from Aug 24, 2022
Merged

Fix stack overflow in FilterExpression.ValidForProperties #3946

merged 16 commits into from Aug 24, 2022

Commits on Aug 19, 2022

  1. replaced recursion by iterative code in ValidForProperties function

    @engyebrahim
    engyebrahim committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    4408b45 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into pr/3905

    @engyebrahim
    engyebrahim committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    f39bfcd View commit details
    Browse the repository at this point in the history

Commits on Aug 22, 2022

  1. validate the filter condition values

    @engyebrahim
    engyebrahim committed Aug 22, 2022
    Configuration menu
    Copy the full SHA
    f81cab3 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into pr/3905

    @engyebrahim
    engyebrahim committed Aug 22, 2022
    Configuration menu
    Copy the full SHA
    681435d View commit details
    Browse the repository at this point in the history

  3. made Evaluate function not recursive

    @engyebrahim
    engyebrahim committed Aug 22, 2022
    Configuration menu
    Copy the full SHA
    9951854 View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2022

  1. applied formate comments and add debuge assert

    @engyebrahim
    engyebrahim committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    6e520b1 View commit details
    Browse the repository at this point in the history

  2. seperate the stacks code using lambda

    @engyebrahim
    engyebrahim committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    ceea457 View commit details
    Browse the repository at this point in the history

  3. fixed formate

    @engyebrahim
    engyebrahim committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    f7859c1 View commit details
    Browse the repository at this point in the history

  4. added a unit test

    @engyebrahim
    engyebrahim committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    7833e91 View commit details
    Browse the repository at this point in the history

  5. add comments

    @engyebrahim
    engyebrahim committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    d5b170b View commit details
    Browse the repository at this point in the history

  6. fixed the unit test

    @engyebrahim
    engyebrahim committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    6894f7f View commit details
    Browse the repository at this point in the history

  7. changed the unit test data

    @engyebrahim
    engyebrahim committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    3ec211b View commit details
    Browse the repository at this point in the history

  8. removed the unneeded null check

    @engyebrahim
    engyebrahim committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    2ba6c5b View commit details
    Browse the repository at this point in the history

Commits on Aug 24, 2022

  1. updated a comment

    @engyebrahim
    engyebrahim committed Aug 24, 2022
    Configuration menu
    Copy the full SHA
    0187823 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into pr/3905

    @engyebrahim
    engyebrahim committed Aug 24, 2022
    Configuration menu
    Copy the full SHA
    95f6324 View commit details
    Browse the repository at this point in the history

  3. updated comments

    @engyebrahim
    engyebrahim committed Aug 24, 2022
    Configuration menu
    Copy the full SHA
    5cf68e7 View commit details
    Browse the repository at this point in the history