View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
5741Composrcorepublic2024-04-28 18:572024-07-22 21:09
ReporterPDStig Assigned ToGuest  
PrioritynormalSeverityfeature 
Status newResolutionopen 
Summary5741: Consider is_readable instead of file_exists where applicable
Descriptionis_readable may be a better option performance-wise than file_exists if we are checking for a file and also planning to read it.

We may need to make our own cms_is_readable like we have cms_is_writable.
TagsRoadmap: Over the horizon, Type: Performance
Attach Tags
Time estimation (hours)
Sponsorship open

Sponsor

Date Added Member Amount Sponsored

Activities

Chris Graham

2024-07-22 21:09

administrator   ~8874

This is true, but it's also a micro-optimization. Should only be done in critical path code. Trying to force it everywhere would lead to counter-intuitive code (the naming is a bit less clear), and yet another coding standard to enforce (we probably already have too many).

Add Note

View Status
Note
Upload Files
Maximum size: 32,768 KiB

Attach files by dragging & dropping, selecting or pasting them.
You are not logged in You are not logged in. This means you will not get any e-mail notifications. And if you reply, we will not know for sure you are the original poster of the issue.

Issue History

Date Modified Username Field Change
2024-04-28 18:57 PDStig New Issue
2024-04-28 19:17 PDStig Tag Attached: Roadmap: Over the horizon
2024-07-22 21:08 Chris Graham Tag Attached: Type: Performance
2024-07-22 21:09 Chris Graham Note Added: 0008874