The actual terms that git uses are: `worktree` and indeed `refspec` (as a term on its own, since it is actually not a "reference specification" but rather a "transformation logic that can be applied to one references in order to get another one" - i.e. it describes the relation between the reference names in two repositories).

`worktree` is more or less consistently used in git (i.e. the config setting `core.worktree` and the environment variable `GIT_WORKTREE`)
But in general, this concept is also known as "checkout" or "working directory".

The correct term then would be "workingDirectoryPath" (We're using camel-case everywhere; like Qt) - contrary to the snake case in libgit2 (`working_directory_path`). However that name's clearly too long (just like `reference specification`).

> The actual problem about the confusion is: Documentation didn't state anything before :smile:.

agreed.

... and in addition: I didn't think the thing that I was worried about at that moment completely through: I was actually looking at the submodule refresh code - nearly at the same time that I discovered this commit. The submodule refresh code uses `workdir()` to find existing submodules (and subtract them from a list, so in the end the list contains all "removed" submodules).
I was confused, because in that scenario it is unacceptable to use a method that possible returns an empty string. But I overlooked the fact that a checked-out submodule is guaranteed to posses a worktree (Otherwise it is actually not there)


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/1103


`--mkpasswd` should prompt the user on terminal for password, so it does not appear in files like `$HOME/.history`.


Done


Fair; I could use `atom_to_binary(X, latin1).`


Romulo Furtado


Adding that single line to the edit course table view was the most tedious thing to figure out. This line allowed the courses to actually be updated when the user returns to the app.


You could do have a private method since the implementation of markdown_title and markdown_body are identical. So, something like this:

```ruby
def markdown_title(markdown)
   self.markdown_text(markdown)
end

def markdown_body(markdown)
   self.markdown_text(markdown)
end

private

def markdown_text(markdown)
  renderer = Redcarpet::Render::HTML.new
  extensions = {fenced_code_blocks: true}
  redcarpet = Redcarpet::Markdown.new(renderer, extensions)
  (redcarpet.render markdown).html_safe
end
```


This breaks client! Thanks for six hours of my life :-(
Any examples? Current not works.


Would prefer to define the constructor out-of-line in the `.cpp` file.


Ah. In general, we would only want to return verified credentials (certificate or DN) to the application. Granted, the application will be configuring the SslContext and could specify InsecureTrustManagerFactory.

It seems what you're wanting could be achieved by using an X509TrustManager or similar (and Netty's SimpleTrustManagerFactory makes that less annoying). Is there an advantage to this PR's approach?


It was that it included a major update to ampersand-sync which had breaking changes (I think https://github.com/AmpersandJS/ampersand-sync/pull/18 and https://github.com/AmpersandJS/ampersand-sync/pull/19). Most of the discussion around it was in #10.

And speaking of this, we really need a unified changelog strategy for Ampersand :grin:


I don't have any better ideas right now, but we should be careful about throwing around "digital repository" (in the same way I feel bad about the repository vs non-repository labels in the UI). The distinction we should try to make is "stuff that exists somewhere else" and "stuff that is exhibit-specific". Add to that, there's actually nothing in the code (other than an i18n string and a configuration key) to do anything with items in a digital repository.


Should call this `parse_vector` since vectors are the basic compound data type in this language.


I am happy to hear that and I really appreciate the effort for their work.
Thank you very much.


90693826-Central-cooling-coils-not-autosizing-to-correct-airflow-in-some-cases-#4810 (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1666 of 1673 tests passed)

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


@liushaog
if (urls == null || urls.isEmpty()) {
         	int i = path.lastIndexOf('/');
         	String category = i < 0 ? path : path.substring(i + 1);
         	URL empty = consumer.setProtocol(Constants.EMPTY_PROTOCOL).addParameter(Constants.CATEGORY_KEY, category);
             urls.add(empty);
         }

-------------------------------
if (urls ==null ||...)  //true
{
     ...
     urls.add();
}

^_^


Since all of these clauses are checking whether `cur.type` is equal to some integral constant, it would be better to use a `switch` statement.


`warnings.warn` only prints the first time the line is executed (unless CPython is started with some specific -W option, the PYTHONWARNINGS environment variable is set, or the warnings filter is switched programmatically), so this warning would only be printed the first time a function that doesn't depend on its in put is encountered (i.e. the first time in the interpreter process). It might be better to use a print statement instead.


ok


Done


There's the tape in there from BS12 if you update. Also, I'm sure Nienhaus
would help with making some sprites.

On Mon, Mar 23, 2015 at 3:36 PM, Thomas Stoeckert <notifications@github.com>
wrote:

> They work like tables ATM, I'm working on more stuff
>
>
>
>
> > On Mar 23, 2015, at 4:23 AM, Graham Lloyd <notifications@github.com>
> wrote:
> >
> > I really want this to happen
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10353136>
> .
>

--
Graham Lloyd


On second look, I think you need to address the line-breaking issue.


Move to 0.100


@mvl22 this should schedule the cron job for the planning application.


Per quanto riguarda il fatto dell'oozing come mai lo hai rimosso??
Se c'è il runout, che a mio avviso è inutile, perchè non tenere anche quello che tanto male non fa?
Stavo anche per fare il pull per delle modifiche per evitare che a forza di retrazioni il filamento venisse completamente estratto...
Scusa il disturbo,
Simone


This one stayed, and that cause the build failure on the build server: because this file is added 2ice. Once explicitly and second time via wild carded entry. This needs to be removed.


Why are you doing this with tostring, super messy?

```
var jsonFormatter = context.InputFormatters.Where(t => t.GetType() == typeof(JsonInputFormatter)).FirstOrDefault();
```


Was it the intention to change the logic here? The new line is not a full inversion of the old one because ```and``` has not inverted to ```or```.


I saw that culture_group issue on the forums, but I haven't gotten to test it yet. I'll probably do that when I work on the culture related event elements from PB, which is up next.


Hello World this is my 1st github repository and I


yup


You should print here either `Image` or `From`, where the latter has higher precedence when both are specified, IIRC.


=> void init_reg_file_inode(struct regular_file_inode* , char* file_name , struct dir_inode* parent_dir);

This interface is more universal and clear.


develop (macumber) - 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)


Not the file it gone :-)


The construction of the import line could probably be extracted into a method to make this already long `write_imports` method a bit shorter.


=> void init_dir_inode(struct dir_inode* , const char* name, struct dir_inode* parent);


It seems like most of this is pretty much copied from the request tests. Is there any way we could DRY this up? I'm thinking write two functions with the same interface that make requests through requests and urllib3 and then parameterize the tests by those functions.


Except 382 links to 384 now, @ctb


:)

On Mon, Mar 23, 2015 at 11:26:05AM -0700, Andy Lulham wrote:
> I am not not OK with it. (I am OK with it.)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/frabcus/mpcv/commit/52b1777d769d2c08be63a936b65c264fa70a8792#commitcomment-10351769

--
Help get every MP candidate's Curriculum Vitae!
http://cv.democracyclub.org.uk/


Also, I made all the changes but still keep getting a syntax error, unexpected end-of-input, expecting keyword_end on line 27 of reddit.rb? I pushed my code again. Thanks


> This should've rather been added to a major version change as its a change to the dependencies.

It's technically not a change. Boris has required this extension from the very start, it was just forgotten from the composer.json.


Oh, what did I do?


POSIX types are not allowed in FS tree lib interface. FS tree lib should abstract details of its implementation.
You could move this function out of FS lib interface to fix this.


````
    InboundTransactions() = default;
    InboundTransactions(InboundTransactions const&) = delete;
    InboundTransactions& operator=(InboundTransactions const&) = delete;
````


+1 problem too all the above. The name doesn't tell me anything about this function.


What it does? :)


~~Why didn't you just make one commit for the past two changes you made?~~ -- I realised the answer to that question before I had even asked it.


Non interface function. Or should be changed to work with struct inode parameters.
The same for next functions.


long line


I expect Parks will want to provide their own text for this message and that they won't want blockface IDs, but this seems fine for now.


Having the issue of error with unexpected token because combobox.js is in JSX in the npm_modules. Did rackt merged your PR? Right now it seems that I can't use react-autocomplete of rackt. Is there a workaround? Should I use yours? did you upload it to NPM?


Con sto caspita di github non ci sto capendo nulla... Ti spiego, la cosa dei watt era interessante e non incide sul funzionamento, mentre quella dell'oozing volevo verificare prima che tutto andasse bene con il singlenozzle... Per questo l'ho tolta.. Tanto ora ce l'ho e posso vedere se non fa pasticci. Quel sistema serve per evitare che il multi hotend faccia perdere plastica all'hotend non funzionante... e' vero che tanto si può disabilitare, ma volevo esserne certo... Tutto quì.. Anzi ti ringrazio per l'aiuto....


Like you do that in `DescribeSTIStrategy` couple lines below. Btw. why it is twice, I know one is for Build and the other one for BuildConfig, maybe we should DRY both of these as both print BuildParameters.


