Skip to content

Viewer limit setup for "Executables" view.#1216

Merged
jonahgraham merged 1 commit into
eclipse-cdt:mainfrom
raghucssit:viewer_limit_setup_exe_view
Jun 24, 2025
Merged

Viewer limit setup for "Executables" view.#1216
jonahgraham merged 1 commit into
eclipse-cdt:mainfrom
raghucssit:viewer_limit_setup_exe_view

Conversation

@raghucssit

Copy link
Copy Markdown
Contributor

"Executables" view makes eclipse unresponsive if it has thousands of entries. We can limit the number of items shown by applying viewer limit.

see #1215

@ruspl-afed

Copy link
Copy Markdown
Member

@raghucssit could you please update your PR with rebase, the problem with Code Cleanliness Checks should disappear

@ruspl-afed ruspl-afed left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

"Executables" view makes eclipse unresponsive if it has thousands of
entries. We can limit the number of items shown by applying viewer
limit.

see eclipse-cdt#1215
@raghucssit raghucssit force-pushed the viewer_limit_setup_exe_view branch from 5f52b5f to 3793cfb Compare June 24, 2025 07:54
@raghucssit

Copy link
Copy Markdown
Contributor Author

@jonahgraham This change is similar to 'C/C++ Projects' view..
#1181
Please check and merge it.

@jonahgraham jonahgraham merged commit 74fb09c into eclipse-cdt:main Jun 24, 2025
4 checks passed
@jonahgraham

Copy link
Copy Markdown
Member

Thanks @raghucssit for this fix.

@jonahgraham jonahgraham added this to the 12.2.0 milestone Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants