On Sat, Oct 10, 2009 at 1:55 PM, Brett Wilson <bre...@chromium.org> wrote:
> First, please consider putting it inside your component's directory.
> When everybody adds all their crap to base, it ends up that nobody can
> find anything and the build is more interdependent. If you're not
> actually sharing anything between toplevel projects, it shouldn't be
> in base.

string_util is one of the worst in base:
% wc -l base/*.h | sort -n | tail -10
   500 base/time.h
   514 base/tracked_objects.h
   519 base/message_loop.h
   538 base/file_util.h
   587 base/string_util.h
   588 base/histogram.h
   677 base/task.h
   823 base/logging.h
   907 base/tuple.h

But for the project, it's not too bad:
% git ls-files "*.h" | egrep -v
'third_party|generated|gmock|syscall_support' | xargs wc -l | sort -n
| tail -10
    1150 ipc/ipc_message_macros.h
    1168 chrome/browser/tab_contents/tab_contents.h
    1239 views/view.h
    1270 ipc/ipc_message_utils.h
    1326 chrome/browser/sync/syncable/syncable.h
    1921 chrome/common/render_messages_internal.h
    2116 tools/traceline/traceline/syscall_map.h
    2173 chrome/common/render_messages.h
    3323 o3d/command_buffer/common/cross/cmd_buffer_format.h

--~--~---------~--~----~------------~-------~--~----~
Chromium Developers mailing list: chromium-dev@googlegroups.com 
View archives, change email options, or unsubscribe: 
    http://groups.google.com/group/chromium-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to