[GH-ISSUE #81] Save a page by removing WhereToSlideTable #251

New Issue

2026-05-20T22:35:00+02:00

Pecus commented

2026-05-20 22:35:00 +02:00

Originally created by @pkali on GitHub (May 28, 2022).
Original GitHub issue: https://github.com/pkali/scorch_src/issues/81

WhereToSlideTable is almost empty and can be replaced by a few "if's". Do this only when memory is really tight.

Originally created by @pkali on GitHub (May 28, 2022). Original GitHub issue: https://github.com/pkali/scorch_src/issues/81 WhereToSlideTable is almost empty and can be replaced by a few "if's". Do this only when memory is really tight.

Pecus added the optimization label 2026-05-20 22:35:00 +02:00

Pecus closed this issue

2026-05-20 22:35:00 +02:00

Pecus commented

2026-05-20 22:35:00 +02:00

@Pecusx commented on GitHub (May 28, 2022):

After a quick analysis...
In my opinion a "few" "if's" will be longer than the table. Simply - there must be many more "if's".
Table is more elegant and simple to correct/change.

@Pecusx commented on GitHub (May 28, 2022): After a quick analysis... In my opinion a "few" "if's" will be longer than the table. Simply - there must be many more "if's". Table is more elegant and simple to correct/change.

Pecus commented

2026-05-20 22:35:01 +02:00

@pkali commented on GitHub (Jun 5, 2022):

ok, I agree. when we are really tight I might revisit.

@pkali commented on GitHub (Jun 5, 2022): ok, I agree. when we are really tight I might revisit.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/scorch_src#251