This looks very much the same as `raiseLocalLevel` - is there no way to make these into a single function (say, by passing an increment that might be +1 or -1?)


`int`?  `m`?


The api seems like a rather dead project, but I submitted a pr to fix the response content-type issue.

https://github.com/zh/timeapi/pull/5


This is a classic example of why copy and pasting code is bad. Looks like you want to check if there is a department, location, position or group assigned before trying to access it and if not prompt the user to fill in that info on their profile. But you forgot to change the tests - so you only check if the department is nil - in all cases, which is a bug, Even if you fix that it's still bad code because it is repetitive.
```
def request_if_missing(user, attribute)
  user.try(attribute) || "Please update your profile."
end
```


Because it's autogenerated SQL from MySQL workbench and for whatever reason it includes the explicit "allow null" in the alter statements, but skips it in a create, (the after statements are come from workbench..)


Place the colon outside the translated constant


I couldn't believe we don't have an is_empty_file in the whole code base yet, but apparently we don't (as I cannot find it). The closest I could find is in sha1_file.c:map_sha1_file (line 1505-1527), which implements a check for file emptiness itself.

@peff does a bit of refactoring there currently, so maybe it's worth to move is_empty_file into some common code base providing these infrastructural functions?


Here Nate: https://pkp.sfu.ca/omp/omp_demo/

2015-05-28 14:59 GMT-03:00 Nate Wright <notifications@github.com>:

> Ok, I think that's the best case I've heard yet for keeping some
> components out of the default theme. I'll have to spend some time with OMP.
> Is there a UI/UX install of it populated with data somewhere that I can
> look at? No worries if not, I'll just install and go.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NateWr/ojs/commit/13b3c8e0f456e4f6b5c67132b576b84346e573c6#commitcomment-11412495>
> .
>


types?


I think I'm not expressing myself clearly. Nothing wrong with having a macro named JXCORE_EXTERN. Most open source projects have something like that for Windows/Posix export portability (even GCC has a similar symbol visibility attribute). I'm just saying the macro does not require an argument `x` because the visibility attribute is a property of the function, not the return value.

    #ifdef _WIN32
    #define JXCORE_EXTERN __declspec(dllexport)
    #else
    #define JXCORE_EXTERN
    #endif

...

    inline JXCORE_EXTERN JS_HANDLE_OBJECT getProcess() { return process_; }

Not a big deal either way. Just saves a couple of paren characters on each use.


This commit is the only really important part of the change. Otherwise I just updated testify.


`const` goes after the type...


Thanks. I'm cooking up some additional commits to make this better.


Referenciando #16


Tests are failing the since the C parser is imported on pypy too.


Merge


See the similar macros defined in https://gcc.gnu.org/wiki/Visibility


use the special translation function:
``````
$aco_name=xl_gacl_group($res['name']);


See [my response on the mailing list](http://lists.jabber.ru/pipermail/ejabberd/2015-May/008717.html).


@bboe @dtognazzini I'm all for getting as much out as you can and then fleshing it out as you go. Kinda wiki-ish. Hm. Is this README a better tool for this than a wiki? I know, "not another wiki!" And I don't want to distract from just getting this stuff out there, and we can revisit how it's stored later _if_ we see the need. The paramount point is to capture and share this stuff.


Would it be wrong to reuse BINARY_ALIGN (which happens to be 8) here?


I don't think we need to reset a newly constructed strbuf. Oh wait, it's static!
That feels a bit like premature optimization, though we do it all the time in git, so maybe it's ok.


IOExceptions makes sense here because we would reach this point in code when the appropriate credential files for the ADC feature have not the created (either by the application developer or by auxiliary tools like gcloud).

Let me know if you believe there is another exception type that fits better here.


If this passes LTC I have a patch ready to go to test just that possibility.


hey, why do you get git 2.4.2 and I'm still stuck on 1.9.5? That's unfair :-(


I do not understand why you need to check that Puppet is installed.
If Puppet is not installed, then the validation should not throw an error. Said differently, if the tests fails, then the code is wrong. Not the tests. ;)


haters gonna hate


How we capture and compare screenshots with baseline base on this API? How we choose proper baseline image if file name changes every time I add new source code line?


return what now?


Why HTTP_OK?


@lynnetye Here it would be simpler to do `VetPronto.formSections[sectionName].submitStep()`, because then you could delete `VetPronto.main.submitStep`.


Thanks!

On Thu, May 28, 2015 at 7:10 PM, Bruno Beghelli <notifications@github.com>
wrote:

> Here Nate: https://pkp.sfu.ca/omp/omp_demo/
>
> 2015-05-28 14:59 GMT-03:00 Nate Wright <notifications@github.com>:
>
> > Ok, I think that's the best case I've heard yet for keeping some
> > components out of the default theme. I'll have to spend some time with
> OMP.
> > Is there a UI/UX install of it populated with data somewhere that I can
> > look at? No worries if not, I'll just install and go.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/NateWr/ojs/commit/13b3c8e0f456e4f6b5c67132b576b84346e573c6#commitcomment-11412495
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NateWr/ojs/commit/13b3c8e0f456e4f6b5c67132b576b84346e573c6#commitcomment-11412723>
> .
>


kkkkk ok


Sorry for the confusion.  I realize that.  This file shouldn't be checked in as part of the change.  I only put the code there to demonstrate the usage of the new callback.  I think mentioned that in the comments, maybe I should have made that clearer.

In the real implementation I use my own 'RequestModifierExtension' and change the context to reference mine.

BaseURLExtensions.js, FragmentLoader.js and ManifestLoader.js are the changes for consideration.

Thanks,
Rik.


Hang over from previous… still to be cleaned up


A class object


lol


Hello Toni,
Quick question. Is it okay to change the format of Graphite message string? To replace "_" with "." ; this gives users more options to filter.

Also, I couldn't retrieve cluster name for Datastore and ResourcePool managed entities in InitClusterMap method. For the time being, for these two entities cluster is not included in Graphite messages.

Thanks

Lava

Sent from my iPhone

> On May 27, 2015, at 12:58 AM, Toni Moreno <notifications@github.com> wrote:
>
> Hi @lbasavap . I could wait for some days for the 4 proposed changes . Feel free to ask me for help if you need.
>
> —
> Reply to this email directly or view it on GitHub.
>


@lynnetye This is nit-picky, but this variable name is overly specific. I'd just call it `$formSection`. It's obvious from the code which section it is -- the one that corresponds to the passed-in `sectionName` argument.


I'd rather an Email Notification Tester class, instead of a general helper class for the controller. IMO controllers shouldn't talk to helpers, they should talk to services.


Used the command:
```
zf create form Guestbook
```


The PHPDoc doesn't match


If the name is getting displayed, it will need to be translated somewhere via xl_gacl_group() function


SS


@cossorzano , Do you really think this param *inputMask* is needed? All VolumeMask objects are also Volume, so they can enter in the *inputVolume* param. I'm missing something here?


I'm not sure that will work... we need somewhere "temporary" to first receive the header, as we can only allocate a buffer for the whole message (including header) once we know how big it will be.  This copy might be unavoidable.  But I might be wrong, let me know if you think this can be done another way.


This is going to have about 20+ different tests in it, not just email


Since this is quite closely related to the primary function, some might prefer seeing both entries with a single grep.  Easier to have a broad differential diagnosis and narrow than the other way around.

It would be easy enough to change this to acl_check_document, but that feels redundant since this is already part of the document class.  If you feel strongly that it should explicitly be a different name, I will change it.


Rename


What if argument mu is 0?


PHPStorm :)


@lynnetye `displayHeaderLink` is not a good name here. It does something totally different than what `displayFormSection` does, but it has almost the exact same name. Confusing.


I'm not married to the name, I just needed something to get started


So this class is going to be 800 lines long? I think they should be in separate classes


rip