did you consider:
`def guess(actual, actual, _), do:` ...


Ja, eigentlich schon, da es mehr ein Verweis als eine Quelle ist.


Good. Also please don't copypaste compiler flags (CFLAGS).
Use -o $@. It's rather simple and clear.


Oh whoops! I didn't notice that and messed it up. I'll push a followup commit.


Who would free the memory?


I had the same thought. I'll push a follow-up commit.


They are **MASSIVELY** different commit :wink:


I can't believe I did all of these commits under Anthony.Westover@Pearson.com


seg_to_bez_svg is returning the whole curve with each iteration instead of just the segment necessary.


Issue, https://github.com/openshift/origin/issues/1418 @bparees  you can assign that to me.


Si non c'entra nulla ma infatti sono due cose diverse ho semplicemente fatto un unico pull
Comunque si effettivamente non ho aggiungo le eccezione per il single nozzole.
Ma se tipo intanto lo lasci scrivendo che è da testare nei commenti?? così almeno una volta che hai qui i file nella repository corretti posso farci le modifiche e sistemare tutto.
Thanks


Wouldn't this only enable coverage for C code?


#oops #willfix #tenure


TeamCity xBIM :: XbimExchange :: Build [Build 134](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1187&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 134](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1187&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Compilation error: Xbim.COBieLite.SerialisationHelper\SerialisationHelper.csproj Build time: 00:00:16


Thank you, I've noticed this piece of code and I haven't got any clue what to do about that :+1:


You might be right.  I honestly can't wrap my head around it at the moment but I've gone back and forth in my head 3 times over it now and am doubting myself on it.


Some complexity that come into play here is returning the number of lines changed (or is it number of imports changed?) to the caller. Maybe we can drop that complexity and instead return true/false for whether the import was added?


I used casting inside a de-referenced pointer to get the actual object (*iterator) ====> sf::Vector2f


Not strictly speaking the same standard, but I wonder why mingw32 needed gnu++0x anyway?


E' quello che sto cercando di fare...

Date: Mon, 23 Mar 2015 13:10:05 -0700
From: notifications@github.com
To: MarlinKimbra@noreply.github.com
CC: magokimbra@hotmail.com
Subject: Re: [MarlinKimbra] Fix Power Consumation (3223901)

Si non c'entra nulla ma infatti sono due cose diverse ho semplicemente fatto un unico pull

Comunque si effettivamente non ho aggiungo le eccezione per il single nozzole.

Ma se tipo intanto lo lasci scrivendo che è da testare nei commenti?? così almeno una volta che hai qui i file nella repository corretti posso farci le modifiche e sistemare tutto.

Thanks

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


why not just let them open it but cancel picking up items from it


`using u256_acq_pair = ...`


Should be "namespaces"


You're not using this `event` in the next couple of lines, so I don't see any point in assigning it here.


Which means this wasn't working before :)


r+


You *may* be able to remove this - I don't see how it was working before, so removing and the person who tries this can add it back in once they figure out what they want....


Hi Niels,
Thanks - that's indeed a much nicer solution. I've corrected this. :+1:


This overlaps with code that @ncdc had for exec - we should find out how to have only one copy of it.


This too maybe.


It looks like this will ALWAYS run the previous exception handler, which for most people I don't think that's what they want. If you simply need to custom handle errors, just use `Flight::before('error', ...)`. Alternatively you could use `restore_exception_handler` to restore one that you've set yourself.


develop (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1164 of 1166 tests passed)

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


Let's not save full path in name field. Let's only file name. When you save full path you duplicate information which is owned by file parents. This duplicated information may easily become stale (invalid after change). Just let's remove this hard wiring and only what we should do when we are inode - save our name.


Well, right, I meant `-std=c++0x` there...


Capital D.


This is a bug fix where the row highlighting doesn't extend the full row.


Teste


Hi Diaman, I have checked your file and compared it to the spec, everything looks in order,however there is just one little change you will have to make please, the threadObject you take as a parameter according to the spec, you should  not get it as a parameter, but get it as an object returned from the getThreadAppraisal function when you call it with the ALL keyword. But other than that it looks good thanks a lot.


````
, m_gotSet (std::move(gotSet))
````


Right; I saw your change there, and deleted the others after rebasing, but somehow this one slipped through.


#2230 noticed a problem with this.
We are in a loop here, so after the first iteration `*file` does not need to point to `""` anymore. It can point to a freed memory address that in the next call to `expand_wildcards()` is reused and the following `assert()` could complain.


This version gets text of HTML page and saves him in file.


1. Избавиться от "магических" названий файлов в коде (сделать конфиг, смотреть в параметрах командной строки)
2. Вынести создание StreamReader/StreamWriter за пределы класса SearchInGraph, в класс Program, т.к. иначе получается, что класс SearchInGraph жёстко завязан на эти стримы и их невозможно подменить их на другие стримы
3. Подумать об исользовании структуры, объединяющей was, distance, Person - чтобы был один словарь вместо трёх разных, с одинаковыми ключами.


Typo of `Confimration`


Yeah, in fact it could just be a `for time < startTime + waitTime`, and return the timed out error if you fall through the loop. Saves two channels and a goroutine


Yeah, it seems to be returning the number of imports, which now is not the same as number of lines. If things need to know number of lines, we'll need to update that as well I think. Otherwise, I like the boolean.


Or maybe you could make it protocol-agnostic, which would be more correct

```
@import url("//fonts.googleapis.com/css?family=Lato:400,700,900,400italic");
```


Where is the %h:%p -W parameters getting populated?


Is this a comment? or left over from something?


missing colon here


or?


In che modo il RUNOUT dovrebbe prevenire la perdita? Da quello che vedo nel codice ogni 30 secondi estrude una tot quantità dall'estrusore attivo e basta.
Scusa ma è per capire, se non capisco per cosa è stato pensato non posso darti una mano a sistemarlo :+1:


I'm not sure, but I have not yet had an opportunity to relearn how Perl does this.  Since we can override the default, I'm not sure how big of a priority this should be.

However, there was another issue tied into the discussion on IRC.
The guy we were having the conversation with was using a non-standard prefix, which naturally puts the perl modules outside the path contained in perl's INC.  He was claiming that previous versions of zoneminder have inserted the non-standard path to his perl modules into INC.  However, I have never seen this, nor do I see where that can even occur.... has to happen in makemaker is my guess.


It looks like making it truthy/falsy is good enough, we'll just have to update these two lines in `import`:

```ruby
      return unless lines_changed = import_one_variable(variable_name)
      window.cursor = [current_row + lines_changed, current_col]
```
to something like

```ruby
      return unless import_one_variable(variable_name)
      window.cursor = [current_row + 1, current_col]
```


@munificent or @kevmoo another change to review, please.

This is test code for the streams/ tutorial. It's not intended to have pretty output, just to be runnable.


SSH's ProxyCommand templater does that.


Decomposition within the Gang Garrison 2 universe is interesting, in that the alpha value of skin cells and other matter decreases until it hits zero, then the matter ceases to exist, unlike decay in this world.


@CaptainElliott Originally this was written for chests. When they open a chest, it appears open in game. It's been mostly just cut-and-paste since. Feel free to make a PR if you want to improve this.


On Mon, Mar 23, 2015 at 10:05:44AM -0700, pbrunet wrote:
> I wonder if this code is staticaly typed? There are two possible return type which are not "mergeable".
> Ok, it may be tricky for users to understand this but concider the code:
> ```
> def foo(a):
>     if a:
>         return a[:2]
>     else:
>         return a
> ```
> with `a` a numpy.array
> For boost.Python, we have to know at compile time what is the type of the returned value and in this case, is it is a, we will return the pointer on a, otherwise, we should return a numpy_sliced_array with the a pointer. So it is not known at compile time.

you're definitively right on this one.


That's not exactly right though, because the new import can be 2 lines now. I think we'll be better off returning the difference.


Seems to work well now.


If the intention was to only negate the old line to
 ```if (! (!$app->isAdmin() and ($error->getCode() == 404)))```
it must be
 ```if ($app->isAdmin() or ($error->getCode() != 404))```
This will early exit on case of Admin or non-404 and only continue with the rest of the function in case of 404 on frontend which would match the comment above.


Oh, of course


CBECC (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2043 of 2157 tests passed)

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


What is that for?


Yikes, it took me a while to see this, but yeah, that's nasty. A lesson about reusing variable names!


хохохо


Gracias, no me había dado cuenta ;)

Para la próxima intenta acordarte de incluirlo en el título también, será más fácil encontrar las cosas.


`MutexType` is a better name than `LockType`.  We need to start avoiding confusion between locks and mutexes.


Prefer `using` in these three places.


Thanks Erik


Things compile with `lock_guard`, therefore it should be preferred (easier to read and ever so slightly more efficient).


Sure, I have not done any research, only simple look and it looked usable. I wondered if you thought about this earlier before pursuing this idea...


We would have to increase Human.getMaxStupidity()


