Skip to content
Snippets Groups Projects
  1. Nov 10, 2015
  2. Oct 21, 2014
  3. Oct 25, 2013
  4. Aug 20, 2013
  5. Mar 29, 2013
  6. Mar 13, 2013
    • Don Kjer's avatar
      Large changes to the LLCurl::Responder API, as well as pulling in some changes... · f9454152
      Don Kjer authored
      Large changes to the LLCurl::Responder API, as well as pulling in some changes to common libraries from the server codebase:
      
      * Additional error checking in http handlers.
      * Uniform log spam for http errors.
      * Switch to using constants for http heads and status codes.
      * Fixed bugs in incorrectly checking if parsing LLSD xml resulted in an error.
      * Reduced spam regarding LLSD parsing errors in the default completedRaw http handler.  It should not longer be necessary to short-circuit completedRaw to avoid spam.
      * Ported over a few bug fixes from the server code.
      * Switch mode http status codes to use S32 instead of U32.
      * Ported LLSD::asStringRef from server code; avoids copying strings all over the place.
      * Ported server change to LLSD::asBinary; this always returns a reference now instead of copying the entire binary blob.
      * Ported server pretty notation format (and pretty binary format) to llsd serialization.
      
      * The new LLCurl::Responder API no longer has two error handlers to choose from.  Overriding the following methods have been deprecated:
      ** error - use httpFailure
      ** errorWithContent - use httpFailure
      ** result - use httpSuccess
      ** completed - use httpCompleted
      ** completedHeader - no longer necessary; call getResponseHeaders() from a completion method to obtain these headers.
      
      * In order to 'catch' a completed http request, override one of these methods:
      ** httpSuccess - Called for any 2xx status code.
      ** httpFailure - Called for any non-2xx status code.
      ** httpComplete - Called for all status codes.  Default implementation is to call either httpSuccess or httpFailure.
      * It is recommended to keep these methods protected/private in order to avoid triggering of these methods without using a 'push' method (see below).
      
      * Uniform error handling should followed whenever possible by calling a variant of this during httpFailure:
      ** llwarns << dumpResponse() << llendl;
      * Be sure to include LOG_CLASS(your_class_name) in your class in order for the log entry to give more context.
      
      * In order to 'push' a result into the responder, you should no longer call error, errorWithContent, result, or completed.
      * Nor should you directly call httpSuccess/Failure/Completed (unless passing a message up to a parent class).
      * Instead, you can set the internal content of a responder and trigger a corresponding method using the following methods:
      ** successResult - Sets results and calls httpSuccess
      ** failureResult - Sets results and calls httpFailure
      ** completedResult - Sets results and calls httpCompleted
      
      * To obtain information about a the response from a reponder method, use the following getters:
      ** getStatus - HTTP status code
      ** getReason - Reason string
      ** getContent - Content (Parsed body LLSD)
      ** getResponseHeaders - Response Headers (LLSD map)
      ** getHTTPMethod - HTTP method of the request
      ** getURL - URL of the request
      
      * It is still possible to override completeRaw if you want to manipulate data directly out of LLPumpIO.
      
      * See indra/llmessage/llcurl.h for more information.
      f9454152
  7. Apr 25, 2012
  8. Dec 06, 2011
    • Nat Goodspeed's avatar
      LLSD-14: Extract remaining conditional LLSD mbrs to namespace llsd. · 3e6c5220
      Nat Goodspeed authored
      Per Monty's code review, it's dubious practice to have a class in which
      certain members are sometimes visible, other times not. If these were virtual
      methods, or non-static data members, the error would be obvious -- but even
      with static data members and non-virtual methods, it looks like an ODR
      violation. Extract conditional methods as free functions, as in changeset
      07cd70e75473.
      3e6c5220
  9. Dec 05, 2011
    • Nat Goodspeed's avatar
      LLSD-14: Optional entry points need conditional decls turned on. · 1a684644
      Nat Goodspeed authored
      Changeset 07cd70e75473 moved LLSD::outstandingCount() and allocationCount() to
      free functions so we could turn their visibility on/off via LLSD_DEBUG_INFO.
      But on some platforms, without proper LL_COMMON_API declarations visible when
      we compile llsd.cpp, those free functions lack proper linkage directives.
      Declare LLSD_DEBUG_INFO in llsd.cpp so that when the llcommon library is
      built, the free functions get proper linkage -- independent of compilations of
      LLSD consumers.
      1a684644
  10. Dec 01, 2011
    • Nat Goodspeed's avatar
      LLSD-14: Move LLSD::(outstanding|allocation)Count() to free functions. · 95fb0249
      Nat Goodspeed authored
      Free functions can be unconditionally compiled into the .o file, but
      conditionally hidden in the header file. Static class methods don't have that
      flexibility: without a declaration in the header file, you can't compile a
      function definition in the .cpp file. That makes it awkward to use the same
      llcommon build for production and for unit tests.
      Why make the function declarations conditional at all? These are debugging
      functions. They break the abstraction, they peek under the covers. Production
      code should not use them. Making them conditional on an #ifdef symbol in the
      unit-test source file means the compiler would reject any use by production
      code. Put differently, it allows us to assert with confidence that only unit
      tests do use them.
      Put new free functions in (lowercase) llsd namespace so as not to clutter
      global namespace.
      Tweak the one known consumer (llsd_new_tut.cpp) accordingly.
      95fb0249
  11. Nov 19, 2011
    • Nat Goodspeed's avatar
      Make LLSD diagnostic methods conditional on LLSD_DEBUG_INFO. · e97fb232
      Nat Goodspeed authored
      This establishes that there are no viewer-side unit tests relying on these
      methods. The point is to try to clean up the LLSD public API. In the same
      vein, remove from LLSD public API a diagnostic method which is nothing more
      than an implementation detail for the corresponding LLSD::Impl method. The
      same effect can be achieved by making LLSD::Impl a friend of LLSD, moving the
      method with the messy signature (classic-C arrays!) into LLSD::Impl itself.
      e97fb232
  12. Nov 17, 2011
  13. Oct 13, 2010
  14. Sep 21, 2010
  15. Aug 13, 2010
  16. Dec 01, 2009
    • Kent Quirk's avatar
      DEV-43622 : API change (no functionality change) to fix a design error in LLSD · f496c2b1
      Kent Quirk authored
      I made it about a year and a half ago; Zero found it while reading code. I had added a return value to LLSD::insert(), but a) did it wrong, and b) broke the STL-like semantics of insert(). So I've put insert() back to returning void and created LLSD::with(), which does what my earlier insert() did. The compiler then caught all the cases where insert()'s return value were being used, and I changed those to use with() instead.
      f496c2b1
  17. Aug 24, 2009
    • Richard Nelson's avatar
      merge -r 130399-131510 skinning-21 -> viewer-2.0.0-3 · 138bf113
      Richard Nelson authored
      DEV-11254 DEV-11254 DEV-2003: DEV-21567 DEV-37301 EXT-104 EXT-138 EXT-217 EXT-256 EXT-259 EXT-259 EXT-328 EXT-348 EXT-386 EXT-399 EXT-403 EXT-460 EXT-492 EXT-492 EXT-531 EXT-537 EXT-684
      
      improved text editor (handles multiple fonts simultaneously as well as inline widgets)
      138bf113
  18. Jul 16, 2009
  19. Jan 07, 2009
  20. Jan 05, 2009
  21. Jun 02, 2008
  22. May 13, 2008
  23. Nov 30, 2007
  24. Nov 07, 2007
  25. Oct 04, 2007
  26. Jun 21, 2007
  27. Feb 09, 2007
  28. Feb 07, 2007
    • Andrew Meadows's avatar
      Fixed a linux build issue when the llhavok suddenly required rtti in order to link. · bd48685d
      Andrew Meadows authored
      I added a #ifndef hack to v3math.h to eliminate LLString's from the llhavok project.
      I also removed lluuid.h's dependency on LLString (using std::string) instead and
      then fixed a bunch of bad dependency fallout on a few files around the project
      that suddenly lost their hidden access to some fundamental includes.
      The important parts were reviewed with James.
      bd48685d
  29. Feb 05, 2007
  30. Jan 02, 2007
Loading