@justinforce I agree with you. @petekinnecom has been working to move all af_academy information to our google sites wiki. While I think Google sites is fairly terrible (and who knows how long it'll be around), I think having all information in one searchable location is idea. Google sites does at least work well for search, whereas github's search is very lacking.


Thanks for the explanation. I indeed suppose caching this meta information can help a lot for larger expressions.

You could create a local, private function `_derivative` which is used recursively, and pass an extra argument with the cache to this function. Then you could export a public `derivative` without this second argument. Alternatively you could still define the cache globally like you propose in your last comment, and clear it at the start of the public function `derivative`. The first solution gives less side effects (no globals). I don't know which of the two versions will be least verbose though. I'm fine with both, it's not a big deal.


This entire section gave me  cancer


BINARY_ALIGN is 8, but that is only needed on SIMD. On SIMD builds, the binary is dereferenced as uint64, so that is why 8 byte alignment is requested.  But on non-SIMD, the compares are done with memcmp, so 1 byte would work.

For this allocation (it probably should be changed to non-tiny), I really would rather have a specific 8 kept, vs using BINARY_ALIGN, which is NOT tied to these variables.


Dooh. I should have thought of this one right away. LOL.
We continually come back to these types of perf improvements in many of our components.


Yeah, when it provides me with a usable workflow, I'll switch in a heartbeat.


xl_gacl_group to be applied here to translate names.


Fair point, very fair point


"a file" or "files"


I was also aiming for 2500 lines… Pfft… 800!


content of "duration.yaml"


hehe


:smile:


"by duration" instead of "of duration"? "only extended of duration intermediary" node doesn't parse.


No strong opinion on this, so would leave as is.


This object only has the section_value and value, the result will ultimately be translated by matching up with the correct entry in the aco_list.  aco_list's names will be translated after retrieval from the database above.


comma in the wrong place. Missing an "and"?


"a setup" or "setups"


done


My philosophy on comments is not to state the obvious and then to give reasons why, not describe what is happening -- why repeat in English what has already been said in C++? My expectation of someone reading program code is that they know the language the code is written in...

(The comment for `CellmlFile::cmetaId()` is a result of editing a cut-and-paste of `CellmlFile::xmlBase()`)


contains


the current value


This is weird


Should be something different here.


setup


agreed


explained in :doc:`MultiplexConfig`,


uploading UC site


I missed your answer here, but that's OK because you're right :p


fix the commit


Yes, it was an honest typo.


low_hanging_fruit (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1666 of 1666 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Hey Greg,

This exists so that when Galaxy attempts to kill a running tool, it kills all the children of that tool. Unfortunately, if `setpgrp` isn't used, when Galaxy attempts to kill a tool, it's only going to get the top level process (probably a shell script) and the rest of the tool's processes will keep running.

Is there any chance we can figure out why the tool is hanging with `setpgrp`?


add more options, ie. mods, paid mod, add paypal option


Not directly related to this patch, but in the context of the large number of memory changes under way I think we should revisit this logic.  Specifically, I think after the ABD changes are merged and the possibility exists for page cache integration this needs to change.  FWIW things have changed on the illumos side here as well.


Can we default to line-delimited and remove the configuration option?


Pushed to HaxePunk commit: https://github.com/HaxePunk/HaxePunk/commit/6758c978fa2e9a33ea0e89ee43e2d5f670f3b90c


Ah, gotcha. Not a critical thing for this PR, for sure.


Thanks for the details. What I was trying to say was; The result of JXCORE_EXTERN (possibly) will be depending to the location where it's used. It may either declare import or export. So, making it look like a function may give a further signal / warning to other developer. This is the intention.

If there was no possibility of marking some parts of the project as import, you are right, there is no use case and waste of extra 2 parentheses.

Maybe I'm over-thinking..


remove this line, AFK bot has too much power


Unresolved rebase.


do not change the commit only supports english


Directly after introduction of cmh it was tried to remove countermoves from move sorting, but failed.

http://tests.stockfishchess.org/tests/view/5501cf730ebc5902160ec0fa

But in the mean time other patched for move sorting and search improvement has committed and the complete countermove table + update can be than removed. So perhaps now it could pass.


teste


oops... forgot to uncomment this after testing.


Unresolved from rebase.


I was surprised capekfilip didn't do a pull request.  But thank you X 100 capekfilip for catching that.


You may want to take a look at lines 577 to 586 of io_manager.cpp. That is the part that parses the GLF file.  It suggest that you should have the file formatted as SNPs as rows, and GLF triplets for each individual as columns.  So a P x 3N matrix for P snps and N subjects.  Let me know if this helps.


Pretty cool with the generics.


u suck


next step should probably be to move things like this to the app they belong in. questions in this case.


Actually, the intent here is to deprecate and remove the instance variable entirely--runner appears to be entirely local.


Thanks for taking the time on this, I want to share with you what I think you guys can do to help us manage events from new zenpacks:

1.  For external events (ie windows events, vsphere events, events for ucs polling), deliver them to us with an eventClassKey(preferably make them URL safe (eh hem...not with | in them)  (i wish that prepId would just run on creation or mapping of new keys).  Never hard code the event class.  use default mapping if you have to...those are good sources of documentation.    if you give it an event key, clearly explain the logic.

    That way if I need to implement custom clearing logic I can construct and use eventKeys to handle clearing logic that is technology or implementation specific.  a great example is vspere zenpack, I always have a good and logical tiered eventClassKey (nice job Josh and everyone else who contributed!).  i can then implement eventKeys to do things like clear vsphere status alerts or handle situations where the eventKey is based on multiple components.  sometimes i will even have to use a hash for an eventKey or dedupid...

2.  For events that indicate that something is sick with the daemon or datasource, understand that those are different types of events and I really want to know what I should expect so I can watch out for unintended clearing issues.   these for me are critical with new zenpacks.  i dont mind that there are problems it drives me nuts if I dont know or have to find major issues in logs.  ..  These should have an eventKey that the developer builds and an eventClassKey.  I need an event Key because I don't know what the clearing logic is.  the biggest mistake i see here made is one event clearing a bunch of unrelated events, having an event key IMO forced you to think through the logic of what should clear what.

* always create one eventClassKey per condition you are checking for...do not (please).. create one eventClassKey for an error condition and another that clears it...that is very confusing.  instead use one eventClassKey and an eventKey and change your severities.... I run into this all the time.  the problem is one event can go off and clear another ie one poll fails for say hard disk status (OMG super critical) then fan polling completes and it clears the disk drive polling failure.*

3.  For events that indicate a polled (not external event) monitoring issue with the device, those should have eventClassKeys and potentially an eventKey...if there is an eventKey the logic should be explained.

4.  Never hard code the event Class,  if you have to place it somewhere hve the eventclassKey install wth the zenpack but watch out for unintended clears...my end users expect things to show up in certain event classes... consistency is important and since there is no universal standard for eventclass structure (although I have my opinions) having things show up in unexpected places can cause our noc or any automated triggers etc to miss things..

5.  This is what I do after every zenpack  or install where the zenpack is zenpython/zencommand, or has a new daemon or has a new threshold type (ie status threshold) :
*check wiki for documentation,
*if no documentation, go through the daemon or plugin code and identify any eventClassKeys (and eventClasses but hopefully not)
*check the mappings and see if they were added on install, if they where I moved them based on what they are.
*if there are not event ClassKeys in the zenpack, i add them and set the severity (hopefully) to original but usually i'll set critical if its not a clear at least at first....sometimes i'll add a count transform if there is no zproperties for timeout/retries..
*see if there is a way to test failures...but with us you can bet we will see unexpected things anways...What I need to know in documentation is what the intention was for things... that way i can see quickly if the issue I see is due to a "bug" in the zenpack or a bug in the vendors api/software/hardware or a unexpected result due to not being able to test every variety of a specific device.  I see these all as different things and if you can help me determine this, you will get less support calls and the ones I send in will get right to the point.
*check for default mappings, set the sequence, make sure that they arent going to try to run rules against attributes that are not always present (ie agents doesnt exist for some zenoss internal events still, so rules / regexes will fail)
*if there is a default mapping with a regex...we need to be super careful there especially if there is no rule..we will keep a close eye on those..ideally use a rule with alot of hasattr/getattrs.

6.  if you want to create eventClassKeys that get added on install, ideally add them to unknown.  try not to create event classes if possible on install.  actually more ideally there is a standard event class that has a heirarchy where new eventClassKeys land when you install.  that way I can look in one place and dont have to look though thousands of mappings to find them or go through the zenpack persistance links.

for example:  if we have a polling error that is a zenpack issue, we will put it under /Status/(ZenPack) or something like that, if there is a hardware event generated for a polled event we will probably put it under /status/hw/vendor/component type...everyone has a standard i guess but when my team is checking for zenoss health, we need to know where to look..

what I need on documentation:
-> list of event class keys for events generated when there is a polling problem or an unexpected error, do they have eventKeys, if they are installed where are they in the eventClass
-> for external events, the format I should expect eventclasskeys to look like, doesnt need to be every possible key if there is a standard.
-> list of polled events/status threhsolds, their ecks..etc
-> if there are default mappings installed, clearly identify this...


Yes, I think the odds are against it passing but worth a try.


This just downloads and sets up URLs for the JCE archive. However, JCE is not actually installed. In java cookbook, `default['java']['oracle']['jce']['enabled'] = false` and so jce installation is [skipped](https://github.com/agileorbit-cookbooks/java/blob/master/recipes/oracle.rb#L70). We need to set this attribute to true in bcpc-hadoop:java_config.rb in order to install jce.


Hahahahahaha


Nice, can you integrate that in the previous commit so we don't have a revision that won't work in our code base. https://help.github.com/articles/about-git-rebase/ has instructions on how to squash commits.


This is so weird. I'm trying to establish a safe no-SIMD build for Linux. I run this on well: `./configure CFLAGS="-g -O2 -mno-sse2" --disable-cuda --disable-opencl`

I have verified that the compiler does NOT define `__SSE2__` and pomelo_plug.c is completely wrapped in `#ifdef __SSE2__` but **despite that** I get this:
```
pomelo_plug.c: In function ‘PHS_pomelo’:
pomelo_plug.c:278:49: warning: unused variable ‘mask2’ [-Wunused-variable]
     unsigned long long state_size, mask, mask1, mask2;
                                                 ^
pomelo_plug.c:274:28: warning: unused variable ‘k’ [-Wunused-variable]
     unsigned long long i,j,k,temp;
                            ^
```
How is that even possible?!


Boy, the answer to that is AMPG I'm afraid.  I can do some googling, but I'm not an expert in this area.  I'm wondering why this is necessary only in the local job runner.  Is there another approach that can be leveraged from the other runners?


It would be great to keep the indentation consistent.


Used the command:
```
zf create action sign Guestbook
```


Also this patch seems to be touching more than just creating a few new dialogs. Perhaps some of the edits to the existing files might be causing the freeze to the UI.


NB: the command worked only after I added these 3 lines to ` .zfproject.xml`


xd


Ha! I forget you see all this as I do it...


This is not a proper fix. You basically reverted part of my commit and broke Gentoo support.

Proper fix would be to fix the package names in your pillar or the map.jinja


The import/export thing is outlined in https://gcc.gnu.org/wiki/Visibility as well. It has to be driven by some external compiler flag defining a macro like -D"USE_DLL" for imports or -D"BUILDING_DLL" for exports, as an example.

Lua does this:

    #if defined(LUA_BUILD_AS_DLL)       /* { */

    #if defined(LUA_CORE) || defined(LUA_LIB)   /* { */
    #define LUA_API __declspec(dllexport)
    #else                                               /* }{ */
    #define LUA_API __declspec(dllimport)
    #endif                                              /* } */

    #else                               /* }{ */

    #define LUA_API             extern

    #endif                              /* } */


I obtained some weird figures using EPS as export format from Octave (completely black figures, syntax errors, ...)
So I would suggest we keep the PDFs for now (and it would be probably better if we can produce PDF output from MATLAB instead)


Interface is wrong.  `PolicyList, error`


hey thats me!


I just noticed that ir broke Debian as well, as I had Debian also in my mind when updating the formula

Package names do not always begin with "dovecot-" in any distribution on the planet ;)


@mschiff, can you submit a pull with the fix for debian please ?

> On May 28, 2015, at 1:47 PM, mschiff <notifications@github.com> wrote:
>
> I just noticed that ir broke Debian as well, as I had Debian also in my mind when updating the formula
>
> Package names do not always begin with "dovecot-" in any distribution on the planet ;)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/saltstack-formulas/dovecot-formula/commit/73e2b79a2668cf09b0d7962911e93a67a73e8657#commitcomment-11413471>.
>


u0_a87@mx4pro:/ $su
enter main
start command :am start -a android.intent.action.MAIN -n com.android.settings/com.android.settings.root.RootRequestActivity --ei uid 10087 --ei pid 12443 > /dev/nullu0_a87@mx4pro:/ # u0_a87@mx4pro:/ $su
sh: u0_a87@mx4pro:/: not found
127|u0_a87@mx4pro:/ # enter main
sh: enter: not found
 uid 10087 --ei pid 12443 > /dev/nullu0_a87@mx4pro:/ #                        <
sh: can't create /dev/nullu0_a87@mx4pro:/: Is a directory
1|u0_a87@mx4pro:/ #amstart -a
sh: amstart: not found
127|u0_a87@mx4pro:/ # am start -a
usage: am [subcommand] [options]
usage: am start [-D] [-W] [-P <FILE>] [--start-profiler <FILE>]
               [--sampling INTERVAL] [-R COUNT] [-S] [--opengl-trace]
               [--user <USER_ID> | current] <INTENT>
       am startservice [--user <USER_ID> | current] <INTENT>
       am stopservice [--user <USER_ID> | current] <INTENT>
       am force-stop [--user <USER_ID> | all | current] <PACKAGE>
       am kill [--user <USER_ID> | all | current] <PACKAGE>
       am kill-all
       am broadcast [--user <USER_ID> | all | current] <INTENT>
       am instrument [-r] [-e <NAME> <VALUE>] [-p <FILE>] [-w]
               [--user <USER_ID> | current]
               [--no-window-animation] [--abi <ABI>] <COMPONENT>
       am profile start [--user <USER_ID> current] <PROCESS> <FILE>
       am profile stop [--user <USER_ID> current] [<PROCESS>]
       am dumpheap [--user <USER_ID> current] [-n] <PROCESS> <FILE>
       am set-debug-app [-w] [--persistent] <PACKAGE>
       am clear-debug-app
       am monitor [--gdb <port>]
       am hang [--allow-restart]
       am restart
       am idle-maintenance
       am screen-compat [on|off] <PACKAGE>
       am to-uri [INTENT]
       am to-intent-uri [INTENT]
       am switch-user <USER_ID>
       am start-user <USER_ID>
       am stop-user <USER_ID>
       am stack start <DISPLAY_ID> <INTENT>
       am stack movetask <TASK_ID> <STACK_ID> [true|false]
       am stack resize <STACK_ID> <LEFT,TOP,RIGHT,BOTTOM>
       am stack list
       am stack info <STACK_ID>
       am lock-task <TASK_ID>
       am lock-task stop
       am get-config

am start: start an Activity.  Options are:
    -D: enable debugging
    -W: wait for launch to complete
    --start-profiler <FILE>: start profiler and send results to <FILE>
    --sampling INTERVAL: use sample profiling with INTERVAL microseconds
        between samples (use with --start-profiler)
    -P <FILE>: like above, but profiling stops when app goes idle
    -R: repeat the activity launch <COUNT> times.  Prior to each repeat,
        the top activity will be finished.
    -S: force stop the target app before starting the activity
    --opengl-trace: enable tracing of OpenGL functions
    --user <USER_ID> | current: Specify which user to run as; if not
        specified then run as the current user.

am startservice: start a Service.  Options are:
    --user <USER_ID> | current: Specify which user to run as; if not
        specified then run as the current user.

am stopservice: stop a Service.  Options are:
    --user <USER_ID> | current: Specify which user to run as; if not
        specified then run as the current user.

am force-stop: force stop everything associated with <PACKAGE>.
    --user <USER_ID> | all | current: Specify user to force stop;
        all users if not specified.

am kill: Kill all processes associated with <PACKAGE>.  Only kills.
  processes that are safe to kill -- that is, will not impact the user
  experience.
    --user <USER_ID> | all | current: Specify user whose processes to kill;
        all users if not specified.

am kill-all: Kill all background processes.

am broadcast: send a broadcast Intent.  Options are:
    --user <USER_ID> | all | current: Specify which user to send to; if not
        specified then send to all users.
    --receiver-permission <PERMISSION>: Require receiver to hold permission.

am instrument: start an Instrumentation.  Typically this target <COMPONENT>
  is the form <TEST_PACKAGE>/<RUNNER_CLASS>.  Options are:
    -r: print raw results (otherwise decode REPORT_KEY_STREAMRESULT).  Use with
        [-e perf true] to generate raw output for performance measurements.
    -e <NAME> <VALUE>: set argument <NAME> to <VALUE>.  For test runners a
        common form is [-e <testrunner_flag> <value>[,<value>...]].
    -p <FILE>: write profiling data to <FILE>
    -w: wait for instrumentation to finish before returning.  Required for
        test runners.
    --user <USER_ID> | current: Specify user instrumentation runs in;
        current user if not specified.
    --no-window-animation: turn off window animations while running.
    --abi <ABI>: Launch the instrumented process with the selected ABI.
        This assumes that the process supports the selected ABI.

am profile: start and stop profiler on a process.  The given <PROCESS> argument
  may be either a process name or pid.  Options are:
    --user <USER_ID> | current: When supplying a process name,
        specify user of process to profile; uses current user if not specified.

am dumpheap: dump the heap of a process.  The given <PROCESS> argument may
  be either a process name or pid.  Options are:
    -n: dump native heap instead of managed heap
    --user <USER_ID> | current: When supplying a process name,
        specify user of process to dump; uses current user if not specified.

am set-debug-app: set application <PACKAGE> to debug.  Options are:
    -w: wait for debugger when application starts
    --persistent: retain this value

am clear-debug-app: clear the previously set-debug-app.

am bug-report: request bug report generation; will launch UI
    when done to select where it should be delivered.

am monitor: start monitoring for crashes or ANRs.
    --gdb: start gdbserv on the given port at crash/ANR

am hang: hang the system.
    --allow-restart: allow watchdog to perform normal system restart

am restart: restart the user-space system.

am idle-maintenance: perform idle maintenance now.

am screen-compat: control screen compatibility mode of <PACKAGE>.

am to-uri: print the given Intent specification as a URI.

am to-intent-uri: print the given Intent specification as an intent: URI.

am switch-user: switch to put USER_ID in the foreground, starting
  execution of that user if it is currently stopped.

am start-user: start USER_ID in background if it is currently stopped,
  use switch-user if you want to start the user in foreground.

am stop-user: stop execution of USER_ID, not allowing it to run any
  code until a later explicit start or switch to it.

am stack start: start a new activity on <DISPLAY_ID> using <INTENT>.

am stack movetask: move <TASK_ID> from its current stack to the top (true) or   bottom (false) of <STACK_ID>.

am stack resize: change <STACK_ID> size and position to <LEFT,TOP,RIGHT,BOTTOM>.

am stack list: list all of the activity stacks and their sizes.

am stack info: display the information about activity stack <STACK_ID>.

am lock-task: bring <TASK_ID> to the front and don't allow other tasks to run

am get-config: retrieve the configuration and any recent configurations
  of the device

<INTENT> specifications include these flags and arguments:
    [-a <ACTION>] [-d <DATA_URI>] [-t <MIME_TYPE>]
    [-c <CATEGORY> [-c <CATEGORY>] ...]
    [-e|--es <EXTRA_KEY> <EXTRA_STRING_VALUE> ...]
    [--esn <EXTRA_KEY> ...]
    [--ez <EXTRA_KEY> <EXTRA_BOOLEAN_VALUE> ...]
    [--ei <EXTRA_KEY> <EXTRA_INT_VALUE> ...]
    [--el <EXTRA_KEY> <EXTRA_LONG_VALUE> ...]
    [--ef <EXTRA_KEY> <EXTRA_FLOAT_VALUE> ...]
    [--eu <EXTRA_KEY> <EXTRA_URI_VALUE> ...]
    [--ecn <EXTRA_KEY> <EXTRA_COMPONENT_NAME_VALUE>]
    [--eia <EXTRA_KEY> <EXTRA_INT_VALUE>[,<EXTRA_INT_VALUE...]]
    [--ela <EXTRA_KEY> <EXTRA_LONG_VALUE>[,<EXTRA_LONG_VALUE...]]
    [--efa <EXTRA_KEY> <EXTRA_FLOAT_VALUE>[,<EXTRA_FLOAT_VALUE...]]
    [--esa <EXTRA_KEY> <EXTRA_STRING_VALUE>[,<EXTRA_STRING_VALUE...]]
        (to embed a comma into a string escape it using "\,")
    [-n <COMPONENT>] [-f <FLAGS>]
    [--grant-read-uri-permission] [--grant-write-uri-permission]
    [--grant-persistable-uri-permission] [--grant-prefix-uri-permission]
    [--debug-log-resolution] [--exclude-stopped-packages]
    [--include-stopped-packages]
    [--activity-brought-to-front] [--activity-clear-top]
    [--activity-clear-when-task-reset] [--activity-exclude-from-recents]
    [--activity-launched-from-history] [--activity-multiple-task]
    [--activity-no-animation] [--activity-no-history]
    [--activity-no-user-action] [--activity-previous-is-top]
    [--activity-reorder-to-front] [--activity-reset-task-if-needed]
    [--activity-single-top] [--activity-clear-task]
    [--activity-task-on-home]
    [--receiver-registered-only] [--receiver-replace-pending]
    [--selector]
    [<URI> | <PACKAGE> | <COMPONENT>]

Error: Argument expected after "-a"
u0_a87@mx4pro:/ #


Is there a "the most stupid lintian warning of the year" nomination? This one should get the first prize.


For posterity, you can find an example and more info in #9.


the meaning of the  commit is like that


Do you intend to print stack trace here or Log.e is enough?


This has already  been corrected in a more recent commit.


With the other runners we leave it up to the DRM to kill the jobs - I am not sure how they track which processes belong to a particular job, although I think in most cases the DRM starts a separate "manager" process that is responsible for the job. It may be possible to walk a process tree starting with the parent process?


Por temas de depuración metí una línea adicional en el consumidor para ver en ejecución si los datos que llegaban eran malos o no


this should do to your global git ignore IMHO, because it depends on your environment: `git config --global core.excludesfile '~/.gitignore'`


Do we have a ticket to track this?


I know that this is a trivial case here, but it is like that everywhere. So, if anything, it looks inconsistent here (and I like consistency).

More importantly, there is so much code around that has never been commented because it's self-explanatory. Yet, it often happens that it's not. So, nowadays, I prefer to comment too much than not enough.


test comment


I will remove it. Not my intention.


You shouldn't call this directly. You should instead update the data and then call updateUI.


I think that Masks were not Volumes (once upon a time), and it is true
that in the current version I did not check whether the mask was offered
or not as volume.

El 28/05/2015 a las 20:21, Jose Miguel de la Rosa Trevin escribió:
>
> @cossorzano <https://github.com/cossorzano> , Do you really think this
> param /inputMask/ is needed? All VolumeMask objects are also Volume,
> so they can enter in the /inputVolume/ param. I'm missing something here?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/biocompwebs/scipion/commit/ad721c0f1a2f7b9e9374503fdd90f878a1d62bec#commitcomment-11412962>.
>


`@mdo`*


How clear the LUA definitions... Amazing though. Below from SpiderMonkey;
```c++
#define JS_EXTERN_API(type)  extern MOZ_EXPORT type
#define JS_EXPORT_API(type)  MOZ_EXPORT type
#define JS_EXPORT_DATA(type) MOZ_EXPORT type
#define JS_IMPORT_API(type)  MOZ_IMPORT_API type
#define JS_IMPORT_DATA(type) MOZ_IMPORT_DATA type

#if defined(STATIC_JS_API)
#  define JS_PUBLIC_API(t)   t
#  define JS_PUBLIC_DATA(t)  t
#elif defined(EXPORT_JS_API) || defined(STATIC_EXPORTABLE_JS_API)
#  define JS_PUBLIC_API(t)   MOZ_EXPORT t
#  define JS_PUBLIC_DATA(t)  MOZ_EXPORT t
#else
#  define JS_PUBLIC_API(t)   MOZ_IMPORT_API t
#  define JS_PUBLIC_DATA(t)  MOZ_IMPORT_DATA t
#endif

#if defined(STATIC_JS_API) || defined(EXPORT_JS_API) || defined(STATIC_EXPORTABLE_JS_API)
#  define JS_FRIEND_API(t)    MOZ_EXPORT t
#  define JS_FRIEND_DATA(t)   MOZ_EXPORT t
#else
#  define JS_FRIEND_API(t)   MOZ_IMPORT_API t
#  define JS_FRIEND_DATA(t)  MOZ_IMPORT_DATA t
#endif
```
IMHO both ways are okay. If you want to change it, feel free to replace them (please include the ones under 'public/' folder.)


Do we really need that?  Is there no public site you can point to?


Misunderstanding related to AdNauseam? https://github.com/gorhill/uBlock/issues/67


* bring IS not created..


@lynnetye Delete this line. You already know how to wait for an AJAX response: call the `done` function on the request object and pass it a callback function.


This change makes it so you can _only_ choose directories rather than individual files.


Had to create this manually and copy-paste the contents.


Should we add key-value Configuration options to the Read.Bound object to do things the InputFormat will need, like (e.g.) set Avro schemas?

I also sort of expect that CDF will add support for something like this relatively soon (since we would need it, Flink would need it, etc., etc.) but I get that we shouldn't wait for them on this so we can make progress on other stuff.


thats a NOP and not needed for the screensaver


Ah, sorry :)


What is the use case for this? As the standard classes are not all replaced.


add one sentence to introduce the wizard.

> First, we create a site with sample content in it. We’ll use the \`\`nikola init\`\` wizard (with the \`\`--demo\`\` option) for that::


No problem. Currently, Mask 'is' Particle and VolumeMask 'is' Volume.
A que mola github?


This line should be after the `var var = var` line. Maybe add a comment explaining why we include it.


Will revert removing this variable...


Hi, can you sign this commit too ?

I will patch the Starter to use JsonObject constructor for properties with no setter, so it can work.


@lynnetye This is unnecessary, because you've already initialized `VetPronto.state.currentStep` to the string `'schedule'`.


This still seems worth including.


We should provide here more infos yes


saw approval from jonmeredith
at https://github.com/basho/riak_kv/commit/b595ab5735d490a20706f2fe5a8999becacf3c82


merging basho/riak_kv/bugfix/jrd/vnode_blocking = b595ab57 into borshop-integration-1121-bugfix/jrd/vnode_blocking


basho/riak_kv/bugfix/jrd/vnode_blocking = b595ab57 merged ok, testing candidate = 2a863857


This has been corrected in a more recent commit.


Linking to master can break things if we introduce new things in that file.  We should recommend to copy it from your local installation instead.


Okay, I now see why you did it that way.  I guess my complaint is with Mozilla's export macros. ;-)


Very nice, good job


well, yes http://lanyon.getpoole.com/


rax retry tests


This link requires theme support (CSS for `.sr-only`), are you sure lanyon has that?


It does not.


since when are we jekyllrb.com?


since when is @ralsina mdo?


I could find no simple way to do that, but it looks like

``nikola install_theme base`` and looking at the error can do it.


I think the actually step9 file got missed


I'm just not getting this. It says that when `__index__` is defined, you should have `__int__`.  Not vice versa. And it says   `__index__` makes the type look like a number.   And I certainly don't want that for EnumItem.


Code needs refactoring


For future references, Make sure this is always true...


submit to chrome store?


I think we should discuss that face to face to resolve this more
efficiently.
On Thu, May 28, 2015 at 11:18 AM slk9 <notifications@github.com> wrote:

> How we capture and compare screenshots with baseline base on this API? How
> we choose proper baseline image if file name changes every time I add new
> source code line?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ohenak/KIF/commit/9592b0923c6bdf3753ca8525493d8c24d21d7d72#commitcomment-11412882>
> .
>


Maybe link to the definition of SameValue in the spec instead of (see implementation for specifics)?


защищенным полем*


@rgrunber I pushed another version of the commit where I fixed the problems you reported. Can you establish a connection now ?


The repeated "helper function that will" prefix (in both the new helpers and the existing table entries) is a bit silly.


looks good!


I'm on mumble now since i closed the issue.


This commit isn't ready for anyone.


We are not, but what it says there is true. I did not do the theme, I just ported it. Credit where credit is due.


Is this doc meant to say "Approximation"?


idea: `nikola install_theme --get-path base`. #1762


Since I wrote this much doc in one hour ;-) Fixing...


I needed to declare the 'channel' variable again inside the T.get() callback. I'm not sure why.


Yes, I definitely learned a lot by this mistake. Everything is clean now.


Typo here?


- Se ha añadido un paquete para generar ventanas modales de bootstrap de forma sencilla.
- Se ha añadido un paquete para administrar las suscripciones generadas en las rutas.
- Se ha añadido un paquete que incluye una gran variedad de helpers de mucha utilidad.
- Se ha creado un fichero en el que se incluyen todas las variables y helpers globales. Así mismo se han quitado helpers repetidos entre las templates.
- Se han movido todas las suscripciones (a excepción de las del layout maestro) a los métodos waitOn de las rutas.
- Se ha añadido un menú desplegable en el layout maestro para ver las notificaciones y tomar acciones en ellas.
- Se ha cambiado la vista de la publicación para reflejar dar uso a las opciones de crear publicación, regatear y cancelar trato. El estado cambia cuando cuando una publicación es aceptada y si se ha aceptado tras un regateo, se ve el precio inicial tachado y el precio final de forma normal.
- Se han añadido diversos métodos para reflejar lo anterior.
- Se han creado la colección de regateos, la de tratos y la de notificaciones.


why are these whole things committed?


Looks promising :D
I love your pure french revolutionary spirit :P
To not mention how much I love the "let's dance" statement ahahha


This order seems more logical to me: First explain the basic Authority (to do stuff, to "Act"), _then_ speak about the Authority over Domains (a specific way to act).

In this edit, I introduce paragraph breaks in one section - this would make the formatting inconsistent with the rest of the constitution. I propose to add them, and will be happy to go through the rest of the text, and introduce linebreaks there too.


The screenshots look dirty (you could use `capty` with png saving, for example — just make sure to `nikola serve` things for it), and you are [part of a botnet](http://www.businessinsider.com/hola-used-for-botnet-on-chrome-2015-5).


:o

On 28 May 2015, at 20:06, Maciek Kurek <notifications@github.com> wrote:

Code needs refactoring

—
Reply to this email directly or view it on GitHub.


we should make a note of this at the meeting.... I actually test in the browsers my contract says I will support so it usually looks something like ["PhantomJS", "Chrome", "Safari", "Firefox"] on my machine but it may be a helpful tip for our newer to development members. either way this is good for the sake of performance for those with weaker machines


Hi,

I can, but I am still confused and now I'm not sure what should be in my pull request. Are you saying you'll patch `MetricsOptions` so it will retain the JSON passed to it? Because right now the JSON constructor looks like this:

```
  public MetricsOptions(JsonObject json) {
    enabled = json.getBoolean("enabled", DEFAULT_METRICS_ENABLED);
  }
```

Which means that no matter what we pass in, it will be discarded before it gets to the Dropwizard code.


Could you submit this as a pull request to my github repo? https://github.com/hexmode/xml-rpc-el


capty totally screws this theme (check themes.getnikola.com#lanyon)

Also, the screenshots are taken at steps of the creation which are not stored anywhere so they are basically impossible to reproduce unless someone follows the tutorial :-(

I'll try to clean them up a bit later.


So we can `bower install` from the git repo. They can be removed again if we set a specific tag/version


you probably did something wrong with CSS at some point.


comaprisons


Paragraph already says "one or more of the following", so the "and/or" is redundant.


when I see you you can tell me the story why "directors" came to be, sounds like they must have made your life easier in some way with what gets executed in C++ or Java automagically, that's what I understand at least from the docs, but not sure.


this took a really dark turn, all of a sudden.


Looks like I forgot to add the assets!


Bundles are used by Nikola to speed things up by putting all JS and CSS in big files that encompass everything. In order to use them, you need to install `webassets` from PyPI and `USE_BUNDLES` needs to be set to `True` (which is the default setting).

In order to create bundles, you need a `themes/lanyon/bundles` text file.  Its format is `output.filename=input,files,separated,by,commas`.  A sample file for lanyon is:

```
all.css=rst.css,lanyon.rss,poole.rss,code.css,custom.css
all-nocdn.css=rst.css,lanyon.rss,poole.rss,code.css,custom.css
```


I think this is the default? - do we still need the "?"?


You're right. It was introduced mainly because `apt-cache search tcmalloc` returns libtcmalloc_minimal4 and libtcmalloc_minimal4-dbg packages instead of libgoogle-perftools-dev, which contains proper version of libtcmalloc. It will be fixed tomorrow, thank you so much for finding the bug.


These are never getting logged.


Yep...that's a cherry-pick from HSC, so I inherited it!  I'll fix it before putting this in for review.  FYI, I use my u/lauren/DM-NNNN branches as work-in-progress.  They are not ready for review until I explicitly put them in review status in JIRA.  I am in the habit of making official tickets/DM-NNNN before merging to master.  Thanks for the catch, though!


Looks good; for those afflicted with the need for the JCE, a simple test can be found at https://gist.github.com/jehrhardt/5167854


Just curious - the reason for this change from 'types' to 'type'


is this temporary?
if not, does it have to be virtual? looks like a private method, I only see it used inside what looks like a constructor.


@Background möglich hier?


resolves #6


Added manually.


could we consider using string.equals, with a specific string comparison?


The two actions are auto-created.


what are opportunistic updates?


Reviewed PR and good with merge.


ErrorController is auto-created.


Feel free to add that to the doc :-)


Well, I dont know what was broken... can you sched some light please?


It has been.


Isn't `msg.realservice`  redundant if `msg.service` is checked?


This all was created using the command:
```
zf create project quickstart
```


Just read and do how it says.


This was added manually by me...


Referenciando Issues #20 e #21.


There is a #else in pomelo that builds it without SSE

NOTE, I can not build (at least on cygwin-64), without having SSE2.  When including windows.h the intrisic headers get included (deep down).  There might be some flag to cause different compiler behavior, but I am not too worried about it.


@tamird I think this addresses both of your concerns (at the expense of code clarity IMHO) without changing the message format. In any case, let me know how to proceed and I will squash and/or modify.


Sorry - been spending too much time in other languages recently, hadn't context-switched back.


A matter of consistency between the search filter parameters and these parameters.


Missing a wrapping <ul> tag


Added manually.


Missing a semi-colon on the ©


Sweet commit message Paul


comment this field? do we want to follow the same naming pattern for `file_flags` as the rest of the fields?


Perf library is needed for powerHAL and mpdecision.


I was unclear. Should we rename this to `Default` rather than `FallBack`?


final


Similar to jQuery UI's 'tooltipClass' - http://api.jqueryui.com/tooltip/#option-tooltipClass


hah, good point. Well at least submit a pull with the following reverted

```
-      - {{ name }}
+      - dovecot-{{ name }}
```


See https://github.com/libgit2/libgit2/issues/2113


I will later replace SQLITE with MySQL.


I'm wondering why the `join` method is missing in the API docs.
Wasn't this updated against 2.0?


There was an extra div for some reason - thats why all these divs are moved.


I replaced PDO_SQLITE with **PDO_MYSQL**.


I like `Fallback` - it is clear how it will behave


As commented at the standup, I am not sure you need the complexity of an async queue at all here; every time that a request to upload metrics is received, the set of data that will be uploaded is different. For that reason, assuming the server supports simultaneous connections, I don't think requests need to be queued, and could just be performed simultaneously.


These are empty now. I assigned values to them after I added this file to `.gitignore` for obvious reasons.


Why?


T.uber.com/qwer

You probably hate me right now

> On May 28, 2015, at 3:29 PM, Will Washburn <notifications@github.com> wrote:
>
> Sweet commit message Paul
>
> —
> Reply to this email directly or view it on GitHub.
>


this commit LGTM


the backtick is a new feature of ES6 template strings. https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/template_strings

As far as I know, the regular ' should not work.

I have something similar for webpack like this: https://github.com/tcoopman/boilerplate-webpack-react. It's also a bit out of date, but it's in better shape then this repo. There are many more starter packs that you can find on webpack if you search a bit.


I will patch (mostly done) the Starter so any metrics property that is not on MetricsOptions is passed to the JsonObject constructor and can be then used when building metrics options.


On parle de _uple_ ou _uplet_ en français, mais je pense que tuple est acceptable et plus courant. [source](http://fr.wikipedia.org/wiki/N-uplet)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2098


Just as the tutorial prescribes.


In case somebody read this, the changes here are to address AMO reviewer's feedback:

> After installing I got in the Browser Console following error:
NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [mozIStorageConnection.createStatement] vapi-background.js:234:0 (
(chrome//ublock/content/js/vapi-background.js)

However I haven't been able to reproduce this. If anybody can reproduce, please let me know, as currently I am working on fix in blind mode. In the commit here I am assuming that in the reviewer's case, the `connectionReady` flag was `false`.


nit: spacing after typeof


$28M. Agregaría el $.


There is a small conflict I introduced here from #359 (sorry) to provide minimum required protocol version for a DataType.  This is mostly needed for our tests when they run against older C* clusters.   If you could add a third argument as ProtocolVersion.V4 for each of DATE, TIME, SMALLINT, and TINYINT that should resolve the conflict (I think).


Please ensure that the code prints out the results as directed for each challenge.  FizzBuzz is correct.  Challenge 2 and 3 should also output a result to stdout.


There's still a divide by zero exception waiting to happen ... but the underlying code has changed quite a bit.  Should we still try to change the code as above or re-think the logic more broadly?


I was flip/floping on what tense to use, so I think should be just "...is downloaded", but thanks for the catch!


I don't think I need this file now that I am using a MySQL database.


to found


here is the test case, we can see that:

1/ MetricsOptions is modified to retain the JsonObject
2/ Starter is modified so the unrecognized `metricsProp=metricsPropValue` (no setter) is passed as a JsonObject entry

````
  @Test
  public void testJsonOptionsConfiguration() throws Exception {
    System.setProperty(Starter.METRICS_OPTIONS_PROP_PREFIX + "enabled", "true");
    System.setProperty(Starter.METRICS_OPTIONS_PROP_PREFIX + "metricsProp", "metricsPropValue");
    AtomicReference<MetricsOptions> metricsOptionsRef = new AtomicReference<>();
    ConfigurableMetricsFactory.delegate = (vertx, options) -> new DummyVertxMetrics() {
      {
        metricsOptionsRef.set(options.getMetricsOptions());
      }
    };
    MyStarter starter = new MyStarter();
    String[] args = {"run", "java:" + TestVerticle.class.getCanonicalName()};
    Thread t = new Thread(() -> {
      starter.run(args);
    });
    t.start();
    waitUntil(() -> TestVerticle.instanceCount.get() == 1);
    JsonObject json = metricsOptionsRef.get().toJson();
    assertEquals("metricsPropValue", json.getValue("metricsProp"));
  }
````

I believe that should solve the problem, right ?


PPP


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Do we really need this here? It's more of a computer-based problem than a tool for our service. How would you run the executable `certified-update` that this provides from the bundle folder where the gems from the `Gemfile` gets installed?


I think you need to run `oasis setup` and commit the changed files. Building from a fresh copy of the repo currently fails because it can't find the `Mparser` module.


About time for a new patch release with this just merged @dasilvacontin?


It's not :)
The reason is, because the services will be used like plugins, someone can send you the command "!!tgservice (whatever)", and the plugin will math. The realservice flag say you that this service message is really a service message and not a troll user XD

This is really interesting, because you can add some commands to configure the plugin that don't need to match the realservice, but other matches that need it (for example a pattern to configure a custom add_user message for a group).

I don't know if I'm explaining, but the TL:DR is to avoid people executing services messages.


this works only for string values though.


`guestbook` is the name of my database table.


Ok.. I've realized what you meaned to say... that msg.service can't be setted by the user...
Damn, it's true jajajajajajaj


TeamCity xBIM :: Xbim Geometry :: Release [Build 81](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1768&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 70 Build time: 03:24:30
### Failed tests
```
VSTest: GeometryTests.XbimModelExtensionsTests.CanGetRawGeometry: <no details avaliable>

```


woohoo!


Il y a une espace avant et après les deux points.


Ça semble intéressant comme langage. Il faudra que je tente un jour !


The sequence of library importing should be ascending.


Yeah, that looks like it would solve the problem of having to use a system property within the Dropwizard module. Thanks :)

I'll modify my code accordingly and resubmit.


Este párrafo es un poco confuso, porque mezcla "we" ("we went through...") y "I" ("I have a solution").

También, en la primera oración hablas de "make it better" (incremental improvement), y en la segunda de "share a problem" (i.e. something is broken and needs fixing).

Yo intentaría ver cuál era el intent del párrafo y volverlo a escribir.


@bradcray - note that this turned out to be the bottleneck (redefining the domain of the array). When I replace this by simply creating new arrays, that sped things up considerably.


@deads2k Here's a position where previously casting was done inside of the cache, now removed, but what to do with cast error (potentially) coming from Policies?


I used to use the term "n-uplet" everywhere, but I changed it to "tuple" when I realised that the latter was also used in French, including in [this book](http://www.amazon.fr/Programmer-en-Erlang-Francesco-Cesarini/dp/2744024457/ref=sr_1_1?ie=UTF8&qid=1432842093&sr=8-1&keywords=programmer+en+erlang).


you may want to squash these kinds of commits as they demonstrate you are learning src control rather than leveraging it to track and manage feature progress


Those changes have been made in the Unified branch of frenetic-lang/frenetic and frenetic-lang/frenetic-vm, which is due to be merged into Master fairly soon.


Can this be made a macro or better an API with a default value,  This is to have the application/client use the device of choice.


## Unit testing completed.

Wireshark was able to display the trap that I had sent. `VarBinding` has the `Test-Data` successfully

![image](https://cloud.githubusercontent.com/assets/7476271/7869372/e8c96a98-059f-11e5-89c5-c6763c3cdde4.png)


Can you expand on this? How does this relate to the idea in the previous sentence, and to the one in the next? Creo que le hace falta un poco de glue.


Not for mpdecision. Perfd actually uses the mpdecision socket, so it's the other way around.

I derped with this commit. I checked later on and saw that mpdecision was the only thing that was forcing 2 CPUs online.


Fixed that. Can you re-check the tests?


@caiosacramento coloquei na wiki do repositório o guia de estilo para o sublime text, dois dos itens que constam lá são ```"tab_size": 4``` e ```"translate_tabs_to_spaces": true``` que são referentes à tabulação do código a fim de melhorar a legibilidade. A wiki posteriormente terá outros guias de estilo, como de php por exemplo.


Hey @s-u, I know you're working on update.notebook stuff right now. Here's a change I'm considering in order to allow renaming notebook "folders". In general we don't have APIs for modifying notebooks that are not the current one - what do you think of this? It's on the `feature/folder-ops` branch, so no rush.

BTW, as an aside, if you do want to support forcing binary or non-binary, why not just add a flag to the `filename: { objects }` in the gist? It is very much like the language tag (only input, not output).


I have no idea how long a scan with a huge mod base will take, I was surprised after i had finished this tab in the last version. It was quite fast. But we'll see =)

Yes, i thought about custom sorting for the tab headers too, May be it will be similer to the column selection on the ModSelection.


`config/database.rb` -> `./.gitignore`
Leave in repo only a sample file: `database.rb.sample` with sample credentials.


This line is doing this:
```
SELECT * FROM guestbook;
```


> Here's a position where previously casting was done inside of the cache, now removed, but what to do with cast error (potentially) coming from Policies?

Not a cast error, just some kind of an error.  I'd chain it up and make sure that synchronization is retried at some point.


where do you refer this function?


Can we use a StringUtils method for this?


Maybe this API can be expanded to also specify the device if needed. i.e. client needs a different device from /dev/dri/card0


But I don't think text_search will ever be implemented for topics-- it should be removed.


Can we use StringUtils method for this?


It sounds like I don't understand the pattern well enough to explain it. I thought the example was written this way to ensure that the SyntaxError is produced before any code is executed. Is that not right?


Je pensais avoir fait attention à ça... Je vais mettre ma PR à jour.


Fixed here 29aca7e91a


You can use Amazon S3 to store images and use it with carrierwave (read its docs).


force full path without subdomain


This line is doing this:
```
"SELECT * FROM guestbook WHERE id= {$id}"
```


Argh aretha must've picked up 0.11 at the time you had it in the ppa,and
didn't downgrade of course. I assumed 0.11 was the mainstream Ubuntu
version...

OK, so I reinstall blitz -- how do I back out that commit?

Cheers,
Oleg

On Thu, May 28, 2015 at 7:43 PM, Gijs Molenaar <notifications@github.com>
wrote:

> Yes, i made a package with version which is 0.9 actually. This to fool the
> Debian package manager. Where did you get the 0.11 package? That should not
> be in the ppa anymore
>
> Op do 28 mei 2015 19:12 schreef F. Huizinga <notifications@github.com>:
>
> > I believe @gijzelaerr <https://github.com/gijzelaerr> did some rename
> > voodoo. @gijzelaerr <https://github.com/gijzelaerr> can you elaborate
> > what the case is here? Is 0.9 equal to 0.11 but for ubuntu 14.04?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ska-sa/meqtrees-timba/commit/59b784c88d2a90fd946c0253b3b418c783ea2c68#commitcomment-11411599
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ska-sa/meqtrees-timba/commit/59b784c88d2a90fd946c0253b3b418c783ea2c68#commitcomment-11412168>
> .
>


Super minor - lines 72, 74, 77, 78 could all make use of the ```get = Ember.get;```


Looks correct to me.


where is the retry loop?


what does it mean to "sell in education"? And I guess you mean "sold..", no "not sold..."


Looks good!   Running a local regression against some older versions, should know within 10 minutes or so how it goes.  Will post update when it finishes.


Old hash-syntax:
```ruby
:model => 'theme'
```
New hash-syntax:
```ruby
model: 'theme'
```


@fivetanley just curious why you reverted? Into which issues did you run? I couldn't find a ticket for that ...


fast-forwarding develop to borshop-integration-1121-bugfix/jrd/vnode_blocking = 2a863857


Git revert

Op do 28 mei 2015 21:48 schreef Oleg Smirnov <notifications@github.com>:

> Argh aretha must've picked up 0.11 at the time you had it in the ppa,and
> didn't downgrade of course. I assumed 0.11 was the mainstream Ubuntu
> version...
>
> OK, so I reinstall blitz -- how do I back out that commit?
>
> Cheers,
> Oleg
>
> On Thu, May 28, 2015 at 7:43 PM, Gijs Molenaar <notifications@github.com>
> wrote:
>
> > Yes, i made a package with version which is 0.9 actually. This to fool
> the
> > Debian package manager. Where did you get the 0.11 package? That should
> not
> > be in the ppa anymore
> >
> > Op do 28 mei 2015 19:12 schreef F. Huizinga <notifications@github.com>:
> >
> > > I believe @gijzelaerr <https://github.com/gijzelaerr> did some rename
> > > voodoo. @gijzelaerr <https://github.com/gijzelaerr> can you elaborate
> > > what the case is here? Is 0.9 equal to 0.11 but for ubuntu 14.04?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/ska-sa/meqtrees-timba/commit/59b784c88d2a90fd946c0253b3b418c783ea2c68#commitcomment-11411599
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ska-sa/meqtrees-timba/commit/59b784c88d2a90fd946c0253b3b418c783ea2c68#commitcomment-11412168
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ska-sa/meqtrees-timba/commit/59b784c88d2a90fd946c0253b3b418c783ea2c68#commitcomment-11414590>
> .
>


fixiki suschestvuyut


Won't work if you use rollback. `self.down` is empty.


fixiki huixiki


This exception catch is not enough to cover all cases. See next comment for example.


This line does
```
INSERT INTO guestbook VALUES bla bla bla;
```


You should remove the test above:
```ruby
it 'errors out stating "One or no line must match the pattern"' do
  expect { provider.create }.to raise_error(Puppet::Error, /One or no line must match the pattern/)
end
```


Good idea. Do you think Jason's HTML version is stable enough to reference from here? Or should I link to the PDF hosted on ECMA's wiki?


If `$fd` is defined, please close the original file descriptor, and set the fd of the environment variable to the number specified in the command line (i.e. if the command option is `--port=1234:0` then the environment variable should become `SERVER_STARTER_PORT=1234=0`.


Fair point! If you submit a PR to `dev` I'd be glad to accept.


I've changed it to InvalidOperationException. Let me know if this addresses your concerns.


This is already handled by the `before :each do` section earlier in the context.


I've created a ClientCredentialParameters type and have followed the same pattern as ClientSecrets. Let me know if this addresses your concerns.


Yeah. I think the pattern started because the table also includes `NotEarlyError`, which is not a function. A better approach would be to document that object outside of the table so that the entries can be more concise. I'll follow up with this shortly


I've moved the "magic" to the ServiceAccountCredential.Initializer's constructor. Let me know if this addresses your concerns.


This is handled by [a previous `before :each do`](https://github.com/CENGN/puppetlabs-stdlib/blob/55e80d5617c7af131e2602f84dc01f4129813390/spec/unit/puppet/provider/file_line/ruby_spec.rb#L140-L142) section.


One I wouldn't consider minor would be the fix to the behavior of descending stairs (the first sector no longer incorrectly lowers instantly). Other than that, I think you covered it pretty well. Maybe add "and friendly rebel behavior" along with the teleport beacon, as there were interrelated but separate issues there.


HTML for sure


I had the same thought, but I suspect that we will end up removing the possibility of overlapping network requests when tackling endlessm/eos-sdk#599. Otherwise, we will have to keep track of which data is associated with which request so that we know when we can delete it. A more ancillary benefit of only having one connection at a time is that each request will be more likely to succeed in this manner on weak connections, and we will generate less contention with higher-priority user-facing network traffic.


What happens if the api call fails and no 'upload_session_id' in the response? you will get a out_of_index exception, and unfortunately, no code to catch it.

You need to check whether these 4 fields are existing in the $res. If any of them missed, you should throw a well defined FacebookSDKException with detailed error message.


Ok so I downloaded a fresh debian network install image and created another virtual machine. Now it compiles without issues. No idea what happened to the other one but something got funky :-D


```
SELECT * FROM guestbook;
```


witam , tak ale to sa podane namiary na oprogramowanie ubuntu a ja mam windowsa


get code from here
https://github.com/floe/RF24/commit/c73baa7f40ad36530d1142f4495b80ae61319465


I'm not too worried about our "secret sauce". We have other public repositories and we're comfortable with them. Also, best practices and/or coding guidelines are universal; I think they're good to share and hopefully we'd get some community feedback as well.

>Is this README a better tool for this than a wiki?

I think so.


Are you sure you have to define this? I thought a variable ```current_user``` or something like it is provided by whatever authentication plugin/gem you use.


Was this to fix a regression in ExampleTheme?


... to create a new record in the guestbook.


Oh also, the description of the fix to the Stalker is actually backward. The problem was that they were making their active sound too often - they're supposed to tick constantly, but in vanilla this ticking has a higher priority than the active sound and will thus almost always immediately cut it off.


get code from here
https://github.com/repeat83/RF24/commit/caf699f7c15fee87752edd5507be1ea3e5682cce


Testing


This function is supposed to be in the User store?


This collapses a multiline paragraph down to a single line and removes extra leading whitespace.


No need to lookup ui_str anymore.  This method would be reduced to: `load_ar_obj(src[:ems]).try(:hosts) || []`


This puts each sentence on it's own line. You can use vim's `=` to fix indentation after the paragraph is formatted.


Testing Testing 1 2 3....


> There is a #else in pomelo that builds it without SSE

LOL, of course. I must be totally exhausted.

I know I could build 64-any some time ago but now we have new formats with trouble. The pomelo_plug.c were just warnings but then it fails in sunmd5.
```
sunmd5_fmt_plug.c: In function ‘init’:
sunmd5_fmt_plug.c:226:6: warning: variable ‘ngroups’ set but not used [-Wunused-but-set-variable]
  int ngroups = 1;
      ^
sunmd5_fmt_plug.c:225:12: warning: unused variable ‘k’ [-Wunused-variable]
  int i, j, k;
            ^
sunmd5_fmt_plug.c:225:9: warning: unused variable ‘j’ [-Wunused-variable]
  int i, j, k;
         ^
sunmd5_fmt_plug.c: In function ‘crypt_all’:
sunmd5_fmt_plug.c:538:97: error: ‘input_buf’ undeclared (first use in this function)
 #pragma omp parallel for default(none) private(idx) shared(ngroups, group_sz, saved_salt, data, input_buf, input_buf_big, out_buf)
                                                                                ^
sunmd5_fmt_plug.c:538:97: note: each undeclared identifier is reported only once for each function it appears in
sunmd5_fmt_plug.c:538:108: error: ‘input_buf_big’ undeclared (first use in this function)
 #pragma omp parallel for default(none) private(idx) shared(ngroups, group_sz, saved_salt, data, input_buf, input_buf_big, out_buf)
                                                                                ^
sunmd5_fmt_plug.c:538:123: error: ‘out_buf’ undeclared (first use in this function)
 #pragma omp parallel for default(none) private(idx) shared(ngroups, group_sz, saved_salt, data, input_buf, input_buf_big, out_buf)
                                                                                ^
make[1]: *** [sunmd5_fmt_plug.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [default] Error 2
```


Oh, you're totally right; I got that backward. Uh, maybe add a comment explaining what you just said :P.


Do we need to handle `src[:vm]` being nil here?


+4 indent


Is this line necessary? I think the plugin should get loaded by default.


could you name it as chunStartOffset? We will introduce parallel uploading in Graph api V2.4, which will introduce more offset name(such as partition_start_offset, partition_end_offset etc.) in this class after July 8th.


Shoule it be maxLen?


Yeah... it's still kinda buggy. On/off textures don't trigger when they should.


@elyscape That test would still apply if the :multiple parameter is not set to 'true'


@jk3064

you wrote the fixme, is the change correct?

at least it seems to work / doesn't crash.


Ok, I'm recommending this:
https://github.com/HowardHinnant/rippled/commits/hash-append
This sends all 16 `mHashes` at once as recommended above.  Additionally it makes `base_uint` contiguously hashable for Hashers with native endian requirements.  And for Hashers without native endian requirements it ignores the Hashers requirements for `base_uint`.  This eliminates the need for the `wrapped_hash_algorithm` which simplifies `base_uint::hash_append`.  And it gives SHAMap a significant optimization.


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Did you mean to make the server uri property construct only? Otherwise every time this setter is called, the previous value is leaked.


> What do you think about adding option ANDROID_NONCMAKE_BUILD_SUPPORT or ANDROID_EXPLICIT_CXX_FLAGS?

as per the comment in https://github.com/ruslo/polly/commit/243039895062501dd0c099ee16163863b71da189#commitcomment-11398172 we either do that, or have some  get_property-way of extracting those flags either in polly or hunter.

Should we really burden the Android toolchain with the decisions of having a `ANDROID_NONCMAKE_BUILD_SUPPORT` and then setting those flags? It looks quite big already....

As per the comment in https://github.com/ruslo/polly/commit/243039895062501dd0c099ee16163863b71da189#commitcomment-11398172 , this is a design decision that I would let you decide, since your CMake-fu is higher :-)


Yes. The issue is on Windows as it checks out files with CRLF endings. When download-deps.py copies the tools/framework-compile files, git status will show all the files that have been modified since they have only LF endings. Also, one or more files in tools/framework-compile are being deleted.


If you made the server uri property CONSTRUCT you could rely on the setter having been called already here.


I'm fine with not changing that behavior for now and keeping the network check.


This is definitely an in-memory search, so why not do it in place like I showed in the email I sent you guys about sorting? Instead of reassigning ```@items``` use  ```sort_by!``` like
```
@items.sort_by!(&sort_by_number_of_bids)
```
the method name doesn't need to be prefixed with a colon by the way.


Is this meant to protect `priv->uploading` when this function is called from multiple threads? How can it be called from multiple threads, since it's not part of a library's public API?


So, the test was added for an improper use case. In hapi, you do not pass query params as csv. As a result, the proper was to do this is to make your scheme use define it as 'multi' format and not 'csv'.


Is it correct to call `giterr_set_str` from a random background thread?


good


The first iteration of the `compute_fitness()` for loop handles this:

    for index in range(len(solution)):
        waypoint1 = solution[index - 1]
        waypoint2 = solution[index]
        solution_fitness += waypoint_distances[frozenset([waypoint1, waypoint2])]

when `index` is 0, `waypoint1` points to the last item in `solution` (index -1) and `waypoint2` points to the first item in `solution` (index 0). Fancy Python feature: http://stackoverflow.com/questions/11367902/negative-index-to-python-list


i'm an idiot... can't believe I missed this


yes, that is what I was trying to say! :)


no good, redo :)


r+


pkgname=geary-plank-bzr
pkgver=r1836
pkgrel=1
pkgdesc='The Pantheon Mailer- this package provides plank integration'
arch=('i686' 'x86_64')
url='https://launchpad.net/geary'
license=('GPL3')
groups=('pantheon-integration')
depends=('gmime' 'gnome-keyring' 'libcanberra' 'libgee' 'libnotify' 'librsvg' 'webkitgtk' 'libunity' 'dee-bzr')
makedepends=('bzr' 'cmake' 'gnome-doc-utils' 'gobject-introspection' 'intltool' 'vala')
provides=('geary-plank-bzr')
conflicts=('geary' 'geary-unity' 'geary-git')
install='geary.install'
source=('geary::bzr+http://bazaar.launchpad.net/~yorba/geary/trunk/')
sha256sums=('SKIP')

pkgver() {
  cd geary

  echo "r$(bzr revno)"
}

build() {
  cd geary

  ./configure \
    --prefix='/usr'
  make
}

package() {
  cd geary

  make DESTDIR="${pkgdir}" install
}

# vim: ts=2 sw=2 et:


ClientPrets.getInstance(..), the other method was intended for non-ui code or code that uses little or no android frameworks to avoid polluting it with android-specific frameworks unnecessarily. We should probably try remove that function it possible. Yes, this undocumented, sorry!


@moollaza @zekiel I commented out the part encouraging an actual PR for IsAwesome as well as a more consistent ending


saw approval from cmeiklejohn
at https://github.com/basho/riak_kv/commit/9a7576528d5fc73d51b49a567f8553b5f6c87ba0


merging basho/riak_kv/bugfix/jrd/2.1/vnode_blocking = 9a757652 into borshop-integration-1133-bugfix/jrd/2.1/vnode_blocking


basho/riak_kv/bugfix/jrd/2.1/vnode_blocking = 9a757652 merged ok, testing candidate = 0165611b


:D LGTM ...


saw approval from MonsieurNicolas
at https://github.com/jedmccaleb/stellar-core/commit/2ea0279f5a3e75360da1d160df9f5640a2deec6f


merging jedmccaleb/stellar-core/fetch = 2ea0279f into auto


jedmccaleb/stellar-core/fetch = 2ea0279f merged ok, testing candidate = 351ada6b


Remember DRY? Don't Repeat Yourself


If in the future you are going to add an element to a list that appears sorted, please keep the list sorted unless there is a good reason not to.

(There is no need to fix this now. One day we will do a housekeeping PR to fix things like this. But still, I feel it is worth pointing out the practice now to reduce the amount of future housekeeping needed.)


yes, I can make that clearer.


^^ that's correct!


fix: Flatten files array.

Closes #142


True. Not sure if it is best to specify that the grammar is the same, and then describe the semantic restrictions, or to expand the grammar to show only what is allowed. What do you think?


I just looked at the FreeType code to see what the difference between FT_Done_Library() and FT_Done_FreeType() actually is. The only difference is that the latter calls FT_Free_Memory on the `library->memory` member. If you called FT_New_FreeType() in the first place, this is fine. If you called FT_New_Library() (as Servo does) and provided your own `FT_Memory` struct (see FontContextHandle::new), then things could go wrong... though in Servo's case it doesn't, because that struct is allocating with libc::malloc.

So although this change fixes the use of mismatched API functions, it actually introduces a leak because the `FT_Memory` struct is no longer freed! Argh.

I think the proper fix would be to add a `mem` field to FreeTypeLibraryHandle. It would store the pointer to the `FT_Memory` struct passed to FT_New_Library. And then FreeTypeLibraryHandle::drop() can call libc::free() on that pointer after calling FT_Done_FreeType().

Jinwoo-Song, does that make sense? If not, I can write a patch that does what I've described.


You are correct, I will revise with use of the term "emitted whitespace" - I struggled a bit with how to explain this...


I think this is 2 changes.  If it was just testing the simplification, the values should have been the average of the old values.  These are not.  These represent new tuning.  So both new values and the simplification are being sent through.  We have no idea if the simplification is worthy.


yes


typo, an earlier version used ( ) around the arguments instead of | | (pipes). (i.e. it once ran :-))


@claudehenry dafuq is this syntax


So no way to have a good support of swagger using hapi :-( thx
Le 27 mai 2015 7:06 PM, "Trevor" <notifications@github.com> a écrit :

> So, the test was added for an improper use case. In hapi, you do not pass
> query params as csv. As a result, the proper was to do this is to make your
> scheme use define it as 'multi' format and not 'csv'.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tlivings/swaggerize-hapi/commit/bba1e650b2711d5fc8755951038824c709fd7684#commitcomment-11398327>
> .
>


On 28/05/15 02:19, Kenneth Heafield wrote:
> Missing a closing parenthesis?

Argh!  I was convinced I'd compiled it.  My apologies.

Jeroen


a


It has already supported big endian. The same method doesn't work on uClibc, is that caused by the differences between  toolchain themselves ?


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (625 of 625 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-5.22%25-green.svg)


翻译了文档，不错不错！不过需要改进以下几点
1. 数据库交互逻辑不应该写在modules里面的，modules里面是业务逻辑。数据库交互逻辑应写在dao里面
2. 应该用驼峰命名法，在ls的写法应为get-user-by-id（相应地就会编译成getUserById）
3. 交互逻辑的callback没有用到，而只用了console.log，应该进一步修改成业务逻辑层可用的借口


It's an annotation declaration. Like @Override, I declared @Legacy to put above functions that are only there for legacy purposes.


this example is based on intelliJ with jdk1.6


pry-rails does not depend on Rails and this won't replace irb with pry.


@skrub


Great job! why did not merge into  the master?


Way to make this professional, Mr. Plump.


xD


No difference from using the internal API directly: 14 seconds on mozilla-central.


should be `getNextId`, no?


formatting \o/


"...there is a orientation..."  => "...there is an orientation..."


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2126 of 2202 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-96.55%25-yellow.svg)


saw approval from cmeiklejohn
at https://github.com/basho/riak_kv/commit/c28a4b0529706291ab9ca7eae7a7fbef0b40197c


merging basho/riak_kv/bugfix/jrd/2.1/vnode_blocking = c28a4b05 into borshop-integration-1133-bugfix/jrd/2.1/vnode_blocking


basho/riak_kv/bugfix/jrd/2.1/vnode_blocking = c28a4b05 merged ok, testing candidate = c5cfbf65


"...has a orientation..." => "...has an orientation..."
Nesse caso o cenário teria que ser alterado também.


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/64352297


XD


rax retry tests


The weird thing is that i run rake and it gives me this:
107 scenarios (107 passed)
926 steps (926 passed)
2m3.627s


我才在想說那個一直 revert commit 的是誰，原來就是你...


Nope. It don't. But with this change it is not mandatory anymore. You can add it alongside with `jazz_fingers` and it will keep working as before.


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


fast-forwarding master to auto = af81db54


How to take advantage of this application,for example in Secure E-mail system?And how to handle this system?I am eagerly looking forward to the author's help


Its just does not add to have it as a direct dependency. If it is not a rails project, the gem `pry-rails` does not do much.
This is somewhat to solve an old issue to when the project were maintained @nixme and it was called `jazz_hands`.

The irb replacement i belive we just gotta have to play with it a little more, am i right?


Is there a reason that this is being notated this way

```
+    PERSONAL = 0
+    ORGANIZATION = 1
+    EVENT = 2
+    SOFTWARE = 3
+    BUILDING = 4
+
+    NAME_TYPE_CHOICES = (
+        (PERSONAL, 'Personal'),
+        (ORGANIZATION, 'Organization'),
+        (EVENT, 'Event'),
+        (SOFTWARE, 'Software'),
+        (BUILDING, 'Building'),
+    )
```

instead of say

```
NAME_TYPE_CHOICES = (
+        (0, 'Personal'),
+        (1, 'Organization'),
+        (2, 'Event'),
+        (3, 'Software'),
+        (4, 'Building'),
+    )
```

There probably is some reason that I'm not understanding.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2089


asd


@teamoflosers : This is kinda cool


Começou. Adicione-me como colaborador


what the hell git


doc/命名及书写规范.md

更新路径


I think I know why.  Nevermind.


hi,
i have used excatly the same command to create the Self-signed Certificate but still have the same Error with IP SAN.

openssl req -x509  -batch -nodes -newkey rsa:2048 -keyout logserver.key -out logserver.crt -subj /CN=SVV-LOG01(Logstash Server Host name.)

thanks in advance


check existing first.


rax retry tests


Never mind, I see that this is necessary to have the provider use the new resource.


saw approval from engelsanchez
at https://github.com/basho/riak_kv/commit/c5af124d72a172032c99eb1bc05c37ece0575bfe


merging basho/riak_kv/bugfix/jrd/blocking-vnode-teardown = c5af124d into borshop-integration-1136-bugfix/jrd/blocking-vnode-teardown


basho/riak_kv/bugfix/jrd/blocking-vnode-teardown = c5af124d merged ok, testing candidate = 26169756


Rationale in the PR description would be good for history.


1. consts go after the class.
2. shouldn't these be of the right type?


Oh, I see what you mean.  Never mind.


@caiosacramento dá uma olhada no guia de estilo para css na wiki do projeto. Essa classe deveria ser ```.div-logo```


Extra =


mesmo caso da ```.div-logo```


THis vs This


Memory footprint is definitely *not* improved. Still waiting to see how speed is affected.


Missing semi-colon


OK, that was surprising.  Why not use what we always always use for this - 'while (true)`?


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-87.68%25-red.svg)


weeee


Thanks for joining the discussion @daveFNbuck, I really hoped for your input since you've been contributing so many related patches. :)

It's as you say something like O(#tasks x #workers) when you have the "all workers subscribe to every task && [cond](https://github.com/Tarrasch/luigi/blob/8d79b9aadd33b0c8a1155c70c99e7162c72a9737/luigi/scheduler.py#L749) is false" scenario. This patch doesn't solve that. Also, it's improbable so lets just say that get_work is O(#tasks x log(#tasks)) (there's a `sort` in there, but let's ignore the log from now on). This still bad enough for us. The reason is I would guess that #workers do `get_work` all the time and it takes O(#tasks) time. I can look up tomorrow exactly what numbers we have on #workers and #tasks.

With this patch, in "trivial" cases as described in the commit message, the complexity is just O(#tasks-for-worker) which is usually much much lower than O(#tasks) in our graph.


I cherry picked some of the changes to my local repo. The coordinates are better, and at least the x & z seem to be spot on. However if I come back from the campaign briefing to the map selection, the camera is little bit tilted to where I came from. I wonder if is the cinematics or are the engine paths not identical (the return paths).

Great work! Judging by your comments you can actually "debug" the original game?


this is wrong. we can not hardcode success. The uploadFinish maybe failed.


This could just go inside the while loop's if statement, so you could use return and get rid of the break (which just takes you right to this spot anyway...)


Hi @rhysd ! So cool that you did this! I also started doing this some time ago, as I told you, but never finished it. How was your experience? Did you find many annoying things in Crystal that we can improve?


This is way over 80 columns.


why the extra spaces here?


The `src_vm_id=` method is defined only for the MiqProvisionRequest model.  This will break other request types.  I think it needs to be moved into the MiqProvisionWorkflow.


check the 'success' field from $res here. And throw FacebookSDKException if the post failed.


Done!


That is correct.


Note that if you remove this, you'll need to change line 231 to:
```ruby
expect(File.read(@tmpfile).chomp).to eql("foo1\ninserted = line\nfoo=blah\nfoo2\nfoo1\ninserted = line\nfoo=baz")
```


We should consider initializing primitives.  Most places we do already.


Created: https://github.com/couchbase/couchbase-lite-java-core/issues/643


Huh?  Shouldn't this parameter be a boolean already?  Why are we accepting it if it's a string saying `"true"`?


Here too.  Do we really want to accept a `bool` value here, for example?


Is this configuration?


bueno, thx


_Please_ make this a named constant.

Also, consider `return (tx_type_ == ttAMENDMENT || tx_type_ == ttFEE) ? 0 : defaultTransactionFee;`


Normally, we don't do retry immidiately since that might be increasing the server load. You can sleep for 500ms or 1s before next try.


I added this to see if I could get a better understanding of my error messages.


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Where does ```user_id``` come from?
What is this method intended to return?


As a side note, it might be a good idea to change [the context name](https://github.com/CENGN/puppetlabs-stdlib/blob/55e80d5617c7af131e2602f84dc01f4129813390/spec/unit/puppet/provider/file_line/ruby_spec.rb#L204) from `'with two lines matching the after expression'` to `'with multiple lines matching the after expression'`.


Please change to 4 space indent


`return requiredFee? getTransactionFee().mantissa() * baseRef / required_fee : 0;`


I'm confused why there is is ```current_user``` method and then another one taking a parameter. Why is there commented out code?


Need a wrapping ul


extra =


duplicated


This method should be called only on Windows https://github.com/gitextensions/gitextensions/blob/43740235b79631fb8a7265c09e160482247a1993/GitCommands/Utils/EnvUtils.cs#L8-L20


Also I will start creating a simple wiki & changing the read me we now have. So that it is easier to contribute to the project and understand it. Currently the instructions are appalling and many issues are too big & general.

It would be easier to process the data by pull requests etc. Now it would be just impossible to merge your branch since it would also change the project name and so much logically different changes in one commit/branch. So I'll just cherry pick these now.


missing ;


duplicted


I see. You only retry for transfer. Make sense. But you need to define this variable with more clear comment.


Außerdem sind die Elemente in denen die "Geschichte" und der "Regex Vorgang" stehen auch immer noch nicht so gestyled wie die content Klasse... Die Ecken haben ein andere Rundung (weniger pixel) und die Hintergrundfarbe ist auch nicht richtig - vermutlich beides Standardwerte von Bootstraps.


I have seen that at line #237, the bulkRevsToPull is synchronized so I guessed that there could be multiple threads accessing the bulkRevsToPull variable. Does this mean that we need to do synchronize here too for getting the size of the bulkRevsToPull.


Removed the "http_basic_authenticate". Only registered users that are logged in and admin can create new articles. Only admin can edit or delete an article. Included the ApplicationHelper because I needed to call methods in both the users view as well as the index view.


FYI, I could use some advice on what more can go in the description here.


@bfcoder Thanks for the response James, in the latest commit of this project I cannot find the `token()` function in the `client/js/stores/user.js` file. I'm I looking in the wrong place?


Tell me when you are ready for another round review.


Added these flash messages just because I like them haha


NOT TESTED!!!!


All of these patches are going to get removed again later in the pull request - in other words, there is no need to be picky here.


Is this actually going to work in light of /root/RESULT being on an LV?


I kept them primarily for reference.


Oh man why are you working on ELM??


I see, you created the synchronized list.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2100


Missing ul tag


I'm trying really hard to find the sign_in method, where does this come from?


IMHO the enumerations that are used in myhdl are a convenient way to represent a set of integer numbers, but using text as mnemonics. But if you don't like, I can remove __index__ from the push.


man, this code just screams for tuple assignment (release, pkg_type  = 'centos7', 'rpm') (I know, not this change, but...)


Will development remain active on github?


ok i was being real picky...let me finish up in 15 minutes then

From: Patrick Ohly [mailto:notifications@github.com]
Sent: Thursday, May 28, 2015 1:13 PM
To: pohly/meta-intel-iot-security
Cc: Whiteman, John L
Subject: Re: [meta-intel-iot-security] meta-integrity/linux-yocto: include patches from ima-control-experimental (bf75cc1)

All of these patches are going to get removed again later in the pull request - in other words, there is no need to be picky here.

—
Reply to this email directly or view it on GitHub <https://github.com/pohly/meta-intel-iot-security/commit/bf75cc1e4a861725b42fa0047c4c490fc312f80f#commitcomment-11415031> .  <https://github.com/notifications/beacon/ALGQ_UmROrXiLWpF5CjQIYvC2hD0UhcYks5oN25LgaJpZM4EuCkn.gif>


the label needs an id on the form element to be associated with it. :)


If we don't need it, can we delete this line?


Placeholder doesnt work on a file upload.


missing ;


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


I *think* that `modprobe` is linux only, meaning these tests might fail once we get to running them on solaris/aix/etc.


The checks for vm_id being zero and svm being nil can be combined now that the raises are moved.  I would remove the whole `if vm_id.to_i.zero?` condition and also use `.where(:id => vm_id)` instead of find_by_id.

```ruby
rails_logger('get_source_and_targets', 0)
svm = VmOrTemplate.where(:id => vm_id)
if svm.nil?
  @vm_snapshot_count = 0
  return @target_resource = {}
end
```


Unnecessary whitespace in this line.


Ok


This line has a trailing space.


Does it work?


yes!


:candy:


What about this:

build/core/base_rules.mk:154: *** packages/apps/PhoneCommon: MODULE.TARGET.APPS.Contacts already defined by packages/apps/Contacts. Arrêt.

? :/


No man, i remember that put all fields raise a error with exclude_fields.


indent


:baby:


This function should be wrapped in a retry loop, since it's pretty common for the Set call to fail (it will fail if anyone else is changing metadata at the same time).


These are the methods I need for both users and articles view


Missing ul tag


missing ending h1 tag


missing semicolon


If we're iterating over all the objects in _typePlaying we can just use this.
```haxe
for (sfx in _typePlaying)
```


Here is where I am hiding the buttons unless you are an admin


@mgorczyca Thank you for replying. My point is that the underlying `request


-> https://github.com/pyllyukko/user.js/issues/9#issuecomment-100468785


This is to hide the delete user button unless you are an admin


Why are you returning this optional thing?


I wouldn't expect a method (`extend`) to return the parent class (`results`). That would be weird, wouldn't it? That would mean it would be:

```
def extend(self, args):
    ... do something with args ...
    return self
```


Apparently I still have uncommited code. :/ I'll try to get that pushed up soon. In the meantime, you can add your own.

```
token(){
  return _user.token;
}
```

Underneath the `loggedIn()` method. And you would set the `_user.token` when the user signs in.


Give me the link


My last update


Thanks, fixed.


This regex only matches a string which contains:
- word charters (```[A-Za-z0-9_]```)
- dots (```.```)
- or dashes (```-```)

The order doesn't matter.


First


The test checks that puppet *do* throw validation errors. That's why it fails when puppet is not installed.


Fixed good call.


Also synchronized the whole block in the next commit.


(rebased per in person discussion.)


This validate method is different then the others "validate_*" methods which are designed to be called for dialog fields and therefore take a common set of parameters.  I do not see this method being used that way.  I would not expect it to take the same set of parameters and would suggest it use a different name to set it apart as well.  Thinking something like `source_valid?` and it would get the VM ID similar to the get_source_and_targets method.


A good fix Martin! Thanks!


Why
```
if install_mime is True:
```
Just
```
if install_mime:
```


Specifying `python3` explicitly is not needed, `${python3:Depends}` will pull it.


plz remove


Honestly I forgot why I was trying to associate the comment with the article....I'll rollback that haha I think I ment to add a User id to the article and for some reason was going ahead and adding a comments one as well


Warunek w tej linii zadziała ci dla każdego obiektu typu Option z jakąś ustaloną wartością. Pisząc Some(item) deklarujesz nową zmienną "item", która chowa "item" będący parametrem itemsPositions.


this change breaks use of --without-libstdc in configure command obviously because of TRACEPRINTF


Why did you change the div id?


We can do it :) Next try with you :neckbeard:


Just install the files directly into python3 path, no need to symlink.


Oops this accidentally caused a bug where drawing on the canvas was 15 px off. Will fix.


this should be why you were getting those errors. I forget to name the second one "other_user"


abc - nie nazywaj tak zmiennych.


stdint.h is only support from VS2010:
http://herbsutter.com/2012/05/03/reader-qa-what-about-vc-and-c99/

So I've made the #include conditional in stdafx.h.
If you start using uint32_t etc then please add these typedefs to stdafx.h


nice work, Michelle!


No need to wrap in list() when iterating.


Done, along with .screen-share-video.


The same.


Por que mudou o id da div?


Sorry for the delay, was doing some debugging, everything looks great except for a few issues with the tests expecting a Date but getting an int instead.  (see: https://github.com/datastax/java-driver/pull/355/files#r31273139)


nice job, David!


:goat: :heavy_check_mark:


Nie kopiuj klas, nie powielaj tego samego kodu.


Remove double brackets.


Foresterdev see https://github.com/TrinityCore/TrinityCore/issues/14791 but cant be commited until instance header data is sorted out as extra conditions are also needed for this commit and probably script to save state on death of involved bosses too but that is being worked on but needs testing.


Hi,

Why do you multiply score by 10 here?

Thanks


just curious. what is this key used for?


Re u/ branches. Yes. I realised that but one downside of user branches appearing within the repo is that Github notifications arrive without any weighting as to importance. This is why a case could be made for it to be cleaner for people to fork the repo they are working on, work on the fork whenever and then only make it visible to the main repo via a pull request.


You could probably speed up the average case for most workers even under priority use by cutting the iteration off once a best task is found. We then do your optimization to count the number of pending workers, which probably accounts for most of the work most of the time.


Renamed _mockPublisherContainer to _mockPublisherEl to make it consistent with the other two.


OK, those are good points, even though you could solve the first issue by just stealing the data while transmission is in process, like it works today, and putting it back at the end when the upload fails.
In any event, I think the code could be changed to use a simple `GQueue` because I am not sure you need the locking protection for multi-threads that `GAsyncQueue` gives you  (same comment I had for the mutex).