icu icon indicating copy to clipboard operation
icu copied to clipboard

ICU-22068 Cleanup inconsistent annotations between declarations and d…

Open cyndyishida opened this issue 3 years ago • 8 comments

…efinitions

This cleans up inconsistent annotations between declared APIs in headers vs defined implementations in cpp's. This better ensures the API's referenceable in headers represent what is exposed and defined in the ultimate binary library's symbol table.

  • [x] Required: Issue filed: https://unicode-org.atlassian.net/browse/ICU-22068
  • [x] Required: The PR title must be prefixed with a JIRA Issue number.
  • [x] Required: The PR description must include the link to the Jira Issue, for example by completing the URL in the first checklist item
  • [x] Required: Each commit message must be prefixed with a JIRA Issue number.
  • [x] Issue accepted (done by Technical Committee after discussion)
  • [x] Tests included, if applicable (current regression tests should suffice)
  • [ ] API docs and/or User Guide docs changed or added, if applicable

cyndyishida avatar Jun 22 '22 16:06 cyndyishida

Notice: the branch changed across the force-push!

  • icu4c/source/common/unicode/utf_old.h is different
  • icu4c/source/stubdata/BUILD.bazel is now changed in the branch

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

Notice: the branch changed across the force-push!

  • icu4c/source/common/unicode/utf_old.h is different
  • icu4c/source/common/utf_impl.cpp is now changed in the branch

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

Waiting on @markusicu feedback before moving forward.

cyndyishida avatar Jun 30 '22 17:06 cyndyishida

@markusicu You probably want to have a look at this one.

pedberg-icu avatar Jun 30 '22 18:06 pedberg-icu

Is it possible to add "TAPI" to a build bot, so that these kinds of things don't backslide?

markusicu avatar Jul 05 '22 23:07 markusicu

Is it possible to add "TAPI" to a build bot, so that these kinds of things don't backslide?

I would love to add TAPI to ICU's CI! Unfortunately the tool itself only works out of the box with Xcode. The tool hasn't been completely upstreamed as apart of LLVM (slowly working on this). I'd love to touch base again after we've finished open sourcing TAPI as apart of LLVM's toolchain to get this supported.

cyndyishida avatar Jul 06 '22 01:07 cyndyishida

Also, there is a merge conflict. Please rebase & resolve.

markusicu avatar Jul 07 '22 22:07 markusicu

Notice: the branch changed across the force-push!

  • icu4c/source/i18n/collationbuilder.cpp is different
  • icu4c/source/i18n/collationdatabuilder.cpp is different
  • icu4c/source/stubdata/stubdata.h is different

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot