Libelektra: merge: getConflicts name too general

Created on 3 Dec 2019  路  7Comments  路  Source: ElektraInitiative/libelektra

The API getConflicts needs to be renamed to begin with elektraMerge. E.g. elektraMergeGetConflicts.

Most helpful comment

Makes sense. I'll get to that as soon as I'm not in bed with fever anymore.

All 7 comments

Is the elektra prefix not "reserved" for internal usage, or is getConflicts only used internally?

The elektra prefix is normal for all the libs except of the core. But also in the core we have quite some public symbols starting with elektra, e.g. for plugin loading (elektraPluginOpen, ...) Adding the elektra prefix to all functions in the core would be nice but it would be at least as much effort as #3115, so way beyond our capacity.

But for the extra libs (like libmerge) we should take care that everything starts with elektra. Thus this issue.

Makes sense. I'll get to that as soon as I'm not in bed with fever anymore.

I mark this issue stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping the issue by writing a message here or create a new issue with the remainder of this issue.
Thank you for your contributions :sparkling_heart:

@Chemin1 this looks like a very simple task, maybe you have some time for this?

I closed this issue now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue.
Thank you for your contributions :sparkling_heart:

Was this page helpful?
0 / 5 - 0 ratings

Related issues

sanssecours picture sanssecours  路  4Comments

markus2330 picture markus2330  路  3Comments

markus2330 picture markus2330  路  3Comments

mpranj picture mpranj  路  3Comments

mpranj picture mpranj  路  3Comments