Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Optimization of App#initialize

#136

This is potentially a slow initialization since a config like {:root, :relative_paths, :absolute_paths} will reassign paths 3 times (see Root#assign_paths)


    # Creates a new App with the given configuration.  
    def initialize(config={}, logger=DEFAULT_LOGGER)
      super()
      
      @state = State::READY
      @queue = Support::ExecutableQueue.new
      @aggregator = Support::Aggregator.new
      @dependencies = Support::Dependencies.new
      
      reconfigure(config)
      self.logger = logger
    end

The assigned configurations should be removed from config and reassigned as one... the optimization is probably best done in Root itself, within reconfigure.

Reported by bahuvrihi · January 2nd, 2009 @ 11:07 AM

State: resolved
Milestone: none
Assigned to: bahuvrihi bahuvrihi

Activity

  1. bahuvrihi
    bahuvrihi
    • Assigned user set to bahuvrihi
    • State changed from new to resolved

    March 5th, 2009 @ 03:42 PM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.