@blanquer, here is your XML parser/generator code rebased onto my branch, with tests. As agreed, we don't register the XML handler by default (that's up to the app).


:+1:

I pushed a fix to the mock buffer here: https://github.com/trotzig/import-js/commit/127f7e5e2da46d67669f3f89c429cf69311a90dc


This fixes the only failed test in the previous commit.


I have no clue.  I will ask asciidoctor folks on twitter about it.


@blanquer, here is www-form encoding represented as a kind of handler, plus media-type-aware request body parsing.


Talvez um objecto: canDelete: true fique mais expressivo


You should report that if it was unusual.


r+


Ahh, I see, then you'll need to change how this works... because as it is right now the 'account' in your public method is not getting changed.  Could return a tuple instead, a bool and an Account object...


Ok perfetto, allora attendo  che sistemi e poi continuo a lavorarci su.
Se hai bisogno di una mano, son sempre disponibile.
Simone


I don't know how that moved. Done!


I should also say that since there is no pika in epel for rhel7 one would have to manually make and install the rpm to satisfy the dependencies. Alternatively the dependency can be removed from the spec file and pip or some other tool can be used to install pika so that shoal will play nice.


Done.


CBECC (kbenne) - 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 you need the number to be adjacent to "fixes" if you want GitHub to mark the issue as fixed


oyoyoyoy


Hai sul tuo github un fork completo con tutte le modifiche?Adesso è un casino recuperare tutti i pezzi!!!

Date: Mon, 23 Mar 2015 13:29:48 -0700
From: notifications@github.com
To: MarlinKimbra@noreply.github.com
CC: magokimbra@hotmail.com
Subject: Re: [MarlinKimbra] Fix Power Consumation (3223901)

Ok perfetto, allora attendo  che sistemi e poi continuo a lavorarci su.

Se hai bisogno di una mano, son sempre disponibile.

Simone

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


I dont have the solution for those... I though it's usage was more like some throw away experiments or for the IPython notebook.


should be - tip
See https://github.com/tmilewski/goenv/blob/master/.travis.yml


Keep the return type annotation.


I'm going to personally nit for `true`, though this doesn't *really* matter.


changed with width and height to 50%


Done. Thanks for the review!


I already did all my tape stuff, what did bay do this time?

> On Mar 23, 2015, at 4:00 PM, Graham Lloyd <notifications@github.com> wrote:
>
> There's the tape in there from BS12 if you update. Also, I'm sure Nienhaus
> would help with making some sprites.
>
> On Mon, Mar 23, 2015 at 3:36 PM, Thomas Stoeckert <notifications@github.com>
> wrote:
>
> > They work like tables ATM, I'm working on more stuff
> >
> >
> >
> >
> > > On Mar 23, 2015, at 4:23 AM, Graham Lloyd <notifications@github.com>
> > wrote:
> > >
> > > I really want this to happen
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10353136>
> > .
> >
>
>
>
> --
> Graham Lloyd
> —
> Reply to this email directly or view it on GitHub.
>


That's a great idea. It's probably also possible to do a better job of guessing container dimensions by traversing up the DOM.

> On Mar 23, 2015, at 2:10 PM, Adam Schwartz <notifications@github.com> wrote:
>
> What about retina detection?
>
> —
> Reply to this email directly or view it on GitHub.
>


Perhaps add:
```ts
else {
    Debug.fail(..)
}
```


What about some standard rules? http://www.restapitutorial.com/lessons/restfulresourcenaming.html

I'll go with safe, tested template:

GET /users - fetch all users. But wait. If I'am authenticated user, should I get all existing users? Nope. Delete this API.
GET /users/{key} - get user details with key = email
POST /users - create new user if key = email doesn't exist
DELETE /users/{email} - restricted to "me" for now
POST /users/{key}/someCustomEntityChangingAction


Terima kasih notifikasi nya @diagramatics, sudah diganti


Not Year 8 banter; @BadgerBoy1002 banter...


It is some throw away experiments.
How can I set the db-dir?


You need d801 kernel? Just download it from my mirror.
I build for all models.
Linaro compiler, dont care about model or firm.

Its made for cortex a15 neon vfp.


How's this? http://docs.chef.io/analytics_monitor.html#zookeeper


This will have to be revisited once computed properties are allowed in binding elements


https://github.com/simone97/MarlinKimbra
Però non sono sicuro ci sia tutto
nel caso comunque ho in locale una versione con tutte le modifiche e funzionante
Fatta eccezione delle ultime modifiche che hai fatto oggi
Te la allego?


I could see someone customizing the docker agent (to work around a bug, or handle a corner case, or integrate with local custom software); maybe make this an "advanced" parameter?


try this:

    script:
      - cd $HOME/gopath/src/bosun.org/
      - go test -v ...


The verbiage for Zabbix and New Relic look good, though I'd recommend being a little more specific for New Relic and calling it a "hosted service" instead of just a "platform".


try rsync -az ${TRAVIS_BUILD_DIR}/ $HOME/gopath/src/bosun.org/


If you haven't done so, please add a test where you emit an array binding pattern with empty elements. Also, put a comment that says we intentionally do nothing if we hit an empty element in an array binding pattern.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/91bcaaa0379ee47f3cc7e4831a60a2a9cb1bc979/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/91bcaaa0379ee47f3cc7e4831a60a2a9cb1bc979/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/91bcaaa0379ee47f3cc7e4831a60a2a9cb1bc979/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/91bcaaa0379ee47f3cc7e4831a60a2a9cb1bc979/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/91bcaaa0379ee47f3cc7e4831a60a2a9cb1bc979/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


We could, but it would require parsing the selector.  I had a concern that doing so might make the code less reliable.


Si perché in quel brach non c'è

Date: Mon, 23 Mar 2015 13:34:29 -0700
From: notifications@github.com
To: MarlinKimbra@noreply.github.com
CC: magokimbra@hotmail.com
Subject: Re: [MarlinKimbra] Fix Power Consumation (3223901)

https://github.com/simone97/MarlinKimbra

Però non sono sicuro ci sia tutto

nel caso comunque ho in locale una versione con tutte le modifiche e funzionante

Fatta eccezione delle ultime modifiche che hai fatto oggi

Te la allego?

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


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1115
success: http://build.servo.org/builders/linux2/builds/1115
success: http://build.servo.org/builders/android/builds/1149
success: http://build.servo.org/builders/mac1/builds/1328
success: http://build.servo.org/builders/mac2/builds/1331
success: http://build.servo.org/builders/gonk/builds/297


For TM_4_0_Design/issues/492 - sorry, the link is wrong in the commits


I signed the CCLA


Thanks! :smile:

Yeah possibly, although that will get very tricky, especially if, for example, someone puts this file (or the Eager embed) before a stylesheet within the head. With respect to that note I think what you’ve done already is about as good as we can probably guess without testing on various sites. The only addition I can think of right now might be maxing the height/width with viewport height/width.


a, ...c (missing comma)


This whole thing seems ripe for refactoring, maybe later.


Delay construction of lambdas for guards of *condacts*


So you could either return a tuple, or just make a new class that represents your response (the class has 2 properties, a bool and an Account)....  Or you could just go back to naming the method RefreshAuthTokenAsync and have it throw (either rethrow the exceptions it catches or throw a new exception type RefreshAuthTokenFailedException or something like that).  You and Sean can work it out but my vote is for the latter.


Why do we omit the dotdotdot?


thanks to neo2shyalien for empty MAIL var update


These values are pure guesswork and their main benefit is that they are as small as possible and they work.

If we could set the hash table size to 1, then this test would be by far easier to do.

Come to think of it, I will maybe change this so as to test the list directly...


`ZooKeeper is a centralized service that is used to maintain configuration information for Chef analytics, including the queue that moves actions data from the Chef server to Chef analytics.` is incorrect. RabbitMQ is the queue that sits between Chef Server and Analytics.

Apache Storm uses Zookeeper (sometimes written Zk) to coordinate internally. This is what the "Alaska Pipeline" is build on top of. This includes the rules system.


fast-forwarding master to auto = 5ce7d8ac


Can you either assert that it's an identifier, or check it via `else if`?


This is not correct. GuidIndexSize is the size of a reference to GUID heap (in bytes - it's either 2 or 4).

Since GUIDs accumulate on the GUID heal (unlike other heaps) metadataSizes.HeapSizes[(int)HeapIndex.Guid] already accounts for the GUIDs of the previous generation. Thus the fix is even simple: just delete "_previousGeneration.GuidStreamLengthAdded".


As d


Shouldn't you test this with cmhPP?


I think that it would be better to show the info about new methods that will replace deprecated ones in the documentation rather than display it directly.


Questa è quella che ho in locale anche con qualche altra piccola modifica.
PS: I file di configurazione devi rimettere i valori di default, perchè ora ti sto allegando quelli della mia stampante.
http://www.mediafire.com/download/6wvasg1aka9yzxm/MarlinKimbra.rar


2 public classes in the same file?


If we run once before committing, I think we can avoid these type of issues.


Line is too long. [101/100]


Extra empty line detected at block body beginning.


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/1163c6b699347c5424b67e723e663ed4fe23e587/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/1163c6b699347c5424b67e723e663ed4fe23e587/appconfig.json)


