search

AppImageCommunity / libappimage

Implements functionality for dealing with AppImage files
https://appimage.org
Other
46 stars 27 forks source link

Use templating and type traits instead of abstract base classes at AppImage Traversal #52

Open azubieta opened 5 years ago

azubieta commented 5 years ago

We should try to avoid any form of (slow) virtual binding for such low-level stuff by using templating and type traits instead of abstract base classes and implementations thereof.

azubieta commented 5 years ago

This could be useful to address the issue https://accu.org/index.php/journals/479

probonopd commented 5 years ago

Omg, I have not the slightest clue what you are talking about. Virtual binding? Type traits? Abstract base classes? These are the situations in which I wish we had stayed with C... because I realize I can't be helpful here in any way anymore and won't know what to do if stuff breaks. Without getting a diploma in C++ philosophy, that is.

azubieta commented 5 years ago

getting a diploma in C++ philosophy

:+1: :laughing:

@probonopd don't panic, we are here to help. Anyway, this is an issue for those times when we have absolutely nothing else to do. It will provide a slight performance improvement but it's also very dark stuff.