MSVC compatability issue with %z format specifier in oni2pcd sample

[strontium5]

In line 68 of pcl/tools/oni2pcd.cpp, the %zu format specifier is not compatible with the MSVC compiler. Changing to %lu resolves the issue for the MSVC compiler.

[rbrusu]

Can you please submit a patch through the pull request system?

[jspricke]

Which MSVC is this? We are using %z all over the place so I doubt we can fix this easily. Some information on this: http://stackoverflow.com/questions/174612/cross-platform-format-string-for-variables-of-type-size-t

[aichim]

Ping, @strontium5 - could you reply to @jspricke 's question please? We have been using %zu for quite a while and never heard complaints. You might be using an exotically old version of MSVC.

[rbrusu]

I'm pretty sure now that %zu is not supported by MSVC.

[bouffa]

I confirm that MSVC doesn't support %zu modifier which is part of C99 (MSVC doesn't support C99). A while ago, I've modified several %zu with %lu in PCL sources..

[rbrusu]

Does %lu have the same behavior with GCC/CLANG? If so, sed to the rescue for changing all PCL lines from one to the other.

[strontium5]

Sorry for my late reply, but I was using the MSVC in visual studio 2010

On Tue, Jul 9, 2013 at 11:52 AM, aichim notifications@github.com wrote:

Ping, @strontium5 https://github.com/strontium5 - could you reply to
@jspricke https://github.com/jspricke 's question please? We have been
using %zu for quite a while and never heard complaints. You might be using
an exotically old version of MSVC.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/146#issuecomment-20684041
.

[rbrusu]

Is there an action item for this issue? If %zu is not supported by MSVC, then we need a patch/pull request in order to solve it.

[rbrusu]

Any follow up to this issue?

[rbrusu]

@bouffa is that an older version of MSVC? I believe newer ones handle it just fine, otherwise we would have heard more bug reports.

[rbrusu]

Closed due to lack of activity.

[yangyanli]

Well, I confirm that %zu doesn't work on VS2012. And this is very annoying...

Can someone confirm whether %lu works for other platforms?