Unnecessary spacing detected.<br>Trailing whitespace detected.


The first argument to describe should be the class or module being tested.


Line is too long. [102/100]


Avoid parameter lists longer than 5 parameters.


Use empty lines between defs.


Extra blank line detected.


saw approval from jdm
at https://github.com/frewsxcv/servo/commit/2764ec0f7bae00c79a754acd68c93983aed93421


merging frewsxcv/servo/handle-message-str = 2764ec0f into auto


frewsxcv/servo/handle-message-str = 2764ec0f merged ok, testing candidate = f62ab247


Foram removidas as tabelas por algum motivo?


@tonydspaniard

I found mistake.
The problem was that the js files with languages locales must be include after bootstrap-datepicker.js.
So in DatePickerLanguageAsset.php in $depends variable need to add: 'dosamigos\datepicker\DatePickerAsset'

DatePickerLanguageAsset.php will be:

namespace dosamigos\datepicker;

use yii\web\AssetBundle;

/**
 * DatePickerLanguageAsset
 *
 * @author Antonio Ramirez <amigo.cobos@gmail.com>
 * @link http://www.ramirezcobos.com/
 * @link http://www.2amigos.us/
 * @package dosamigos\datepicker
 */
class DatePickerLanguageAsset extends AssetBundle
{
    public $sourcePath = '@vendor/bower/bootstrap-datepicker/dist/locales';

    public $depends = [
        'yii\web\JqueryAsset',
        'dosamigos\datepicker\DatePickerAsset'
    ];
}


when docs arrive be sure to give @ef4 some credit :)


Consider adding this to the schema as optional parameter.


This local is virtually unused.


great, thanks James!


One easy way to spell this is `return {};`.


Seems weird that y is required here.


This is wrong, `KILL` should only be sent to the user directly.  Everyone else, including the user, should see `QUIT`.


I agree that the case statement is less than ideal here, especially since it's trying to be a "one liner" when it really isn't. Adding another case for jruby, topaz, or whatever Ruby implementation comes along can quickly make this list painful. I'd prefer to urge those using older Rubinius versions to just update to the latest version that _does_ define `RUBY_ENGINE_VERSION`.


@spongebolt there is no need for `yii\web\JqueryAsset`, I fixed version 1.0.4 :)


Why do we need these labels, anyway?

Also, why are there whitespace changes mixed in to this commit?


It was a trailing line of a deleted comment. This function was tested thrice and then a pull request was created. Apparently, some changes as required by the comments left the trailing line. I will be more careful henceforward.


@cihancoskun mevzu ne :)


If msg.Line is []byte, you needn't cast it to string explicitly.


I think I like it better with `...w`. One reason is because if it were originally `[x, z, w]`, then the type would be `[any, any, any]`, and not `any[]`.


please make samples!

sample of bad match, sample of good match. put it inside three-backticks so github would format it differently.

you mean css as in someone writing it in issue description? or css as in eventum's own css styles?


Perhaps use CSS here instead? (text-transform)


That's more of an issue with the types though. I can file a bug.


  {
    "remote": "THLG",
    "revision": "cm-11.0-old2",
    "repository": "android_hardware_qcom_display-caf",
    "target_path": "hardware/qcom/display-caf"
  },
  {
    "remote": "androidarmv6",
    "revision": "cm-11.0",
    "repository": "android_external_chromium",
    "target_path": "external/chromium"
  },
  { "remote": "androidarmv6",
    "revision": "cm-11.0",
    "repository": "android_external_webkit",
    "target_path": "external/webkit"
  },
  { "remote": "androidarmv6",
    "revision": "cm-11.0",
    "repository": "android_packages_apps_Browser",
    "target_path": "packages/apps/Browser"
  },
  { "remote": "androidarmv6",
    "revision": "cm-11.0",
    "repository": "android_frameworks_webview",
    "target_path": "frameworks/webview"

this repos are needed to patch webview


git commit "Clone student-work to c:\all\hello"
git commit "Clone java to c:\all\jwork"


I used the 'fire-and-forget' unlock API, which was touted as the preferred endpoint for most apps.
https://developer.android.com/reference/com/google/android/gms/games/achievement/Achievements.html#unlock(com.google.android.gms.common.api.GoogleApiClient, java.lang.String)

We should fully implement a more robust achievements piece thought :)


Looks like it happens just with subcontainer with more than 2-3 sections. For example doesn't happen in Advanced options or Footer.
And when you expand a section which makes appear the sidebar, it looks fine.. how to handle all this? js ok but how? :D


Long line.


Suggested:
```
WriteLog (lsTRACE, InboundLedger) <<
    "Got data (" << packet.nodes ().size () <<
    ") for acquiring ledger: " << hash;
```


interesting that AR has still that much of overhead there ... thought they nailed this one already (maybe in 4.2)


I definitely like the direct display. You run some code, stuff barfs out... kinda hard to 'miss it'. I do agree that having the same information about deprecated calls should also be in the documentation. If you're up for it you could do another PR for that :)


:cry: :cry: :cry: :cry: :cry: :cry: :cry:


@Xethron Boris never worked without it.  Also, it probably should be a "require-dev" and not installed on production.


You probably don't need anything besides the `beforeModel` right? The other stuff is unnecessary.


I agree with @brifordwylie about direct messages. Having it in both places (code+docs) is the best way.


Hi! After this change, a lot of mistakes, is not added to the parent of a child!


I Should get an email about every fatal error as they happen. Will modify this to send an email on EVERY ERROR.. but not yet. Still got a few less-serious errors to fix first.


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1660 of 1667 tests passed)

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


1)Co robi metoda check ?
2) Postaraj się do każdego commita dodawać jakąś wiadomość, która w jakiś sposób opisuje to co zrobiłeś(na razie wszystkie nazywają się 'first_commit'). Możesz nazywać np. 'Dodanie metody check' albo ' Add check method'


Yeaaahhhh! First commit. Welcome to LeanIX :-)


Oh come on! This is not fixing, this is walking the other way to not see the badness!
https://github.com/javalite/activejdbc/issues/381#issuecomment-84316516

This test might be testing if Base class is truly stateless I think. And if DB class has no shared data...
Because everything here is thread local, for sure not concurrency or thread safety...


I'm very glad to see this project, I was hoping somebody would do exactly this!


Heads up:  This is going to conflict with PR 940, and to resolve these will need to become `vector`.  When they do, `reserve` them with `packet.nodes ().size ()`.


You really should start using  my premium version.  All this "drive detection" stuff is handled in a much better way, automatically, with Android's ```ActivityRecognition``` API, which intelligently aggregates low-power accelerometer & gyroscope to stream a ```DetectedActivity``` to you at a configurable sample-rate.

```STILL```, ```WALKING```, ```IN_VEHICLE```, ```ON_BICYCLE```, ```ON_FOOT```, ```RUNNING```.

https://developer.android.com/reference/com/google/android/gms/location/DetectedActivity.html

And the battery performance of everything is 1000% better -- my phone can run for days with plugin running.


I missed this one. Thanks!


Updated the PCB for self-production,
pros:
- reduced footprint,
- pulldown resistors for MOSFETS,
- added USB 5V power option, made jumper fool-safe,
- minor changes,
cons:
- compatibility with extension board is BROKEN!
- removed flashing connector,
- more vias,
- not tested yet.


Aha, because we're talking about pseudo elements here, not pseudo selectors.  It might help to clarify the API here:
assert.dom.pseudoElementCss: function(expected, propertyName, selector, pseudoElement);

this matches the existing:
assert.dom.css: function(expected, propertyName, selector)


Pessoal, uma dica: não commitem arquivos específicos que façam referência a IDE como o caso do "nb-configuration.xml". Esse arquivo acaba atrapalhando em alguns casos. Procurem commitar só o que é relativo ao projeto


Add 33 to the `enums` above?


@tonydspaniard

Sorry I didn't see it. Now working fine )))


I've opened #2469 for this issue.


Shoot-  this notification went to my personal account, will add a new commit.


Can you make a screenshot, to be sure we have the same understanding of 'very tiny'? :-)
2nd: that's a known issue, thanks!


Hi.... Did you published it in a maven repository?


Nice


