Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

std:size_t => std::size_t #76

Closed
ramosian-glider opened this issue Aug 31, 2015 · 5 comments
Closed

std:size_t => std::size_t #76

ramosian-glider opened this issue Aug 31, 2015 · 5 comments

Comments

@ramosian-glider
Copy link
Member

Originally reported on Google Code with ID 76

What steps will reproduce the problem?
1. CXX=g++ make -j10 

What is the expected output? What do you see instead?

  COMPILE:   clang_linux/asan-x86_64/x86_64: /home/rurban/Software/llvm-svn157764/projects/compiler-rt/lib/asan/asan_rtl.cc
/home/rurban/Software/llvm-svn157764/projects/compiler-rt/lib/asan/asan_new_delete.cc:38:23:
error: unexpected ':' in nested
      name specifier; did you mean '::'?
void *operator new(std:size_t size) throw(std::bad_alloc) { OPERATOR_NEW_BODY; }
                      ^
                      ::
/home/rurban/Software/llvm-svn157764/projects/compiler-rt/lib/asan/asan_new_delete.cc:39:25:
error: unexpected ':' in nested
      name specifier; did you mean '::'?
void *operator new[](std:size_t size) throw(std::bad_alloc) { OPERATOR_NEW_BODY; }
                        ^
                        ::
/home/rurban/Software/llvm-svn157764/projects/compiler-rt/lib/asan/asan_new_delete.cc:40:23:
error: unexpected ':' in nested
      name specifier; did you mean '::'?
void *operator new(std:size_t size, std::nothrow_t const&) throw()
                      ^
                      ::
/home/rurban/Software/llvm-svn157764/projects/compiler-rt/lib/asan/asan_new_delete.cc:42:25:
error: unexpected ':' in nested
      name specifier; did you mean '::'?
void *operator new[](std:size_t size, std::nothrow_t const&) throw()

What version of the product are you using? On what operating system?
r157764

See attached patch

Reported by reini.urban on 2012-05-31 20:56:43


- _Attachment: [std-size_t.patch](https://storage.googleapis.com/google-code-attachments/address-sanitizer/issue-76/comment-0/std-size_t.patch)_
@ramosian-glider
Copy link
Member Author

Hi, Reini!
The compilation error is weird - I don't see "std:size_t" in the current code or in
your patch.

Reported by samsonov@google.com on 2012-06-01 06:03:03

@ramosian-glider
Copy link
Member Author

I guess this is a fresh regression caused by removal of stddef.h from asan rt headers.

What if we include stddef.h into this particular .cc file? 

Reported by konstantin.s.serebryany on 2012-06-01 06:20:32

@ramosian-glider
Copy link
Member Author

I can see this: 

% make CXX=g++ -j 
asan_new_delete.cc:38:20: error: declaration of ‘operator new’ as non-function
asan_new_delete.cc:38:20: error: ‘size_t’ was not declared in this scope



Reported by konstantin.s.serebryany on 2012-06-01 07:40:52

@ramosian-glider
Copy link
Member Author

Reini, please check LLVM r157807.

Reported by konstantin.s.serebryany on 2012-06-01 07:49:36

  • Status changed: Fixed

@ramosian-glider
Copy link
Member Author

Adding Project:AddressSanitizer as part of GitHub migration.

Reported by ramosian.glider on 2015-07-30 09:12:59

  • Labels added: ProjectAddressSanitizer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant