Consolidate redundant path/file public APIs

Bug #1180220 reported by Chris Hillery
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Zorba
Fix Committed
High
Paul J. Lucas

Bug Description

We have include/zorba/file.h and include/zorba/util/file.h, which both define classes that represent "a file". There's also include/zorba/util/path.h which defines filesystem_path, a base class for the latter file class.

Neither of these classes are actually used in our public APIs much, so it's not clear we need either let alone both. We need an internal class for files (we may have another one). Could we move these out of the API entirely? If not, we need to reduce to one common class that makes sense.

Related branches

Revision history for this message
Chris Hillery (ceejatec) wrote :

Assigning to Paul for now as he has good ideas about at least some of these APIs internally, and can probably at least propose what should be done.

Changed in zorba:
milestone: none → 3.0
importance: Undecided → High
status: New → Confirmed
assignee: nobody → Paul J. Lucas (paul-lucas)
Chris Hillery (ceejatec)
tags: added: incompatible-change
Revision history for this message
Paul J. Lucas (paul-lucas) wrote :

If they're really not used in the public API, can I simply remove them from the public API?

Revision history for this message
Paul J. Lucas (paul-lucas) wrote :

Uhm, ignore my previous comment. You asked the very same question in the description.

Changed in zorba:
status: Confirmed → In Progress
Changed in zorba:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.