![screen shot 2015-03-23 at 2 02 19 pm](https://cloud.githubusercontent.com/assets/5577205/6790493/42a511e4-d165-11e4-9860-ef105aee6148.png)
text-align: center causes inconsistent styling in the dropdown list. And it looks good without centering the text. Can you make a quick fix for this?


Hey Chris,

I've been doing heavy modifications on your free plugin for an app I'm
working on. I'm interested in your premium version, but would need to get a
sense for what the code looks like, as there were a few objections I had to
the free version code, and I wouldn't want to spend $500 without verifying
that the code won't require heavy modification on my end.

Can we schedule time for a code review (can do over screen share), or maybe
you can give me 1-day of private repo access so I can review, and I can
sign your license/privacy agreement in advance so that you're protected
from me pirating your code (which I won't do).

Thanks,
Steven

On Mon, Mar 23, 2015 at 4:55 PM, Chris Scott <notifications@github.com>
wrote:

> You really should start using my premium version. All this "drive
> detection" stuff is handled in a much better way, automatically, with
> Android's ActivityRecognition API, which intelligently aggregates
> low-power accelerometer & gyroscope to stream a DetectedActivity to you
> at a configurable sample-rate.
>
> STILL, WALKING, IN_VEHICLE, ON_BICYCLE, ON_FOOT, RUNNING.
>
>
> https://developer.android.com/reference/com/google/android/gms/location/DetectedActivity.html
>
> And the battery performance of everything is 1000% better -- my phone can
> run for days with plugin running.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/steaks/cordova-plugin-background-geolocation/commit/280dee84987ed6e35cafade7acef41ef1281b2d0#commitcomment-10354606>
> .
>


@sdroege Bad news, I have just realized the design is not going to work well for MJPEG. The problem is that, at least for some cases, notify::caps is only called after the first 200 OK reply has been sent. It looks like the caps negotiation is not done despite of the pipeline being in the GST_STATE_READY state!

I tried entering GST_STATE_PAUSED in main, instead of GST_STATE_READY, and it seemed to work fine for my "videotestsrc" tests. However, when I tried with v4l2src, the same issue came up.

I don't know how I missed it the first time I was testing it. With webmmux is not a big deal, but with MJPEG it makes firefox try to download the stream and save it to a file the first time it connects. Subsequent connections are fine. The test pipeline is:

http-launch 8080 videotestsrc ! jpegenc ! multipartmux name=stream

Is there a way to force caps negotiation without entering GST_STATE_PLAYING?
But, in any case, shouldn't negotiation consistenly happen when the pipeline goes into GST_STATE_READY?


The likelihoods are the "reciprocal of the residual sum of squares"? What does this mean? Is it valid in general?


Hm ... I don't understand how this can have a side effect; the changes should apply only if the given index is negative, which always resulted in an exception before.

Can you give me a code example of one such exception? I must be overlooking something.


W login.jsp stworzyłem formularz do wpisania loginu i hasła.Check w założeniu ma sprawdzać poprawność wpisanych logów - ale to chyba trzeba jakiś skrypt napisać


Sorry @PVince81 Accedently changed directly in core. Should I make a PR instead?


> Does the native python that comes with MSYS2 (pacman ....) gets installed in a path containing whitespaces? This seems very unlikely. Where do the whitespaces come from?

The handling of directory names is different depending on how python is built. The correct version to use with 'waf' requires some native path handling that is not available in the pacman package.

> I think requiring an external python installation is very cumbersome. I'd like to avoid this requirement if possible.

Me too.


Add .DS_Store


`mutable`


develop (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2099 of 2196 tests passed)

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


Should be a `const` member function.


You're welcome @dasilvacontin! :D


@andy-armstrong How about a helper method for getting the response code? Seeing this same line of code in about 6 places makes me feel slightly ill.


Hey Victor,

thanks for the fast reply, just made a screenshot and uploaded to http://pl.vc/qi5w
Sorry for the double-post, I couldn't find a issue about it :-(

Sascha


With this technically nice refactoring, you **overwrite/undo the changes from the pull request of issue #99** in commit `73d5bf1`. Please redo the changes, that accept directories within the file filters (if you changed it unintentional).


@jamietsao Here's how I'm translating from index to the enum


Here's the default value


Works fine :-)


I'm using the distancesArray to translate from index to enum


I think calling `expand_shellcmd()` with `*file` set to `NULL` is okay. It should not matter anyway as long as this is not the only pointer to allocated memory.

`expand_wildcards()` either calls `mch_expand_wildcards()` and `*file` is set to `NULL` initially, or
`expand_wildcards()` returns `FAIL` and either `*file` was not touched or `*file` was set to `""`.
If `expand_wildcards()` returned `OK`, `*file` can always be freed. Otherwise no `free()` is done.


@andibraeu ich glaub Kommentare sind in JSON nicht erlaubt. Nicht, dass uns da irgendwelche tools fehler schmeißen!


i nie wiem gdzie ona powinna byc wywołana - wydaje mi sie ze po kliknięciu SUBMIT


`override`


C'è un errore se inserisci la funzione oozing controlla la variabile filament_changing che però è definita solo se si setta il FILAMENTCHANGEENABLE che si setta se hai un lcd...
Insomma se inserisci l'oozing senza lcd da errore bisogna verificare come aggirare il problema, per ora definisco la variabile a prescindere..

Date: Mon, 23 Mar 2015 13:41:46 -0700
From: notifications@github.com
To: MarlinKimbra@noreply.github.com
CC: magokimbra@hotmail.com
Subject: Re: [MarlinKimbra] Fix Power Consumation (3223901)

Questa è quella che ho in locale anche con qualche altra piccola modifica.

PS: I file di configurazione devi rimettere i valori di default, perchè ora ti sto allegando quelli della mia stampante.

http://www.mediafire.com/download/6wvasg1aka9yzxm/MarlinKimbra.rar

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


okay, i guess


wouldn't be "size" more convenient?


The method generated doesn't need the generated class. It could (should ?) be in the API jar.
It will make the code simpler and reduce the amount of code generated.


Yes.


is there anything preventing me from doing

```rust
struct Pair<A,B> { ... }
impl<T> Drop for Pair<T,T> { }
```
?


Related to #1


I'm having a look at this.


This can be `const`.


`const override`


Would it help if S2SRO data weren't included in the percentages at all? If they are essentially not reducible it makes no sense to include them in the statistics. As written it's a confusing statement because it's already been stated that SRO are not included. PS I think International time was canned in order to make space for the legacy surveys.


this seems ok (in the sense of conservative), it'll probably do the wrong thing around associated types in some cases -- that is, we likely want to normalize! -- but we can also leave that logic aside for now.

it feels like we have to work really hard to set this impl up. Grr. I wonder if it makes sense to do a more implied bounds approach just for this specific case. Probably too late to consider that.


offset is "%"PFMT64d, not %d


develop (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)


ur sure that blackrock battle worgs should drop multiple items ? for me it looks more like a reference loot so there is only 1 item dropping out of that pool.


"unknown"


I'll reiterate: eww :)


I found the problem! This is due to Feathers LayoutGroup

override public function addChildAt(child:DisplayObject, index:int):DisplayObject
		{
			if(child is IFeathersControl)
			{
				child.addEventListener(FeathersEventType.RESIZE, child_resizeHandler);
			}
			if(child is ILayoutDisplayObject)
			{
				child.addEventListener(FeathersEventType.LAYOUT_DATA_CHANGE, child_layoutDataChangeHandler);
			}
			var oldIndex:int = this.items.indexOf(child);
			if(oldIndex == index)
			{
				return child;
			}
			if(oldIndex >= 0)
			{
				this.items.splice(oldIndex, 1);
			}
			var itemCount:int = this.items.length;
			if(index == itemCount)
			{
				//faster than splice because it avoids gc
				this.items[index] = child;
			}
			else
			{
				this.items.splice(index, 0, child);
			}
			this.invalidate(INVALIDATION_FLAG_LAYOUT);
			return super.addChildAt(child, index);
		}


I use it like this:

```javascript
{
  pattern: 'articles/:date/:title',
  formatter: function(date){
    return moment(date, 'DD.MM.YYYY HH:mm', 'de', true).format('DD-MMM-YYYY');
  }
}
```


hm, da kannste recht haben :-) geht glaub ich erst ab draft4


section?section->name:""


`override`


This is the result anyway:

![image](https://cloud.githubusercontent.com/assets/4511254/6790805/8aff27b4-d1aa-11e4-8827-e73be145b485.png)


oh ok. you are using ret as token to check if section is null. it's fine.

use fullname "unknown" instead of "unk"


Please return back to the code, this change should be done in conjunction with Feathers.


@dudekm gettery niepotrzebnie przyjmują jakieś zmienne


s/ret/vaddr/g


Тая, от 21 строки и до этого места нужно выпилить, я считаю. Это подпорка под конкретнуб реализацию поиска по доменам.


This can be `const`.


What I mean is that you won't have access to the arguments or the description here. There is no way to pass it from the validate method to the getDescription() method. So you are forced to explain all the reasons why this specific validator may fail.


Thanks - spotted this just before dinner too :) I see it's ended up in the cyclescape user's crontab automatically which is awesome.


`const override`


Ok, that makes sense.


ok done


They added tape and tape rolls for sticking paper onto things. It
definitely wouldn't be hard for you to add the tape stuff that you were
working on to their system. It's in the master repo right now if you want
to check it out.

On Mon, Mar 23, 2015 at 4:32 PM, Thomas Stoeckert <notifications@github.com>
wrote:

> I already did all my tape stuff, what did bay do this time?
>
>
>
>
> > On Mar 23, 2015, at 4:00 PM, Graham Lloyd <notifications@github.com>
> wrote:
> >
> > There's the tape in there from BS12 if you update. Also, I'm sure
> Nienhaus
> > would help with making some sprites.
> >
> > On Mon, Mar 23, 2015 at 3:36 PM, Thomas Stoeckert <
> notifications@github.com>
> > wrote:
> >
> > > They work like tables ATM, I'm working on more stuff
> > >
> > >
> > >
> > >
> > > > On Mar 23, 2015, at 4:23 AM, Graham Lloyd <notifications@github.com>
> > > wrote:
> > > >
> > > > I really want this to happen
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10353136
> >
> > > .
> > >
> >
> >
> >
> > --
> > Graham Lloyd
>
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10354077>
> .
>

--
Graham Lloyd


This causes failed build for Kore
http://jenkins.kodi.tv/view/Android/job/Kore/19/console


For my C++ culture, what is the rationale behind the kind of syntax used here (test1 -> continue, test2 -> continue, test3 -> continue, do stuff) as opposed to a more classic test1 && test2 && test3 -> do stuff?
Is it to optimise the operation cost, or a convention to make the code easier to read?


Como les comentaba chicas, el profesor Alejandro.. ¡es el mejor!


Downloading and starting the selenium server was key here! Don't rely on gulp-mocha-selenium to do it for you, even though that works on your local machine.


We should probably hide the font import from SVGFonts.
I'm not sure how to handle this in Xml2Html.hs (line 201)?
@byorgey


The thing is that Liferay is a multi-user system. So in a heavily used environment there will be potentially many users trying to create/update a user account with a wrong email address. So the first will cause the validate method to be executed and with your code the _description and _arguments fields will be set. Then right after a second user might cause the method to be invoked changing those values. This can happen so fast that the error message hasn't been shown to the first user and by the time that happens it will show with the description and arguments of the second user. The more simultaneous users the more weird behaviors would arise. The only way to really provide this information would be through a return object that would contain it, but I'm not sure if it's worth the effort.


`override`


Added because there is no other way to just defer filtering.


It has the same API as free version.  There’s little going on now on
Android side, since the HTTP and SQLite persistence layers have been
removed.

When a geolocation is received, the plugin sends it off to your Javascript
callback, just like iOS.

- The ActivityRecognition system is always running (default sample-rate 10s)
- When the activity of ON_FOOT, IN_VEHICLE, ON_BICYCLE, RUNNING are
detected: *GPS ON*
- When STILL is detected:  *GPS OFF*

It doesn’t currently send the detected-activity to Javascript, but that’s a
30min job.

The meat of the plugin consists of ~300 lines-of dead simple code within
the CordovaPlugin file  There are just 2 files (plus a binary lib,
eventbus-2.4.0).  There is no more giant BackgroundLocationService.java

You would migrate all your application intelligence *out of the Android
code* and into Javascript.

On Mon, Mar 23, 2015 at 5:04 PM, Steven Wexler <notifications@github.com>
wrote:

> Hey Chris,
>
> I've been doing heavy modifications on your free plugin for an app I'm
> working on. I'm interested in your premium version, but would need to get a
> sense for what the code looks like, as there were a few objections I had to
> the free version code, and I wouldn't want to spend $500 without verifying
> that the code won't require heavy modification on my end.
>
> Can we schedule time for a code review (can do over screen share), or maybe
> you can give me 1-day of private repo access so I can review, and I can
> sign your license/privacy agreement in advance so that you're protected
> from me pirating your code (which I won't do).
>
> Thanks,
> Steven
>
> On Mon, Mar 23, 2015 at 4:55 PM, Chris Scott <notifications@github.com>
> wrote:
>
> > You really should start using my premium version. All this "drive
> > detection" stuff is handled in a much better way, automatically, with
> > Android's ActivityRecognition API, which intelligently aggregates
> > low-power accelerometer & gyroscope to stream a DetectedActivity to you
> > at a configurable sample-rate.
> >
> > STILL, WALKING, IN_VEHICLE, ON_BICYCLE, ON_FOOT, RUNNING.
> >
> >
> >
> https://developer.android.com/reference/com/google/android/gms/location/DetectedActivity.html
> >
> > And the battery performance of everything is 1000% better -- my phone can
> > run for days with plugin running.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/steaks/cordova-plugin-background-geolocation/commit/280dee84987ed6e35cafade7acef41ef1281b2d0#commitcomment-10354606
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/steaks/cordova-plugin-background-geolocation/commit/280dee84987ed6e35cafade7acef41ef1281b2d0#commitcomment-10354765>
> .
>

--
Chris Scott
Transistor Software <http://www.transistorsoft.com>


Hi Avi,
I'm a little confused on how exactly to use this endomorphism interface with the sudoku case rules. Should I override it when initializing the `caseRuleSelectionHelper` object in the `getSelectionHelperMethod` of the `CasePossibleCellsForNumber` class as I did below?
```java
final CaseRuleSelectionHelper crsh = new CaseRuleSelectionHelper(p) {
    public Endomorphism<Point> normalizePoint = new Endomorphism<Point>() {
        @Override public Point apply(Point p) {
            BoardState state = Legup.getCurrentState();
            int w = state.getWidth(); int h = state.getHeight();
            Point q = new Point(p);
            if(q.x == w) { q.x = -1; }
            if(q.y == h) { q.y = -1; }
            if(q.x > -1 && q.y > -1) {
                \\if the point is not an edge then normalize it to its subgrid
                q.x = q.x/3;
                q.y = q.y/3;
            }
            return q;
        }
    };
};
```
When I ran this, it seemed to have no effect on the program. Am I on the right path or should I be going in a different direction?

Thanks!
Mitchell


It is totally fine. :)

You can find the compatibility table here: https://github.com/sass/node-sass-binaries/blob/master/README.md.

Actually, the io.js issue on Windows is merely the path of executable: https://github.com/iojs/io.js/issues/751. If we run the `npm test` with `iojs` prefix (locally using [nvmw](https://github.com/hakobera/nvmw) or system installed iojs for example), it works fine. But since `iojs.msi` comes bundled with `iojs.exe` and its shadow copy `node.exe` (instead of symbolic link, like in case of *nix), the CI tend to use that alias node.exe instead.

Hopefully that issue will be fixed. Otherwise, consider it all green -- working! :four_leaf_clover:


LGTM


specify that it is degrees east of north


Related to #1


That would make it easier.

Are the S2SRO data really basically unusable? I've never done anything with them, so I don't have a feeling for what state they're in. If the data aren't useable then I should definitely remove them from the comparison.

Excluding it by date range (using the same date as the start of the legacy time period -- 2nd Feb 2011) instead of just specific project types would probably be easiest and clearest. It would also get all the Calibration observations that have just as many problems, but aren't marked as special in any way other than the date.


I am unsure how these tests were passing at all. I needed to change this spec to get all green in my machine.
Curiously the Travis CI runs the server in http://0.0.0.0:3000 in the tests, but rails states in the server help command that the default binding is localhost:

```sh
rails server -h
Usage: rails server [mongrel, thin etc] [options]
    -p, --port=port                  Runs Rails on the specified port.
                                     Default: 3000
    -b, --binding=IP                 Binds Rails to the specified IP.
                                     Default: localhost
   ... other things ...
```

in addition I receive (note the newlines and tabs)

```html
<h1>
  Welcome#index
</h1>
```

instead of `<h1>Welcome#index</h1>` in welcome/index view.
Are we minifying html?


Ah, I see! I didn't think of that side effect, you're right. Thanks for reporting that!

Thankfully, there's no need to remove that feature completely; I just needed to revert the implementation of "addChild()". See: 9fd6733a0d645f95

Please try it out and let me know if that helps!


Nice


Was always the plan to give tons of credit to @ef4 in docs.


Awesome.  Minor nit comment: add a helper method in SessionService that combines these two lines to return the current radius filter distance.


Looks good.  Just one minor comment.  Merge it!


Why revert this?


As always big thanks JeanFred !


Problem with loop could be easily fixed by moving `char_u **prev_file = *file;` out of the loop (to function entry, for example).

Not so sure if changing `expand_shellcmd()` callers not to use `""` instead of NULL would be ok. I considered that possibility when doing this, and found a reason not to. But I'd have to recheck, I don't remember now.

Do we go for the quickfix or do you want to explore the other possibility?


The return value of this function can be null however it's uses within SnippetHelper don't check for null


Now everything is fine! :) Thank you for your efficiency!


Si lo so, tra le modifiche aggiuntive avevo previsto anche quello, appena hai finito rifaccio il fork faccio le modifiche e faccio un unico pull.
Comunque se vuoi direttamente risolvere tu basta un ifdef dove cè il controllo della variabile
Perchè se FILAMENTCHANGEENABLE non è definito il controllo su filament_changing che è una variabile che ho aggiunto io apposta è inutile, perchè quella variabile non fa altro che impedire al OOZING e al RUNOUT di impedire che essi si azionino se la stampante sta cambiando filamento.
Se la stampante non può cambiare filamento semplicemente la variabile non sarà definità com'è giusto che sia, e non deve essere controllata nell'if in questione


Do not play with main's signature.


In addition to being concerned about what these functions look like to other modules, I also think it could be confusing to developers. We're talking about `_get_filename` `user_api/accounts/helpers.py`. To me, that functions sounds like it should be scoped larger than it actually is. That being said, the docstrings here are helpful, so I won't make you change it. It just feels odd.


f your face


The ls can probably be removed


Sorry to say, this won't work on Linux machines. "auto" cannot handle https addresses


Ok, I'll just have to remove my entire tape thing

> On Mar 23, 2015, at 5:21 PM, Graham Lloyd <notifications@github.com> wrote:
>
> They added tape and tape rolls for sticking paper onto things. It
> definitely wouldn't be hard for you to add the tape stuff that you were
> working on to their system. It's in the master repo right now if you want
> to check it out.
>
> On Mon, Mar 23, 2015 at 4:32 PM, Thomas Stoeckert <notifications@github.com>
> wrote:
>
> > I already did all my tape stuff, what did bay do this time?
> >
> >
> >
> >
> > > On Mar 23, 2015, at 4:00 PM, Graham Lloyd <notifications@github.com>
> > wrote:
> > >
> > > There's the tape in there from BS12 if you update. Also, I'm sure
> > Nienhaus
> > > would help with making some sprites.
> > >
> > > On Mon, Mar 23, 2015 at 3:36 PM, Thomas Stoeckert <
> > notifications@github.com>
> > > wrote:
> > >
> > > > They work like tables ATM, I'm working on more stuff
> > > >
> > > >
> > > >
> > > >
> > > > > On Mar 23, 2015, at 4:23 AM, Graham Lloyd <notifications@github.com>
> > > > wrote:
> > > > >
> > > > > I really want this to happen
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub.
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10353136
> > >
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > Graham Lloyd
> >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10354077>
> > .
> >
>
>
>
> --
> Graham Lloyd
> —
> Reply to this email directly or view it on GitHub.
>


Ещё 1 окмментарий касательно front-end'a:
по заданию нужно было минимизировать кол-во "ненужных" компонентов ("Например: пароли, логины, комнаты, списки пользователей и тд и тд."). Недочёт.

Также по заданию предусматривалось удаление и редактирование СВОИХ сообщений. Т.е. тех, которые написал человек под именем соответствующему залогиненному.

оценка: хорошо с +


nee, het was wel degelijk uit-4, uit-21 gaat wel over hetzelfde block


Deploy now fails since there's no default for uaa.clients.gorouter.secret in the spec and there's no changes to any of the templates to create a default for this. Rendering gorouter.yml.erb fails


DId you accidentally commit that?


why not __DIR__ ?


These flags anticipate future support for ghcjs and ghcjs-haddock


oops. i'll update.


Looking good so far! Might I suggest trying out a css 3 animation and seeing if you can get it to work instead of a javascript animation? http://stackoverflow.com/questions/16989585/css-3-slide-in-from-left-transition Lemme know how that goes


А зачем тут `wordtype`?


AutoDocsWithNoLTO (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1660 of 1667 tests passed)

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


Sweet! This looks perfect :)


This change anticipates future support for the ghcjs package database.


Single quotes prevent shell variable expansion.


If you want to be correct, then you need to also encapsulate the nil inside a delay, thus changing things as such:

```clojure
(defn- nrepl-stop [server-map-delay]
  (if (and (realized? server-map-delay) @server-map-delay)
    (delay (stop-server @server-map-delay) nil)
    server-map-delay))
(defn ccw-nrepl-stop []
  (deref (swap! ccw-nrepl-server-map nrepl-stop)))
```


I was hoping for a way to not have this blank method added to the module, but I could not think of a way to call this method from ```aui-timepicker```.


Hi, I *think* the color:#fff !important; causes some issues. If a post doesn't contain an image the icon is white on a white background. I found deleting this and adding

body.nav-opened .menu-button:before {
	    color: #fff !important;
}

fixes this. Not 100% sure on this, though.


loot in gerneral must be done, but better than nothing at all


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1116
success: http://build.servo.org/builders/linux2/builds/1116
success: http://build.servo.org/builders/android/builds/1150
success: http://build.servo.org/builders/mac1/builds/1329
success: http://build.servo.org/builders/mac2/builds/1332
success: http://build.servo.org/builders/gonk/builds/298


Итого:
UI более или менее. не совсем интуитивный UX
функционал реализован весь в полной мере
замечания по коду:
названия переменных типа d2 - это печально :-1:
не соблюдает нотация именования переменных в JS, например TheTextBoxTwo и parentElem. второй вариант правильный. первый - из мира дотнета, наверное.

оценка: хорошо


Nah, just move the features you had for it to the (admittedly lackluster)
tape that's in master.

On Mon, Mar 23, 2015 at 5:32 PM, Thomas Stoeckert <notifications@github.com>
wrote:

> Ok, I'll just have to remove my entire tape thing
>
>
>
>
> > On Mar 23, 2015, at 5:21 PM, Graham Lloyd <notifications@github.com>
> wrote:
> >
> > They added tape and tape rolls for sticking paper onto things. It
> > definitely wouldn't be hard for you to add the tape stuff that you were
> > working on to their system. It's in the master repo right now if you want
> > to check it out.
> >
> > On Mon, Mar 23, 2015 at 4:32 PM, Thomas Stoeckert <
> notifications@github.com>
> > wrote:
> >
> > > I already did all my tape stuff, what did bay do this time?
> > >
> > >
> > >
> > >
> > > > On Mar 23, 2015, at 4:00 PM, Graham Lloyd <notifications@github.com>
> > > wrote:
> > > >
> > > > There's the tape in there from BS12 if you update. Also, I'm sure
> > > Nienhaus
> > > > would help with making some sprites.
> > > >
> > > > On Mon, Mar 23, 2015 at 3:36 PM, Thomas Stoeckert <
> > > notifications@github.com>
> > > > wrote:
> > > >
> > > > > They work like tables ATM, I'm working on more stuff
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > > On Mar 23, 2015, at 4:23 AM, Graham Lloyd <
> notifications@github.com>
> > > > > wrote:
> > > > > >
> > > > > > I really want this to happen
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub.
> > > > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > >
> https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10353136
> > > >
> > > > > .
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Graham Lloyd
> > >
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10354077
> >
>
> > > .
> > >
> >
> >
> >
> > --
> > Graham Lloyd
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TGameCo/UristMcStation/commit/34cd298e9d861c7b02e8b110659f460099e1b392#commitcomment-10355234>
> .
>

--
Graham Lloyd


again, this can be inferred from source.


gaat wel degelijk over uit-22


I'm wondering - maybe we should make the multiplier be 3.0, not 2.0, and really run fast.


what changed?


You forgot to push the binary module..


Hey Chris,

That all sounds great, but I'd still need to look at the source before
investing $500 in it. Please let me know how you'd like to go about doing
that.

Thanks,
Steven

On Mon, Mar 23, 2015 at 5:23 PM, Chris Scott <notifications@github.com>
wrote:

