Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[RFC] Change Ada.Containers.Count_Type to a subtype of Natural #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

setton
Copy link
Member

@setton setton commented Sep 11, 2019

@raph-amiard
Copy link
Member

Hi Nico!

I think your RFC proposes a practical fix to the occurrence of a general issue we have in Ada wrt. operator visibility.

I however wonder if we shouldn't try and address the forest behind the tree here.

Stil in favor of this anyhow. Will put that up for discussion!

@raph-amiard raph-amiard self-assigned this Jan 12, 2021
@raph-amiard
Copy link
Member

raph-amiard commented Nov 24, 2022

A problem that I see today with that change, is that it will be very hard to make it affect only one version of Ada, since we can't say:

% if Ada_Version = 202X then
   type Count_Type is ...
% else
   type Count_Type is ...
% endif;

So while I like that proposal, I think it's unimplementable right now. Curious to know what other people think.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants