Cleaning up or removing base/format_macros.h

[Lei Zhang]

This header dates back from around 2009 when Chromium's build
environment was very different. Does anyone have concerns about
getting rid of some of the #defines there? In particular:

1) Use %zu instead of PRIuS everywhere.
2) WidePRId64 and friends are not used anywhere.

I didn't look at the Mac-specific #defines, but if those can go away
too, then there would be nothing left in the header.

[Avi Drissman]

I would be in favor of this cleanup.

--
You received this message because you are subscribed to the Google Groups "cxx" group.
To unsubscribe from this group and stop receiving emails from it, send an email to cxx+uns...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CACSHbcR_gnJhWyJ7Pt37%3DRGPYP%3D7j-U8snFmB1_eV6g641xA%3Dw%40mail.gmail.com.

[Daniel Cheng]

+1 if we don't need these format macros, it makes sense to remove this header.

Daniel

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CACWgwAZYdOaGudrm9RPtYNFa0JnaaSyTyzgOmbYZNGkEPHFmsw%40mail.gmail.com.

[Sylvain Defresne]

Please don't remove PRIuNS, PRIdNS and PRIxNS as they are used on iOS to support both 32-bit and 64-bit builds (32-bit builds are still used from cronet iirc). Thus the header cannot go away yet (at least not until cronet removes support for 32-bit build).

-- Sulvain

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CAF3XrKrs073i6XvdTKjKc4KZjfrgwtD9fayB2mepthJ1pR67mw%40mail.gmail.com.

[dan...@chromium.org]

Would it make sense to move the header to cronet?

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CAJauWCHbHUJbVxtQ%3Dsit%3DNMYJq9mb5g%2BSTEknTDjQgsFwhffNQ%40mail.gmail.com.

[Sylvain Defresne]

It looks like the macro is used in ios/chrome, but we could convert all those uses to the 64-bit version of the macro, and then move the header to components/cronet/ios. So, yes after converting ios/chrome.

-- Sylvain

[Mark Mentovai]

I suggest moving the Apple-specific macros to live in an Apple-only file in base/mac. It’s true that their only utility at present is to support cronet, but cronet depends on base and those macros are lightly used in base, in tests at least.

In general, for formatting NSInteger and NSUInteger, I do prefer the macros to other alternatives.

Mark

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/cxx/CAJauWCEX46yZE%3DbkS_BktmBDJ%3DivbWk9dcXHVoef9wJ9TXVh6Q%40mail.gmail.com.

[Lei Zhang]

If there are no objections, I'll remove the non-Apple macros. When
only Apple-specific macros remain, we can move the file into base/mac.