> It has the same API as free version. There’s little going on now on
> Android side, since the HTTP and SQLite persistence layers have been
> removed.
>
> When a geolocation is received, the plugin sends it off to your Javascript
> callback, just like iOS.
>
> - The ActivityRecognition system is always running (default sample-rate
> 10s)
> - When the activity of ON_FOOT, IN_VEHICLE, ON_BICYCLE, RUNNING are
> detected: *GPS ON*
> - When STILL is detected: *GPS OFF*
>
> It doesn’t currently send the detected-activity to Javascript, but that’s a
> 30min job.
>
> The meat of the plugin consists of ~300 lines-of dead simple code within
> the CordovaPlugin file There are just 2 files (plus a binary lib,
> eventbus-2.4.0). There is no more giant BackgroundLocationService.java
>
> You would migrate all your application intelligence *out of the Android
> code* and into Javascript.
>
> On Mon, Mar 23, 2015 at 5:04 PM, Steven Wexler <notifications@github.com>
> wrote:
>
>
> > Hey Chris,
> >
> > I've been doing heavy modifications on your free plugin for an app I'm
> > working on. I'm interested in your premium version, but would need to
> get a
> > sense for what the code looks like, as there were a few objections I had
> to
> > the free version code, and I wouldn't want to spend $500 without
> verifying
> > that the code won't require heavy modification on my end.
> >
> > Can we schedule time for a code review (can do over screen share), or
> maybe
> > you can give me 1-day of private repo access so I can review, and I can
> > sign your license/privacy agreement in advance so that you're protected
> > from me pirating your code (which I won't do).
> >
> > Thanks,
> > Steven
> >
> > On Mon, Mar 23, 2015 at 4:55 PM, Chris Scott <notifications@github.com>
> > wrote:
> >
> > > You really should start using my premium version. All this "drive
> > > detection" stuff is handled in a much better way, automatically, with
> > > Android's ActivityRecognition API, which intelligently aggregates
> > > low-power accelerometer & gyroscope to stream a DetectedActivity to you
> > > at a configurable sample-rate.
> > >
> > > STILL, WALKING, IN_VEHICLE, ON_BICYCLE, ON_FOOT, RUNNING.
> > >
> > >
> > >
> >
> https://developer.android.com/reference/com/google/android/gms/location/DetectedActivity.html
> > >
> > > And the battery performance of everything is 1000% better -- my phone
> can
> > > run for days with plugin running.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/steaks/cordova-plugin-background-geolocation/commit/280dee84987ed6e35cafade7acef41ef1281b2d0#commitcomment-10354606
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/steaks/cordova-plugin-background-geolocation/commit/280dee84987ed6e35cafade7acef41ef1281b2d0#commitcomment-10354765
> >
> > .
> >
>
>
>
> --
> Chris Scott
> Transistor Software <http://www.transistorsoft.com>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/steaks/cordova-plugin-background-geolocation/commit/280dee84987ed6e35cafade7acef41ef1281b2d0#commitcomment-10355088>
> .
>


Some packages have no modules (e.g. the initial release of transformers-compat) and this was causing a build failure.


SRO is possible but needs work (e.g. recalculating all the flat fields). It would be safest to start the clock from when the 8 subarrays were commissioned.


Typo Excluce => Exclude


Very nice indeed!


ok


We probably want to explicitly set permissions here.


# Dive into Dimple
This series of files illustrates an article about datajournalism, from the raw data to the final JSON data, cleaned out and set up in a CSV file to be consumed by Dimple.js


Works for me!  I'm able to highlight and move around the objects on screen.

When I move one of the cubes, I'd like the updated position to be shown in the code. For instance, if the cube starts at (1, 1, 1) and I move it to (0, 0 ,0), then I'd like to be able to see a line saying `cube.position.set(0,0,0);`
Is that possible to do without crashing the scene?


fast-forwarding master to auto = f62ab247


You're using noncopyable here because we're allocating a bam1_t struct etc internally?


whoops. I'll fix it soon.


If the container isn't running, will this return quickly?


This is one of the tasks for the experiment I was using to try your code.

```javascript
var t3 = THREE;
var light = new t3.PointLight();
light.position.set(-40, 15, -20);
scene.add(light);

var cube = new t3.Mesh(
        new t3.BoxGeometry(0.5, 0.5, 0.5),
        new t3.MeshLambertMaterial({color: 'blue'}));
cube.position.set(2, 1, 2);
cube.name = 'cube';
scene.add(cube);
var target = new t3.Mesh(
        new t3.BoxGeometry(0.5, 0.5, 0.5),
        new t3.MeshLambertMaterial({color: 'red'}));
target.name = 'target';
scene.add(target);
```


Partially resolves issue #4


I see. I will update as you corrected. Thanks!


Thanks. Unfortunately it looks like the gulp-karma plugin got blacklisted recently on gulp.js (https://github.com/gulpjs/plugins/pull/149). I don't see this being merged if I did a PR.


Hi @ozzimpact ,

Create Misc. or Shortcuts section and add commands to it.


We really shouldn't be doing anything at all with `!important`, and if we are, then it should be commented saying exactly why this is mandatory and there is no other solution. /cc @PaulAdamDavis @novaugust


saw approval from jdm
at https://github.com/frewsxcv/servo/commit/2da17b3ce546cec9bea40c3ebfdb6ded3c1e78ef


merging frewsxcv/servo/cleanup-view-source = 2da17b3c into auto


frewsxcv/servo/cleanup-view-source = 2da17b3c merged ok, testing candidate = 0c3615ce


I asked about town links and got this, prob not gonna help:

The town connections are done with 2 separated steps:
Most of the links are computed using the function in SM_F_TownConnect.sqf. The solution depends on town values, their distances and the order of resolution.
The second step is to add links defined statically on some the town inits :
Code TEXT :
nullReturn = [this, ""Paros"", resistance, 350,""Town39""] execVM ""Common\Init\Init_Location.sqf""";
the last parameter is a forced link, it is also optional.

Be also aware that all towns must be named "TownXX" with consecutive numbers in XX or the initialization script for the CTI_TOWNS variable will fail.

If you can't randomize the map, then what about creating a few more options? You have East, West, Central, whole...what about manually making 3-4 more options?  Start LIGHT for fast matches 20 towns, 40 towns, 50 towns, etc...  But use the WHOLE map. You could even focus on strategic positions like tall hills, dense forests, military bases, etc.

Will that make it easier?


Get Master/slave replication information


Partially resolves issue #4 concerns. VRRP traffic still adds to congestion on the networks, but some protection from malicious injection is provided by authentication requirement.


Get Sentinel information


More details on the difference between prop() and attr() in "Attributes vs. Properties" section at https://api.jquery.com/prop/


Boo, I blew it :( Just quick perused the sass and didn't even see that line. Sorry.


This seems to have introduced a bug when switching themes.  Steps to reproduce:
1. Use the scaling to increase the font size
2. Change theme
The font reverts original font-size but the "line-height" spacing remains that of the increased font size.


Seems that this error-handling could maybe be refactored into a PhoneTest method.


Plater -> plaster kanske?


Yes, it is for the code to be easier to read ^^
You know, writing an application is pretty easy and fast but the biggest part is maintenance. And keeping codeas  easy to read as possible is what I try to focus on :-)


> Problem with loop could be easily fixed by moving char_u **prev_file = *file; out of the loop (to function entry, for example).

`expand_wildcards()` could assign another pointer `""` to `*file`. Would the `assert()` not become meaningless in this case?


fyi @jwayman


Also want to inform @karlitschek of my mistake. Can you confirm this change is OK?


By `git-bash.bat`, do you actually mean `Git Bash.vbs`? If not, I'm unclear what `git-bash.bat` you refer to. Could you please clarify?


Nexus Lifecyle suite of tools ... got something better?


Hey this looks good. Is your branch usable for an instagram auth strategy?


Think you want to remove this line :)


:+1: to rehdat ;-)


You could omit the `help` case entirely, since it's being caught by the `else` below. It may be a bit confusing since there will no longer be a case for the help command, but if the help functionality ever needs to change, there's only one place to make that change (in the else block) and therefore reduces the chance of introducing bugs (by forgetting to make the change in both places - a future coder may come in and not know which version of the help case is correct).


TASK: [RHEL6-STIG | PRELIM | List /etc/shadow for account lock status] ********
<127.0.0.1> REMOTE_MODULE command grep -Ev '/home|root' /etc/shadow | cut -d : -f 1,2 #USE_SHELL
ok: [centos6] => {"changed": false, "cmd": "grep -Ev '/home|root' /etc/shadow | cut -d : -f 1,2", "delta": "0:00:00.004843", "end": "2015-03-23 18:51:40.931700", "rc": 0, "start": "2015-03-23 18:51:40.926857", "stderr": "", "stdout": "bin:*\ndaemon:*\nadm:*\nlp:*\nsync:*\nshutdown:*\nhalt:*\nmail:*\nuucp:*\noperator:*\ngames:*\ngopher:*\nftp:*\nnobody:*\nvcsa:!!\nrpc:!!\nsaslauth:!!\npostfix:!!\nrpcuser:!!\nnfsnobody:!!\nsshd:!!\nvagrant:$6$XSSBRBktWLn9O/o5$gyYk6CMoO27rM1wadkGxmSk0BuZSxgAS/uyfn1uZwkxM/RRyjJhUmoDvD4WhAugAbikwpnWhv6Zz7xc3b4RGs/\nvboxadd:!!\nntp:!!", "stdout_lines": ["bin:*", "daemon:*", "adm:*", "lp:*", "sync:*", "shutdown:*", "halt:*", "mail:*", "uucp:*", "operator:*", "games:*", "gopher:*", "ftp:*", "nobody:*", "vcsa:!!", "rpc:!!", "saslauth:!!", "postfix:!!", "rpcuser:!!", "nfsnobody:!!", "sshd:!!", "vagrant:$6$XSSBRBktWLn9O/o5$gyYk6CMoO27rM1wadkGxmSk0BuZSxgAS/uyfn1uZwkxM/RRyjJhUmoDvD4WhAugAbikwpnWhv6Zz7xc3b4RGs/", "vboxadd:!!", "ntp:!!"], "warnings": []}


@Devon1B the curly braces are variables that are substituted with the real values later, and should not be translated. We need to keep the `{version}` and `{language}` as they are, and only translate the strings around them.


This function is a great way to make sure you don't repeat code. Awesome!


Rename all the arguments from `context` to `unmangleNames`


I'm sure there was a valid reason why the other (and this) `!important`'s are there, though I'm unsure what now.
I'll refactor this all and get rid of any – there's a few in the whole menu code.


@bbaassssiiee is there an issue?


Yes