celix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [celix] pnoltes commented on issue #53: CELIX-468
Date Thu, 19 Sep 2019 18:20:55 GMT
pnoltes commented on issue #53: CELIX-468
URL: https://github.com/apache/celix/pull/53#issuecomment-533250446
 
 
   > Renamed constants to celix_constants (including ifdef)
   > Changed all includes to use new file name
   > @pnoltes the issue mentions that some variables should be prefixed with CELIX, however
all variables are already prefixed with either CELIX or OSGI, what would be the best way to
proceed?
   
   I though not all constants had a CELIX_ or OSGI_ prefix. I see that they do, than this
change is enough :)

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message