$.ajax supports then() by default:

http://api.jquery.com/jquery.ajax/

Is it possible to remove the Q dependency or have it more promise-library agnostic? For example in our system we are using RSVP and would be quite bad to have to include another promise library just to make it work.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.82](http://teamcity.monogame.net/viewLog.html?buildId=5872&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 462, ignored: 6 Build time: 00:14:18


Alex,

Thank you for this code development because it take minutes to hours off my time to do the same thing. Lets say that I'd created a curve and want to say it as an expression for later use or sharing, is saving it as an ascii export the only way to do it?


Are you talking about the R or the python code? I fixed the code in python because I had the wrong behavior but in theory I didn't change the R code...did I mess this up??


r+


Is this code called from anywhere?


Sadly not.


Sure! Will do

On Thu, Mar 26, 2015 at 11:21 AM, Seth Holloway <notifications@github.com>
wrote:

> Can you also update the README to indicate that we're MIT licensed? I'd
> suggest a License section at the bottom of the README that says something
> like, "toopher-dotnet is licensed under the MIT License. See LICENSE.txt
> for the full license text."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/toopher/toopher-dotnet/commit/6f066b6e15b66071e2f9b93dd31178d195feecfe#commitcomment-10415801>
> .
>


I guess the pipeline should be started on HEAD, and we wait for the Content-Type there too


Yeah, I figured these were self-documenting :)


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/1163
failure: http://build.servo.org/builders/linux2/builds/1161
failure: http://build.servo.org/builders/mac1/builds/1376


On 03/26/2015 05:20 PM, zatricky wrote:
> Having thought about it, these other potential rules are definitely
> outside the scope of this service definition. They could possibly be
> covered by custom end-user services or new separate services, much like
> mountd and rpc-bind have separate services defined.
>
> However, I do feel that the UDP does fall within the scope of this
> service definition directly.
>
> Please let me know your thoughts. :)
>

There was a bug before to drop the UDP port from the "nfs" service which
is for NFS4 (see the description).
I see both standpoints as valid, therefore I proposed to add the extra
service "nfs3" with the added UDP protocol.

> �
> Reply to this email directly or view it on GitHub
> <https://github.com/zatricky/firewalld/commit/de48db893e35603d9a9914fecb5542c795c7eb1c#commitcomment-10415774>.
>


using ''' docstring '''' format is preferred


ack


punctuation had been overruled, but I really like the braces etc to be reduced.

![screen shot 2015-03-26 at 17 34 55](https://cloud.githubusercontent.com/assets/2543659/6851309/98630bf6-d3de-11e4-8342-32f26e40b52b.png)


Hi Stefan! Just demoing github to the lab.


Si può anche tenere. Volevo solo avvisarti così in futuro non impieghi tempo ad aggiungerne altri simili


No, I meant `[\t ]*` should be used instead of `.?`.


they pass location (FILE, LINENO)


It's nice if the constants stand out from the strings.

![screen shot 2015-03-26 at 17 37 53](https://cloud.githubusercontent.com/assets/2543659/6851369/eb10ddba-d3de-11e4-9ab2-3d1be1061f32.png)


:ok:


ZipException is a direct subclass of IOException. There's no need to have it in the signature if IOException is already there.


why do we need to declare repo_info?


saw approval from Ms2ger
at https://github.com/Manishearth/servo/commit/c9ccf7aeb45540ae6ceb325c633d0f8c0c37d653


merging Manishearth/servo/macro_path = c9ccf7ae into auto


Manishearth/servo/macro_path = c9ccf7ae merged ok, testing candidate = 265f313d


this is checking for some  icon path instead of some string "icon", right?


It feels like there's a better way to handle this....

but I'll be darned if i can think of one right now


Fixed


Ah, I see


I thought there was intention to do a move instead to clean up the existing repo


This shouldn't be ensured here but here: https://github.com/surlemur/node-eventstore/blob/master/lib/eventstore.js#L337


makes the interface more consistent between `gam()` and `bam()` and allows querying of properties of the design that come from using `mini.mf` to set it up without having to actually fit the model -- calling `gam(<args>, fit=FALSE)` does not create the same `G`-object as `bam(<args>)` does


Actually, the upstream fixed the problem differently.

http://www.cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c5d9a8283cfac15b4a5a07f18d5eb10c1f388505


Sure!


What kind of sorcery did you use to get all the opcodes? :smile:


Interesting. Any clue what version of Ruby its running?


I would not want this enabled by default. My Index bucket is not publicly readable. I use it instead of Redis and my application server pull files directly from S3.


HLE, LLE, RDB, and TLB:  You brought up 4 more examples.

"High-level emulation" as an acronym is HLE.
"Low-level emulation" as an acronym is LLE.  You do not capitalize "level" or "emulation".  These are common phrases used to indicate common concepts in emulation, not store brands or marketing/product names.

"RDB" stands for "ROM database".  You do not capitalize the 'b' in "database" or the 'd' at the start.

"TLB" is the translation look-aside buffer...that I suppose in some cases, maybe everything gets uppercase, when presented in a certain way.  Though I don't think the need has arisen.


Thanks, working on it. I guess this is what happens with midnight commits...


Thanks for your work, I've implemented some of your changes upstream and will release a bugfix release of libSDL2pp soon.


@FrankRuns you don't need to keep the *.pyc files in git. To tell git it ignore them, all you need to do is create a file called `.gitignore` at the root of the repo containing file/globs to ignore, one per line.

Here's an example `.gitignore` file which tell git to ignore any file with a `pyc` extension, and also files named `db.sqlite3` (generally you wouldn't commit databases to your repo to, unless you have a good reason to do so).

```.gitignore
*.pyc
db.sqlite3
```


Yes, they're identical in practice. And the `require` example makes a ton of sense. Will open a PR there.


I *am* kinda abusing closures here, I guess…


I think the assumption is that the bucket should be set up for serving if you want it to be and have the proper permissions, instead of on a per-file basis.


additional explanation to make the manual specification of smoothing parameters a little more transparent.


@FrankRuns, what do you mean it's broken? What are you seeing?


No maybe I didn’t committed the change. I’ll pull a request with the version I used and fixed so we can check what is different.

> On 26 Mar 2015, at 16:35, Xavier Rubio-Campillo <notifications@github.com> wrote:
>
> Are you talking about the R or the python code? I fixed the code in python because I had the wrong behavior but in theory I didn't change the R code...did I mess this up??
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/xrubio/models/commit/89af1999c64bec917452ea079fc93decc474ffc5#commitcomment-10416147>.
>


The `copy` import introduces bugs in python 3+. I'll fix that and submit a new patch.


An object by the looks of it. Ill have to look at the Auther model to see
what returns a string then. But can u get the information you need with the
object?
On Mar 26, 2015 10:01 AM, "Han (Jim) Wen" <notifications@github.com> wrote:

> what is this? Is username a string or an object?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nhubui/CMPUT404-project-socialdistribution/commit/c78a9ffb528d7f9552b35dfcf5d7a053c806ec42#commitcomment-10415430>
> .
>


Aonde vc encontrou elementos ``article-title`` com atributos ``@xml:lang``?


Text comes out more readable everywhere. It's also half the file size.


On it.


On Thu, Mar 26, 2015 at 9:30 AM, Richard Newman <notifications@github.com>
wrote:

> That raises some (perhaps minor) concerns.
>
> Fetching the token can result in the server URL changing during any
> request, so we actually *can't* just wrap up authorization and hide the
> details of tokens altogether. I would love to be able to encapsulate the
> token management, but unfortunately the Sync state machine needs to know
> when the token changes.
>
s/token/endpoint included in token/.  I'm not sure we're node
re-allocating, ever -- we should ask rfkelly.  I guess we still might in
the case of disaster :/

> In the face of this, I'm more inclined to use InitialWithExpiredToken to
> explicitly model this: we take that state, issue an i/c fetch to see if
> there are remote changes (which is the only operation that state is allowed
> to perform), and then reuse its scratchpad with a newly fetched token if we
> wish to re-enter the state machine to do more work.
>
We could have the sync auth state be the guarantor of endpoint stability,
although you mentioned in your comment that you foresee a partial
transition of the scratchpad across endpoints.

My put in this area is: Sync is big: handling the Sync client protocol
complicates everything in the Sync records protocol.  (The records protocol
is complicated enough without mixing the client protocol into it!)

I'd like to push the Sync machine to only doing the records protocol and
storage stuff, and to have other machines ensure the Sync client actions
are maintained.  That is, the auth state mechanism maintains "this is the
sync endpoint and I have credentials for it", "this is my device details"
(device name, say, but not client syncid), etc.  So the records is more
about getting the server into a good state and maintaining the synced state
of the device, and less about handling server and auth transitions.

The devil is, of course, in the details -- it's hard to tease out the
client protocol from the records protocol.


r+


I know this is personal attitude, but I discourage using `join()` from a string instance itself. I would use it like this:
`string.join("|", re.escape(v) for v in self.__list_delimiters)`
If you think this is even then you can keep it like it is^^


Passwords are encrypted on the DB.


Ah and since you use `unescaped_split()` that returns strings you could maybe get a memory boost when using a non-capturing group:
`"(?:" + ...`


Don't need to use `#present?` as they will either be a `Money` object or `nil`.


Oops.


Ah and I would wrap braces around the whole statement than escaping the newline:
`"(" +\` --> `("(" + <the next line>)`


you really think that matters from performance view?

join is a perfect member method of the string. Using `str.join(elem, ...)` is just another syntax and its unusual so I wouldn't do that.


Is it worth testing with an empty string as that's what would come via `params`?


"Peripheral Interface (PI)" and "Video Interface (VI)" are only in title case because that is how SGI presented them in their introduction of the RCP technology.  It is also acceptable to refer to **THE** peripheral interface or **THE** video interface, but when referring to the RCP memory map as sections, you would title these sections using title case.

"SP" is just an acronym for the signal processor, which most certainly is a common noun.  The "Reality" in RSP however is a proper noun which does warrant capitalization.


its just as you wouldn't use `list.append(some_list, elem)`


I might be missing something but shouldn't this catch block be identical to the one in `processFile()`? i.e. should we reschedule the task here?


tex is undone later, not necessary


Is it valid to have a fourth draw but not a second and third?


Okay just said, I  think it's better readable when doing `string.join()`, but like I said just personal attitude :)


Mountains are again merely mountains.


undid this later on


Added an option to remove elements by passing a jquery object or a jquery selector string instead of index


From performance view it doesn't really matter, but capturing groups store their matches in a group. Just wanted to mention^^


En mi branch a ese endpoint le puse ensureAuthentication, bsaicamente eso hace que solo se pueda consumir si se esta loggeado, como esta asi le puedo attach el userId en el server side, por tanto no habra necesidad de mandarle "user" no lo cambie en el controller porque sabia que te tocaba modificar ahi.


Having `plot.gam` invisibly return the `pd`-list used for plotting would make it much easier to customize and modify model visualization without having to resort to `Predict.matrix` or any of the un-exported `plot.<smooth-class>`-methods.


understood. Won't do for consistency.

2015-03-26 17:51 GMT+01:00 Makman2 <notifications@github.com>:

> Okay just said, I think it's better readable when doing string.join(),
> but like I said just personal attitude :)
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/60ff2fd8e0b07e5a87b1615bebcc943b8c5dec39#commitcomment-10416482>
> .
>


It's just "flash RAM".  The 'f' doesn't get capitalized.

The only reason I left the 'f' capitalized as in "Flash RAM" in that link you just gave is because it is the very first letter of the phrase, which is fine.


![screen shot 2015-03-26 at 17 53 27](https://cloud.githubusercontent.com/assets/2543659/6851728/107afdae-d3e1-11e4-99ad-6dbcef750954.png)


x


What happens if expiredCallback is not defined? Shouldn't we check it?


puedes ver el branch de signUp que ahi tambien se cambiaron los alert messages como te dije. Lo estaba haciendo como lo hiciste pero lo cambie pa que un service administre todos ls messages.


I don't think you can use 'var' a second time on a variable like that.


moving punctuation to a mixin that includes braces and brackets, so we can insert that here and there to restore specificity


fixed


感谢指正！已修改。


VSHPWH (shenb) - x86_64-MacOS-10.9-clang: Tests Failed

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


Just updated and it seems fine, using aggressive-indent.el with the manual install procedure and with no byte-compiling.

Thanks!


LGTM


folded numbers back into constants later on


no se si se ve bonito que esto devuelva un css class pero aun no veo una forma comoda de poner esto. La unica que pensaba era:
ng-class="{success: isSolved(prob), warning: isPending(prob), todo: isTodo(prob)}"


@fabiangreffrath OK thanks.


If you really prefer "Advanced Block Linking" over "advanced block linking" (or "Advanced block linking") that much, then fine.  I won't fight over it in commits.  You can reverse my change and send a pull request countering parts of my changes you disagree with.

Though I remain convinced that 100% of my changes are 100% correct.  Expressing ABL with title case merely because of the existence of the acronym is more of a semantic interpretation rather than a strict right-or-wrong, so I don't care that much about that.

As for the translation files.  I never did know what those were for. :P I guess all the changes I did here, I need to do to the English translation file as well.  I suppose the translation files are all like "copies" of the language class cpp.  All I really know about multi-language is:  English or gtfo.


Are you sure we want pdmask and not pimask here?


The third link should be [http://doomwiki.org/wiki/GZDoom](http://doomwiki.org/wiki/GZDoom).


THX


Gotcha. I know that with python 2.7.9 SSL cert now verifies by default... This change is to address the effects of this? Thus this change is really only for 2.7.9 users? As I don't recall sab having any validation issues before... Unless your enabling validation on by default in 0.8.x

-----Original Message-----
From: "shypike" <notifications@github.com>
Sent: 3/26/2015 10:48 AM
To: "sabnzbd/sabnzbd" <sabnzbd@noreply.github.com>
Cc: "thezoggy" <thezoggy@gmail.com>
Subject: Re: [sabnzbd] Add Special to enable/disable HTTPS certificate verification. (27ff8da)

This is not about SABnzbd using HTTPS for its own server, it's about accessing indexers with bad certificates. There's no need for a command-line option.
The Wiki "Special" section will be updated.

---
Reply to this email directly or view it on GitHub:
https://github.com/sabnzbd/sabnzbd/commit/27ff8da0561d3e6de90b953f9b0de6b5567bfb83#commitcomment-10415086


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


Nice!


the `.meta` and `.source` selectors really reset a lot of stuff and I had a hard time optimising  lot of stuff because of it. I ended up moving the selectors to above the others which solves a lot of that: https://github.com/braver/duotone-dark-syntax/commit/c789a568106c81b7cc1bca49212cd9da359d8aeb


Okay again a little performance possibility: Since `unescaped_split()` is a generator maybe collect the strings at the end, means you store the iterator in a local variable and process them with lambdas that yield:
```python
list_it = StringProcessing.unescaped_split(pattern, self.value)

if remove_backslashes:
  list_it = lambda it : yield unescape(x) for x in it
if self.__strip_whitespaces:
  list_it = lambda it : yield elem.strip(x) for x in it

self.__list = list(list_it)
```
So you don't have up to 2 copies of your list that have unreferenced strings (since after processing them with `unescape` and `strip` the old ones are not needed anymore).


Looks good. Two more typos:

"[@grayson2001archaeological, @mannino2002depletion ]." Extra space

"This differences leads" -> "These differences lead"


I hope lambdas can yield...


unused


oh looks like i missed one thing.. @nickdesaulniers, you want me to remove that `return`?


Reading again seems not, but then declare two new private function inside the current function body and use them like the lambdas^^


I prefer Apache 2.0 these days but it's too complicated to go back and change Foreman.


This commit also adds a fixed scale for pcap secs, in getting ready to build zoom navigator.


Gnargh. I was certain I'd tried that, but yep, you're right: This doesn't do the right thing.


strings.xml is getting pretty big. We might want to split up into a few different field (ViewEvent, ViewTeam, etc)


A quantidade de pinos é determinada pelo hardware, podendo (devendo) ser uma opção de compilação.
Em outras palavras, não precisa usar alocação dinâmica, um simples vetor estático vai funcionar melhor.


Ah and then I see you can use one of the StringProcessing filters to trim out the empty strings:
Nvm, the function comes in the new update of the module...
But you can again write one inside the function body:
```python
def trim_empty_strings(it)
  for x in it:
    if len(x) != 0:
      yield x
```


`params.sort`?


Does this commit requires any change on how we declare image background on nodes? Because previous to this commit my png image were showing but after that, they don't.


8.3-Release-WrapUp (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1674 of 1676 tests passed)

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


@crosbymichael we are talking about StdFds here. You have asked me the similar question in another pull request https://github.com/docker/libcontainer/pull/485#issuecomment-86621235


generator expressions `[do_something(elem) for elem in something]` are as good as generators, no need for complicating them with lambdas


Yes, actually I've removed it locally, just need to check it in. As I said in the standup on Tue, we are going to allow partners to specify branch key in two ways, including this one. So it's no longer deprecated.


Which is why I wonder how this is a "fix"...


@jrgifford my actual initial comment was a suggestion to remove rubygems-bundler now that Bundler is not included. Why force every rvm user to install a gem that only applies to Bundler of rvm doesn't even include Bundler? That doesn't seem very "absolute minimum" to me.


Making this class a singleton AND a something that can (and will) be instantiated is somewhat contrary. Since the main reasoning for needing a singleton here is the function that registers the widget, it's best to just move that to a global function. (See http://en.wikipedia.org/wiki/Separation_of_concerns)


this needs to go away too @shama


We can use PHP5-style constructors here: `parent::__construct( ... )`


8.3-Release-WrapUp (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-passing-green.svg)


I don't think we can guarantee that node assignments will never change for a user, nor should we; it's still the mechanism for load adjustment and server maintenance. And that will certainly be first noticed when tokens are allocated (perhaps more than any other situation, simply because it'll be the first time the client finds out after a long absence!), so we can't simply rely on correctly handling 401s as the only reallocation point. (Unless, of course, we ensure we fetch a new token at the start of a sync, or refuse to use an expired token — that is, we choose when to do the difficult bits.)

Each time we transition between tokens we need to make sure the URI is the same, and if not we need to abort what we're doing and go to a non-steady-state flow (wipe some timestamps, discard cached stuff, etc.).

That observation either needs to be part of this state machine, or it needs to be explicit in the interaction between the storage machine and auth/token machine, which really just shifts an `==` onto the other side of some function call; we still need to pay attention and do the cleanup.

I think I feel that noticing that a server allocation has changed, and deciding what to do about it, is exactly what this particular state machine is supposed to do; the auth mechanism maintains your FxA state and is a factory for tokens, but it's not in a position to decide what to do about changes in the contents of the tokens over time, right?

`SyncStateMachine` isn't really a records machine, it's more like `GlobalSession` on Android.

The scratchpad handoff is part of that — we wouldn't handoff a cached meta/global or info/collections if the server URI changed, for example.

When you say "so the records is more..." I think you're talking about the thing that `Ready` produces to do syncing, which does exist only within the scope of a single working token, and aborts on 401. I haven't built that yet :D


Got it, we can move it to the state file if it's runtime state


Shadow is applied to the image in the node (png with transparency).  I'm not sure this is the expected behaviour. Should we reset the shadow before the image?


Since we're not actually using the name and description anywhere else in the widget, it's a waste to add them as properties. Local variables would be better.


@sindresorhus what happens now when someone uses a generator without yo?


ok gah this is tricky, how are we going to feature sniff for the tests? so are are the tests now invalid?

I thought since https://github.com/tastejs/todomvc-app-css/pull/7 landed.. things are at odds with one another

I am very confused right now


But you collect them already with `list()`, so all iterators are evaluated and you gain nothing from using then (except the short code form).


At least I think so... https://gist.github.com/gnarf/a59255719f0b06322d96


possible missing "</etc/crypttab" redirector


`sanitize_title` is a very poorly named function :) It'll convert `Hello World` to `hello-world` which is not really what we want here. `sanitize_text_field` is a better choice.


What's the purpose of using `gofmt -d` here?  Couldn't you just use `-l`?

Also, the output of `gofmt` could be large, so do `gofmt ... | head -1`.

Also^2, `[ -n "..." ]` rather than `[ "..." != "" ]`.


This, my friend, is cargo-culting.


r=jdm


aww, i'm pretty sure I removed the br here for a reason! If we want a break here this can be done in the theme.css.

why do you remove all that handcrafted indentation here???? Ugly and overlength HTML is the result!

I currently have no time to work on layout due fixing core stuff, but working against eachother is not good. :-(


We should validate and/or sanitize all the remaining ones too, just to be safe.


Did I mention I hate our non-choices around failable init?


Addresses https://github.com/alpheios-project/alignment-editor/issues/18 for @maximromanov


wot? like i am all over the flipping place in this project?

=D

not exactly what i meant but it is a good point. ;-)


good good study , day day up!


This might be helpful here: [Remove Borders from TextField (JavaFX)](http://stackoverflow.com/questions/23780490/remove-borders-from-textfield?rq=1)


ack


ท่าจะเช็คก็น่าทำก่อน parseInt เนาะ


Handcrafted identation? Are you kiddin me right? Please, open the file and check current identation, it's nothing else than just a disaster. I just fix the identation on the parts i'm working on. Childs are correctly idented now according to the parents


ok good luck


Ditto here.


As above, we should move this to a global function. If the preference is to keep it in the class for whatever reason, it'd be better to change it to a static method that we can hook below as:

```
add_action( 'widgets_init', array( 'Simple_Media_Widget', 'simple_media_widget_init' ) );
```


This will screw my pool up since I use percentages. I've said it before, but we can set this through OMV_FSTAB_MNTOPS_MHDDFS.

If there's going to be options for it it needs to be versatile.


rax retry tests


The title should be escaped on output here.


The description should be escaped on output here.


Heads up: This is causing the demo page to hang (provided the user isn't running LiveReload)


ack, nice little test :)


It is not very good to copy my code and just change a few of the variable names, my friend.


@jeremyfelt We should call `.main-footer` as a part and let that part be overriden. It's strange to have a footer and then a footer.


Just to be safe, we should escape all outputted instances of `$widget_id`.


Doesn't this make a mess of `$lintoutput`, mangling whitespace and newlines?  And what if `$lintoutput` is very large?  Simplest solution is probably to reidrect the output into a temp file, and then check if there was anything with `[ -s $f ]`.


Rolled back this change:
                //DNN-6509 - using empty string will ensure active portal language is used
                GetPortalSettingsDictionary(portalId, Null.NullString).TryGetValue("EnableBrowserLanguage", out setting);
Problem is that GetPortalSettingsDictionary will call activeportallanguage  and it can't be defined when culture has no value as it will cause overflow loop (GetActivePortalLanguage calls EnableBrowserLanguageInDefault when cookie are not defined).


I agree with your explanation: this is the timesheet report and people interested in all analytic lines irrespective of timesheets can use the analytic analysis report.


Trailing whitespace detected.


Huh?  This commit removes the old build system and has nothing to do with the content in the new one.  The checks for these headers in CMake 3.2.1 are only inside `if(NOT UNIX)` conditions:

 https://github.com/Kitware/CMake/blob/v3.2.1/Utilities/cmcurl/CMakeLists.txt#L523-L532

We have nightly testing that includes builds on Cygwin.


Cool :)

As you can see we can still talk, despite the issue being closed.


I need all elements anyway so when I convert it into the list doesnt really matter does it?


These two lines have trailing whitespace.


Posts is not a collection yet. Collections don't yet know how to discern their dates, nor are the url placeholders custom. They're [explicitly defined here](https://github.com/jekyll/jekyll/blob/master/lib/jekyll/document.rb#L126-L138).

In Jekyll 3, we want to make posts a collection and bring with it all the trappings of custom sorting and date consumption but until that time :/


We'll probably need to split this up into two:

1) no-caption: just build the HTML and output.
2) caption: build the shortcode and then run through `do_shortcode` and output the result.


Thanks, I actually caught the error on Sherlock and fixed it, but hadn't updated my local repository, will do that eventually.


Come to think of it, that would only apply on single, since these page templates do not have an existing main-footer.


@greg80303 this looks pretty great.  I want to suggest that we take some more time to review and maybe discuss on this upcoming tuesdays call as this is a significant change before you merge, but this is just a suggestion ;)  Ill take some time in a bit to review the code but at first glance looks great.

One thing I have started to notice is the removal of the dijon injection point and replaced with the system.getObject call.  I was under the impression that both singletons and class objects could be injected with the context string name var marked as undefined but I have also had instances where this did not always work so I am confused now on why.


@steveoh They will have to explicitly install `yo`. This is a good thing. Also npm3 won't do this for us anyways. So this will soon be moot anyways.


I believe the request was to consistently use 3 digit versions, as NPM/semver does.  Moving to 4 digits here is invalid as far as NPM is concerned (hence the failed tests).  We should consistently move to 3 digits for our JavaScript code/packages.  You can still have an optional prerelease suffix after a dash (i.e. "1.5.0-beta" is valid).


Consider replacing ` ()` with `()`.


Fixed. Good news: the deploy loop never broke — jekyll failed because the website was broken, but the deploy loop was still working, so when I fixed the website and pushed the fix, the loop reran Jekyll and updated the website.

Bad news: the problem is that you broke a symlink by removing its target, and IIUC you couldn't have detected that since IIUC symlinks aren't checked out as such on Windows. First I did a minimal fix, then I replaced the symlink by its target in c83b747994c99d297a57c09f1d97922fd5110929. We should probably just avoid symlinks.

```
Configuration file: /afs/informatik.uni-tuebingen.de/home/giarruss/git/website-vanilla/_config.yml
            Source: /afs/informatik.uni-tuebingen.de/home/giarruss/git/website-vanilla
       Destination: /afs/informatik.uni-tuebingen.de/home/giarruss/git/website-vanilla/_site
      Generating...
jekyll 2.5.2 | Error:  No such file or directory - /afs/informatik.uni-tuebingen.de/home/giarruss/git/website-vanilla/research/functors/klaus-wg211.pdf
......................................................................................................................................................................................................................................................................................................................................................................
```


~/cm12$ grep "debug.sf.hw" -r frameworks/native/
~/cm12$ grep "debug.sf.hw" -r frameworks/base/
~/cm12$

lol :P


Okay I think too much in C/C++... I think now there's no real difference between global and local variable handling (except of the scope). Never mind :)


You don't need this. Is it you kept to say install here ?


I don't think that's a valid query string format.


Well, yes, maybe this exact commit was not the best one to start this discussion.

The problem is that for cURL HAVE_WINDOWS_H should be unset, because of this line:
https://github.com/Kitware/CMake/blob/v3.2.1/Utilities/cmcurl/lib/curl_setup.h#L260

If HAVE_WINDOWS_H is set it tries to include tchar.h which does not exist in cygwin. The old CMakeLists.txt used to unset it on cygwin, the new one does not do this.


Nice. Can leave comments line by line.


its in the display hal


You read my mind. :)  `single.php` and `index.php` are the only templates with an existing `<footer>`.

Grabbing `parts/footers` is a little ugly, but it mirrors the general functionality of `parts/headers` at the top of each of these segments. I almost did `parts/main-footers`, but wanted to be consistent with naming.


You probably meant -I**$**{includedir}, and same for libdir below.


r+


Okay just the `+\` should be corrected, we've already discussed that, didn't we?


[no need for action] this would have been a good candidate for a ViewPattern:

    tokenOk (Just (decodeUtf8' -> Right token)) = do
        ...

and nothing else would have changed in the diff.


new comment


He went with another solution that uses a sfdim_ prefix.  In this case, I was just thinking a URL that decode to something like the above.  I think PHP supports a similar syntax encoding the chars like in https://php.net/http_build_query


Thanks for your reply @parkr. This happens not only with posts, but with all collections, say newsletters for example. I can't use the aforementioned variables at all; it just errors out when I use them. I also tried v3.0.0-beta2, but still the same.

This along with paginator are the 2 issues I'm currently having with this branch which I can't solve in order to merge :/

For paginator, maybe I'm missing something, but is it possible to use it in 2 or more places? We need it for posts, updates and basically all collections.


I think I would recommend using main-footer in spite of the naming inconsistency.


Use "Reference Year" instead


rofl

you building this device? i remember your nick from msm8660 stuff


I'm removing the TODO since we have that bug open


When testing with a completely fresh install, this patch breaks the launch file. When backing the patch out it works, but with the patch included I get the following error message:

roslaunch strands_morse generate_camera_topics.launch... logging to /home/user/.ros/log/somehash/roslaunch-algoritm-Z620-7511.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.

ERROR: unknown machine [localhost]
The traceback for the exception was written to the log file


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1164
success: http://build.servo.org/builders/linux2/builds/1162
success: http://build.servo.org/builders/android/builds/1198
success: http://build.servo.org/builders/mac1/builds/1377
success: http://build.servo.org/builders/mac2/builds/1380
success: http://build.servo.org/builders/gonk/builds/346


Just got an e7wifi yesterday and I'm currently dirtymerging all the CAF things by hand. About to go into MDSS territory and then switch to CAF hals to get the display nice n smoooooth


Just my 0.02€: I do think the signature with `s s' a a'` is more understandable, as it directly relates the `s` with the `s'` and the `a, however, the classical lens notation would be more consistent with the huge amount of tutorials, and the `stab` idiom has already somewhat permeated into the community.
But seeing as `basic-lens` is a `lens` redux, `a a'` would be appropriate.
Either way, I don't think it makes much of a difference, and both are fine.


Consider:

```
if (foo)
{
	reason = "foo";
}
else if (bar)
{
	reason = "bar";
}
else
{
	return true;
}

WriteLog (lsWARNING, Ledger) << reason;
return false;
```


HOLA


You can indent on php if you want, but sometimes indentation must ignored for html. And this was the case here with later changed css in mind.

There can be a difference between

```html
<div><div></div></div>
```
and
```html
<div>
 <div></div>
</div>
```


The test is ignored, not fixed. Is this intentional?


ah, cool

well if you figure out why this thing boots so janky and then at 100 seconds from kernel init it decides to break free and be buttery smooth, please let me know =P


it's the minimum that we need to keep things working without extra issues, when users see `unrecognized command bundle` they `gem install bundler`, the flow is not so easy when they see `gem conflict errors`, it does not say `run "bundle exec"` - this is why `rubygems-bundler` will stay


done


Why is `[[` used here, rather than `[`?


Cool, we can probably get away with changing it as the wsu.edu theme is likely the only one using it so far. Will need to coordinate the timing of a push to prod.


I removed it because it seemed redundant. The following slides are the reasons why - Puppet Approved, Workflow, etc. Do you think this would make sense if you could see it in the video?


Use `=` rather than `==`: They are synonyms with `[`, but for `[[` they do something different.


I like this better:
https://github.com/HowardHinnant/rippled/commit/7ba70c3200fea42fb2b57efa7b53c9e637a64560
It removes the duplication of the SSO construction logic.  Tested.


This was here a long time before.  It puts the lib info in _repo.  The new code just makes use of that existing info.


I saw some nasty MDSS errors in dmesg when I tried playing a video (display completely expl0ded), so I'm guessing (hoping?) that a fancy new display hal will fix that


These two lines have trailing whitespace.


Fix: #7007


Let's add the python bindings too


@douglascamata lembra o motivo de ter feito isso aqui? os testes estão quebrados por essa transformação.


fast-forwarding master to auto = 265f313d


The game crashing when trying to render/display the name of a boat that was registered without problem but is missing essential fields? That sounds pretty bad. This is just delaying the NPE, not preventing it.


The quotes should not be there.  Fixed.


Done, thank you!


That needs to be discussed.


Why is handle_ignored_status the new entrypoint for querying the status? This feels totally wrong, it should be the other way around.
I ask for the status and *incidentally* this checks for and handles the ignored status.


@Soldez done with my reivew. (sometimes I'll write that as DWMR). checkpoint complete.


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/578a90986489000e3f80edc7f45c566d15d25e22


merging Ms2ger/servo/net-tests = 578a9098 into auto


Ms2ger/servo/net-tests = 578a9098 merged ok, testing candidate = ac786e29


Add the uploaded record to the accumulator's deletion list.


This bypasses the optimization of not evaluating the logged strings unless the partition is active.


Bug introduced in #2574


and this is a template and the output should also look ok for debugging raw html, while tools like firebug format the dom, but sometimes make false prediction when seeing wrong html code.


looks nice, just 2 things:
- is it plugged into the assets pipeline in a way it generates packs using webpack automatically when the app is built?
- I would immediately start using some JS code style tool to force a unified code style


If these are all null, perhaps we should short-circuit here?


indeed! thanks!


Changing the tracepoints for this is fine, but for I/O path things in the future we might want to avoid changing them to keep binary compatability of the trace format for rbd-replay, and just add new tracepoints instead. I'm not positive if that'll be needed, we should test it when it comes up.


Think you meant to call this `T` instead of `STAmountType`


It's also probably safe to accidentally upload status changes here.


@masterkorp Is there any reason you reverted this and directly committed to the master branch?


\phew, definitely an improvement!


@cmckinsey Unfortunately this change accidentally reverted the fix for the `isinst` null check


Doesn't this look like a hang to you?


What a magic... nice! :+1:


Paoh


yeah I acked this one in my mind btw ;)


@joachimmetz FYI, I didn't think this one needed to wait on review.


ack


### 请切换到web-demo分支测试在web环境下的实际效果


More than using subscription visitor you are now relying on semantic analysis.


agreed i think latin_name is more logical, however original_name is what we used before in the old version, and i did not want to change the schema unnecessarily.


@viniciusmkm o MoIP está respondendo com a acentuação da descrição dos erros ferrada... ele diz que é um encoding, mas na verdade é outro. É uma treta hard.


8.3-Release-WrapUp (kbenne) - i386-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

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


Yes, that would be much better, thanks.


master branch should be fixed now.


Yeah, but it's better to hang!  Then you break and figure out what happened.  Also, timing out a latch during debugging is irritating.


Hi Marko, can I ask why you put the lock here? It seems prevent the auto vacuum of postgres on the retry_queue table. Any concern if we remove the lock here?

Thanks.
Dikang.


Closed issues #10 and #11


não é `ISO-8859-1`? pelo menos é o que diz a documentação :stuck_out_tongue:


I'd prefer it if anyone without permission visiting this url got a 404. This is an ajax type url - redirecting to login doesn't really help anyone!


8.3-Release-WrapUp (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (657 of 658 tests passed)

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


Question of style but you could test for isTypeSymbol to avoid negation...


TeamCity xBIM :: XbimWebUI :: Build [Build 44](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1227&buildTypeId=XBIM_XbimWebUI_Build) is now running


In your my bashrc, I have an entry that looks a little like this:
```bash
# enable programmable completion features (you don't need to enable
# this, if it's already enabled in /etc/bash.bashrc and /etc/profile
# sources /etc/bash.bashrc).
if [ -f /etc/bash_completion ] && ! shopt -oq posix; then
    . /etc/bash_completion
fi

# If git isn't installed yet... these may fail
export GIT_PS1_SHOWDIRTYSTATE=1
export GIT_PS1_SHOWSTASHSTATE=1
export GIT_PS1_SHOWUNTRACKEDFILES=1

if [ "$color_prompt" = yes ]; then
    NC='\e[0m'
    C11='\e[1;32m'
    C13='\e[1;34m'
    export GIT_PS1_SHOWCOLORHINTS=1
    PS1='${debian_chroot:+($debian_chroot)}\['$C11'\]\u@\h\['$NC'\]:\['$C13'\]\w\['$NC'\]$(echo -en `__git_ps1 ":(%s)" 2>/dev/null`)'$NC'\$ '
else
    PS1='${debian_chroot:+($debian_chroot)}\u@\h:\w$(echo -en `__git_ps1 ":(%s)" 2>/dev/null`)\$ '
fi
unset color_prompt force_color_prompt
```


r=jdm


TeamCity xBIM :: XbimWebUI :: Build [Build 44](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1227&buildTypeId=XBIM_XbimWebUI_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:23


also styling but for checks I would not use super to call the addIssue. really does not make things clearer.


`${filename}` vs `$filename` - choose one and be consistent

This script has lots of places where it can coke if it encounters a filename with spaces in it.  Although we don't yet have filenames with spaces in our repo, it would be nice to make the script work with them.  This means using `"$filename"` consistently and using `"$@"`.

The recursion in `lint()` confused me, until I realized that it only happens when a directory is passed, and then it only happens recurses a single time due to `find -f`.  Only one of the calls to it can actually pass directories (the one under `for filename in $@`).  That all seems a bit contorted.

So I'd suggest dropping lint, making `lint_go` return its status with `return` (rather than using the `EXIT_CODE` global), and turning the top-level code into something along the lines of:

    { if [ $# -gt 0 ] ; then
        find "$@" -type f -name "*.go"
    else
        git diff --cached --name-only | grep '\.go$'
    fi } | {
            result=0
            while read f ; do
                lint_go "$f" || result=1
            done
            exit $result
    }


Thanks so much @MarcosMeli and sorry about that!


Successfully opened the i2c bus! The form of:
```
int file;
char * filename = "/dev/i2c-1";
if((file = open(filename, O_RDWR)) < 0){
	printf("Failed to open the i2c bus");
	return 1;
	}
```
is pretty common so I'll probably make a function to do this initial init.


Woops - this is something me and chris missed, isn't it?


@viniciusmkm quem dera fosse, aiuhdsiuahdiasdsa. Esse negócio deu tanta dor de cabeça e tomou tanto tempo que pulamos e ficou para mais tarde.


Sorry! Published a fix on `1.3.1`.


Forgive because I am having a bad day...

How does it screw up your pool??  It only adds mlimit to new pools.  I did test exactly that as well.  The user on the forums had a very good reason to add it to the user interface.  I don't really see the problem.  Why make them use the command line?

On big pools, even 1% may be more than the users want to give up.  So, I don't like percentages.

And I'll stop making changes to the plugins you maintain since you never seem to like them and/or have to "fix" them...

-----Original Message-----
From: "nohatssir" <notifications@github.com>
Sent: Thursday, March 26, 2015 12:15pm
To: "OpenMediaVault-Plugin-Developers/openmediavault-unionfilesystems" <openmediavault-unionfilesystems@noreply.github.com>
Cc: "Aaron Murray" <aaron@ryecoinc.net>
Subject: Re: [openmediavault-unionfilesystems] add mlimit for mhddfs (11f8b5b)

This will screw my pool up since I use percentages. I've said it before, but we can set this through OMV_FSTAB_MNTOPS_MHDDFS.

If there's going to be options for it it needs to be versatile.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenMediaVault-Plugin-Developers/openmediavault-unionfilesystems/commit/11f8b5b15103fdcd9b09041f3ea99ae1846abcc7#commitcomment-10416969


done


esse force_encoding não adianta.. sem ele ficava como? consegue lembrar?


it is nice to get doc review from native speakers :) ... But I ended-up breaking this paragraph using a list.


No problem at all :) Thanks to you !!


Somebody needs to start. Doxy please.


Thanks :)


@lepture
What's your plan for a new release?
I'd say that #88 and #90 would be nice to have in there, too.


@tarcieri Is this url meant to be canonical source for more info? Getting a 404/unauthorized. Is there another link?


How about something along these lines?
```
    size_t i;

    for (i = 0; i < dest_size - 1 && src[i]; i++)
    {
        dest[i] = src[i];
    }
    dest[i] = '\0';

    return src[i] == '\0';

```


Mention that it is an implementation of the spec and link to it?


s#supports systemd#supports the systemd#
link to socket activation protocol pls


accepts a `--spawn-metadata-svc`


mention why? i.e. that it is racy


General note, inconsistent use of the indefinite article ("metadata service" vs "the metadata service"); the former starts to look like Apple marketing materials so I would suggest just standardising on the latter everywhere ;-)


On the other side, details.view can be reviewed/rethink as a whole in FS1.1+


This would also fix https://github.com/lepture/python-livereload/issues/89.


Yes!; already fixed locally;


This commen exclude option -d


I would actually prefer shorter lines, but I couldn't figure out how to get the bullet markup to work right if I put the text on multiple lines.  If someone knows how to get the markup to work I'd be delighted to fix this.


it is kinda awkward to return this giant thing as an error, how does it propagate to the user?


Hi Zhinan, I'm really sorry for introducing this bug.  Thank you for finding and fixing it.  I'll check my other changes.  Ryan


Should this check *tablename as well?


trivial, but why HPCCFile * instead of HPCCFilePtr  ?


Agree, LGTM


I'm still wondering why this was pushed to `master` as a "fix", when it does not even evaluate.  Does it build for @wkennington?


Yeah, I accidentally left my test code in.


The default paginator is all sorts of broken and I don't have the time or interest to fix it. If you have time: https://github.com/jekyll/jekyll-paginate

When I say "we want to," I mean it's not possible at all yet -- the code hasn't been written. So you'll have to continue to use _posts in their current form for now.


yeah, that's what I did on my fork as well :)

also, the `/opt/jboss` is hard-coded a few more often into this file, so the value would not be consistent


so there are multiple ctors, one takes a bool and another one takes a string?


8.3-Release-WrapUp (kbenne) - Win64-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

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


One would think that the test of the lower bits of FCR31, followed by `fesetround` with quite descriptive named constants, would be good enough; however, I could change this to say `/* Round to nearest or even if equidistant */` after `case 0` and so on if you want.


Ficava com caracteres estranhos no lugar dos acentos. Se não me engano, essa foi a melhor quase-solução de encoding que encontrei e acho que até abrimos ticket no MoIP sobre isso. :/


Slightly misleading, the method removes the element, it does not hide it


Will change.


No existe rama "default".


This makes it more convenient to use the Erlang WAMP client, so that the client can publish messages, call services, etc with atoms as parameters, without having to convert the atoms to binaries first.


Ah, I see. https://github.com/jekyll/jekyll-paginate/issues/3 is what we need here :) So, is there any point in using jekyll-paginate for the time being? BTW https://github.com/jekyll/jekyll-paginate#usage is missing something there :/

It's not only _posts I'm having issues with permalink though. It happens with every collection.

>This happens not only with posts, but with all collections, say newsletters for example. I can't use the aforementioned variables at all; it just errors out when I use them. I also tried v3.0.0-beta2, but still the same.


@dougdroper Here's the styling for the `<- Money advice sections` link that appears above the Corporate homepage navigation.

And the HTML...
```html
<a href="#" class="link-to-section-block">Money advice sections
  <span class="icon icon--previous-section"></span>
</a>
```


The double-tildes were there for a reason - to mark out that text.  I'm not sure we have datasets or slices, concepts maybe, but then it's not much of a list.


I guess we can in fact using SyntacticEquivalence, thus avoiding false negative :)


Would it help to include the WUID in the log message?


Klasa się nazywa dobrze: Event.
Widzę, że nie rozumiesz komentarzy na github: one dotyczą konkretnych linii w kodzie. Czyli w tej chwili dyskutujemy sobie komentując linkijkę "+    @Path("/remove")"... W komentrzy odsyłam Cię do moich uwag w klasie Event bo Person jest właściwie jej kopią, więc popełniłeś te same błędy dwa razy.
Poprawianie błędów zacznij od klasy Event - uwzględnij wygląd REST API jakie ustaliliśmy. Np. w tym konkretnym przypadku masz usunąć @Path("/remove") i metodę POST zamienić na DELETE. To samo w Event...
Mam nadzieję, że teraz jest jaśniej. W razie czego pisz. Ja nie wiem czego Wy nie wiecie...


Helpers.js? :[


Why such low tolerance? This means that 0.91 is technically okay... seems too generous.


[Agency] should be replaced by our own properties...so we shouldn't need that specific change.


I know, but I have had time fix it properly. I'll fix it ASAP. (Sorry)

The issue is that readline ignores (does not provide)) pkg-config setting, so we need to hardcode -l<lib> to our build-system and with static builds it's always pretty ugly.

We need $TINFO_LIBS_STATIC, fortunately tinfo provides pkg-config setting, so
      UL_PKG_STATIC([TINFO_LIBS_STATIC], [tinfo])
should be enough, and then we can use this variable for READLINE_LIBS_STATIC too.


Już teraz wszystko wiem :) Wieczorem postaram się to poprawić wszystko.
Dzięki.

Pozdrawiam
Paweł Frydrych

W dniu 26 marca 2015 18:59 użytkownik Maciej Łotysz <
notifications@github.com> napisał:

> Klasa się nazywa dobrze: Event.
> Widzę, że nie rozumiesz komentarzy na github: one dotyczą konkretnych
> linii w kodzie. Czyli w tej chwili dyskutujemy sobie komentując linkijkę "+
> @Path <https://github.com/Path>("/remove")"... W komentrzy odsyłam Cię do
> moich uwag w klasie Event bo Person jest właściwie jej kopią, więc
> popełniłeś te same błędy dwa razy.
> Poprawianie błędów zacznij od klasy Event - uwzględnij wygląd REST API
> jakie ustaliliśmy. Np. w tym konkretnym przypadku masz usunąć @Path
> <https://github.com/Path>("/remove") i metodę POST zamienić na DELETE. To
> samo w Event...
> Mam nadzieję, że teraz jest jaśniej. W razie czego pisz. Ja nie wiem czego
> Wy nie wiecie...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/blstream/AugumentedSzczecin_java/commit/29b04420d03d608b7e9a32afa97ca6d14b35d795#commitcomment-10417946>
> .
>


Hi,

While this patch is fixing the WITH_GSTREAMER{,_0_10} semantic, it introduces a slight difference between OpenCV-2.4 and OpenCV-3 when configuring:

WITH_GSTREAMER | WITH_GSTREAMER_0_10 | gst-1.x installed | gst-0.10 installed | gst support in OpenCV 2.4 | gst support in OpenCV 3
--------------- | -------------------- | ----------------- | ------------------ | ------------------------- | -----------------------
OFF | OFF | - | - | none | none
OFF | ON | - | no | none | none
OFF | ON | - | yes | gst-0.10 | gst-0.10
ON | OFF | no | no | none | none
ON | OFF | no | yes | gst-0.10 | gst-0.10
ON | OFF | yes | - | gst-1.x | gst-1.x
ON | ON | yes | - | gst-1.x | gst-0.10

So, this raise here the same question as mentioned at [1].
Is this the intended behavior?

[1] https://github.com/Itseez/opencv/pull/3310#discussion_r19109505

Regards,


Can this be made to fallback in case of older cmake?

As for Ubuntu/Debian, cmake 3 is rather new (but easy to upgrade):

      Version table:
         3.0.2-1ubuntu2 0
            151 http://archive.ubuntu.com/ubuntu/ vivid/main amd64 Packages
         3.0.2-1 0
             50 http://ftp.de.debian.org/debian/ unstable/main amd64 Packages
            150 http://ftp.de.debian.org/debian/ testing/main amd64 Packages
         2.8.12.2-0ubuntu6 0
            152 http://archive.ubuntu.com/ubuntu/ utopic/main amd64 Packages
     *** 2.8.12.2-0ubuntu3 0
            500 http://de.archive.ubuntu.com/ubuntu/ trusty/main amd64 Packages


ast_optimizations (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (335 of 335 tests passed)

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


Ref: #177


yes


You probably want to remove this line. My specs ran just fine.


ok, thanks for clarify it.


This is user facing documentation. The user does not need to read the internals of the protocol. It's better to provide link to what socket activation is but there's not a great resource. Maybe http://www.freedesktop.org/software/systemd/man/systemd.socket.html ?


This should really be parameterized


@chyld In respect to your comment:
I need the plugin registration to fail - but exporting the version
plugin from index.js seems like a hack just to increase code coverage.

Lines 23 & 24 is where your issue is (which you already know):
Remove:
 if (err) {
            return cb(err, server);
}

And Use:
Hoek.assert(!err, err);
This will get rid of the error messages.
and you get 100% test coverage.

Otherwise, like you wrote before you have to write a test that causes the registration
of Version to fail and create an error.  Why do you not want to use the Hoek method?


Is this working?  I didn't see you add a permalink?


Okay, screw up was maybe not the correct wording. What I really mean is that it should be possible to select different units more than just GB. I'm using a value of 100% which makes it always choose the drive with most space left.


What if the array is `[1, [1,2,3], 1]`? Use `Array#all?`.


sorry, I didn't mean the details of the protocol itself, I meant the higher level description as you suggested. Unfortunately I think Lennart's blog post is maybe one of the only things in that vein :-/


Not needed


yeah the change is probably a bit hasty but well all do that soemtimes. How about we revert the change, fix the current problem and make a new pull request addressing the performance problem with a benchmark? That way the bugfix and the perf improvement are separated :)

For benchmarks please see the `benchmark/`folder for some examples, such as: https://github.com/shoes/shoes4/blob/master/benchmark/dimension.rb

Thanks!


O ciclo de lançamento Wordpress


What about a polygon or mutipolygon ? Maybe set a message to inform that this function only supports with MULTILINESTRING/LINESTRINGs


Done!

- Shane

On Thu, Mar 26, 2015 at 6:12 AM, Han-Wen Nienhuys <notifications@github.com>
wrote:

> Hi there,
>
> thanks for your fix. Would you be able to fill out the Google CLA? It's a
> webform, and we won't have access to any private data you fill in. The form
> is at
>
> https://cla.developers.google.com/
>
> Thanks on behalf of bazel-team,
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/safarmer/bazel/commit/dcddddcd2a3f61d1b760825a8f77f0e069f77e66#commitcomment-10411442>
> .
>


Sure, this on incorrect line, fixed by commit  da99043dc42cbdb963077cb9d8e1c6845db2d1a7.


What about "implements java.io.Serializable, Serializable" ? ;)
I would rely only on semantic for checking actual symbols


[System] here is intended to be replaced with what our data is called.  Since we only have one 'system' - maybe the whole sentence should be rewritten?


Is there a reason why this is INT(10)?


Fixed, thanks!


Why not use aufs then?

-----Original Message-----
From: "nohatssir" <notifications@github.com>
Sent: Thursday, March 26, 2015 1:04pm
To: "OpenMediaVault-Plugin-Developers/openmediavault-unionfilesystems" <openmediavault-unionfilesystems@noreply.github.com>
Cc: "Aaron Murray" <aaron@ryecoinc.net>
Subject: Re: [openmediavault-unionfilesystems] add mlimit for mhddfs (11f8b5b)

Okay, screw up was maybe not the correct wording. What I really mean is that it should be possible to select different units more than just GB. I'm using a value of 100% which makes it always choose the drive with most space left.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenMediaVault-Plugin-Developers/openmediavault-unionfilesystems/commit/11f8b5b15103fdcd9b09041f3ea99ae1846abcc7#commitcomment-10418050


It not the racy part I'm worried about the most. if running under systemd, for example, it'll end up in cgroup of the service that started it. And it'll get killed when the service is stopped. Not sure about other init systems. I didn't want to go this deep and want to specify what the best practice is recommended.


I don't think we need anything but / here, and no permalink needed.


Done.


@hinerm @ctrueden I guess I'm just missing the point here. Why does the version number of a package need to be updated if *nothing* changed in the package itself?

Here's the situation from my perspective: imagescience is officially developed in and distributed by my group. The Fiji team has permission to ship it with Fiji. So far no problems. It gets a little problematic if my group continues developing things (we're currently working on imagescience 2.5.0), and the Fiji team starts changing version numbers too, not because anything changed in the package, but only because of dependencies within Fiji ecosystem...


I don't think you meant to do this.


rax retry tests


Ok :)


It lacks ACLs.


Like all errors, it gets printed to stderr on exit. I think it's ok to point the user to docs. It's also request from https://github.com/coreos/rocket/issues/592.


Note that fdisk and sfdisk are already linked with tinfo due to lib/colors.c (it's generate by libtools (from lib/Makemodule.am: libtcolors.la)), but proper static build (I guess) we need to specify -ltinfo in READLINE_LIBS_STATIC too.


That's just what it is in the current schema


Some minor changes pushed.


I did not say to not rely on semantic but, in case of semantic not available we can get rid of simple false negative like this one.


ack

On Thu, Mar 26, 2015 at 11:09 AM, Eugene Yakubovich <
notifications@github.com> wrote:

> Like all errors, it gets printed to stderr on exit. I think it's ok to
> point the user to docs. It's also request from coreos#592
> <https://github.com/coreos/rocket/issues/592>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eyakubovich/rocket/commit/68fbcb9e4b57e229c441208dbd71b55d30dd019d#commitcomment-10418174>
> .
>


Link pls


OK, as long as we sufficiently dissuade people from using this I am fine.

On Thu, Mar 26, 2015 at 11:06 AM, Eugene Yakubovich <
notifications@github.com> wrote:

> It not the racy part I'm worried about the most. if running under systemd,
> for example, it'll end up in cgroup of the service that started it. And
> it'll get killed when the service is stopped. Not sure about other init
> systems. I didn't want to go this deep and want to specify what the best
> practice is recommended.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eyakubovich/rocket/commit/68fbcb9e4b57e229c441208dbd71b55d30dd019d#commitcomment-10418112>
> .
>


theres a permalink in basics


@AlexMax I have just added the first fix to Crispy Doom and got the second one upstreamed into Choco. Thank you very much for this!

Would you mind filing such issues in the Crispy Doom bug tracker in future, please?


first result for "systemd socket activation" :-)
http://0pointer.de/blog/projects/socket-activation.html


Why not NS_UNAVAILABLE instead?


Is this commit even important to keep in our project?

I think we should do a `git rebase -i master` to undo this work.


(In other words, this statement should change nothing except the default value)


I am going to do some changes in this script, but later ;-)


Another proposed "sharpened" ic_launcher.png


WTF


@garvinhicking Do you have an idea how to test this? Right now, I'm not even sure that initTables is even called in an upgrade. How can I simulate an upgrade of a plugin via spartacus?


ok, added them back.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1165
success: http://build.servo.org/builders/linux2/builds/1163
success: http://build.servo.org/builders/android/builds/1199
success: http://build.servo.org/builders/mac1/builds/1378
success: http://build.servo.org/builders/mac2/builds/1381
success: http://build.servo.org/builders/gonk/builds/347


"WTF" ?? I am Pro!!


TeamCity xBIM :: XbimWebUI :: Build [Build 45](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1228&buildTypeId=XBIM_XbimWebUI_Build) is now running


TeamCity xBIM :: XbimWebUI :: Build [Build 45](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1228&buildTypeId=XBIM_XbimWebUI_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:19


I just copied Lindsay's text change. I'll add [Agency]


8.3-Release-WrapUp (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1668 of 1670 tests passed)

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


LGTM


This is prob my bad, I think I forgot to migrate everything after I changed this.


Can we separate them on each files? I.e.:

- UserService.js
- AuthorizationService.js
- AnnouncementService.js

Having different services in one file can lead to a very big file. Just an idea.


Yeah, LH wanted to drop the "Field References" term and rewrite the whole paragraph. Something to talk about later.


@khadijaibrahim @samaasallam


Same as commit 7307edb825041cade4c2b85f138749aa7e3d65aa. Can we separate them into different files? And rename this file to HeaderController.js.


Sharpened version of ic_launcher.png


Cada ciclo de lançamento do WordPress é liderado por um ou mais entre os principais desenvolvedores do WordPress. Um ciclo de lançamento geralmente dura cerca de 4 meses, a partir da reunião inicial de escopo até o lançamento da nova versão


fast-forwarding master to auto = ac786e29


Has graphite given you troubles? I may add your nice customizations after I get my 5.1 booting. Thank you.


@edwtjo


These are still uninitialized


Sorry about this, but it seems like there was some slight miscommunication; my name is Chris, not Nathan. Wasn't sure on the call that the comments directed at "Nathan" were to me. :)


not implemented yet, so commented out


Just checking as it's not clear to me (!!) that the latest version  62ab7c0 22 has been amended after the observations of @crunnells


Good job.


c++03 compatibility


<p>When this limit...


r+


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/6ea514bc065fddec305a45814f42e94ab7d33102


merging Ms2ger/servo/update-js = 6ea514bc into auto


Ms2ger/servo/update-js = 6ea514bc merged ok, testing candidate = 37a98e40


Hmm fine I guess, though the static method solution might have been less
verbose if we add more targets. I will go with whatever @alexmarsev agrees
with since I don't really have a strong opinion myself.


Commit the generator script too, maybe with a comment inside describing how to download the reference compiler instead of including it as a binary blob. I mean, as long as it's separate and optional it's nice to have.


@vjeux can you publish a new react-native-cli version?


Não era pra fazer isso, vou tentar corrigir meu último commit.


Thanks - I updated the text to more accurately reflect the process you have described.


Dirty and beautiful. I love it.


don't worry, I'm not going to revert something, just for personal reasons without have talked about this first, thats would be totally wrong from my side.

is just that I wanted to clarify few things about acronyms and uppercase letters, before it gets merged in the main repo, thats all.

now you have clarified all the things, so there nothing more to discuss, I guess.


Thanks @braver !


I think we have bug here, default value and value in this copy is the same, so there is no way to set different value that false
Should it be true here?


true - thanks. Just a note: I wrote it that way so that the computer generated Captivate voice would recognize it. :0


Neat trick, @rwaldron.


I can work on that.


I tried applying this patch, but I still have the amount displayed in default currency instead that the actual one...


Oh, yeah. The commit with `update_x86_rounding_mode` has those comments too. For consistency I'll add them here.


I don't think you need the whole `defined __cplusplus` part, your code SHOULD stop compiling if used in C.
I'd recommend renaming the file to "debug.**hpp**", to remove any ambiguity, as to whether it's C or C++.


FYI: I would actually argue that it is useful to have this public, as part of the API. Since the outer class is non-public nobody can use the public methods/classes anyway. If one day we wanted to expose this class (e.g., making it part of Netty) then we would only need to add public on the outer class.

I know this isn't a common practice and is much more nuanced than most developers tend to care about/spend any mental capacity worrying about. So don't bother changing it, but I did want to point it out.


Fixing. Is Travis correctly named?


This is just a formatting change.  Commit hashes are displayed with an extra set of paren's on either side, however I have already included a set of paren's in my `.bashrc`.  So I removed this specific set here, to prevent two sets of paren's from showing up.


oh hai rick


I saw that but again, it's "Socket activation for developers" and talks about the guts. But I guess it's better than nothing.


This was, of course, accidental :)


giri giri chop has .9 scroll speed, so it's valid minimum I've put here, but haven't fixed manifest visual density(?) generate, if you'd like to investigate, please do :)


There isn't a `0.1.0` tag yet, so this breaks the build.


I would rather find out why the table doesn't exist versus manually inserting these rows, only liquibase should write to this table, we should never do have to do so


this is really shared? @fguth


Não deveria ser `br.com.recursive`?


Am Thu, 26 Mar 2015 11:01:52 -0700
schrieb Daniel Hahler <notifications@github.com>:

> Can this be made to fallback in case of older cmake?
I do not now, if that is possible. In case of Debian: testing is soon
to become Debian Stable, so it should not be a problem there.
Furthermore, packagers could always revert the patch and users who
compile awesome themself should also be able to do that, if it is
necessary.

>
> As for Ubuntu/Debian, cmake 3 is rather new (but easy to upgrade):
>
>       Version table:
>          3.0.2-1ubuntu2 0
>             151 http://archive.ubuntu.com/ubuntu/ vivid/main amd64
> Packages 3.0.2-1 0
>              50 http://ftp.de.debian.org/debian/ unstable/main amd64
> Packages 150 http://ftp.de.debian.org/debian/ testing/main amd64
> Packages 2.8.12.2-0ubuntu6 0
>             152 http://archive.ubuntu.com/ubuntu/ utopic/main amd64
> Packages *** 2.8.12.2-0ubuntu3 0
>             500 http://de.archive.ubuntu.com/ubuntu/ trusty/main
> amd64 Packages
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/juw/awesome/commit/45778a7ca4c7b231bae1ad718c2bf8ca7c2399f8#commitcomment-10417999


In my bashrc, I have an entry that looks a little like this:
```bash
...

# enable programmable completion features
# This was copy-pasted from later in this .bashrc, to this location, so that I would
# have `__git_ps1` available
if [ -f /etc/bash_completion ] && ! shopt -oq posix; then
    . /etc/bash_completion
fi

# If git isn't installed yet... these may fail
export GIT_PS1_SHOWDIRTYSTATE=1
export GIT_PS1_SHOWSTASHSTATE=1
export GIT_PS1_SHOWUNTRACKEDFILES=1

if [ "$color_prompt" = yes ]; then
    NC='\e[0m'
    C11='\e[1;32m'
    C13='\e[1;34m'
    export GIT_PS1_SHOWCOLORHINTS=1
    PS1='${debian_chroot:+($debian_chroot)}\['$C11'\]\u@\h\['$NC'\]:\['$C13'\]\w\['$NC'\]$(echo -en `__git_ps1 ":(%s)" 2>/dev/null`)'$NC'\$ '
else
    PS1='${debian_chroot:+($debian_chroot)}\u@\h:\w$(echo -en `__git_ps1 ":(%s)" 2>/dev/null`)\$ '
fi
unset color_prompt force_color_prompt

...
```


Nome minúsculo!


OK @juhan, you're right. Please check #115 and tell me if that works for you. It does for me. @nilsbore, sorry I seem to have created a problem...


it is. titles are all the same on every section. there was no reason to have different markup and style for each.


I was working on just removing the hyphens from the project name. I was kinda thinking that just saying no was a little harsh when the cli tool is named with a hyphen! :)


See #20


That broke quite few of my hydras. Removing `.` from there fixed it. Does that `.` need to be escaped?


I see you started already, hue hue


maybe `git push origin --tags`?


lol yeah. But I think it's too magical to auto fix it for you. Rather just say no than do something unexpected.


@angieelfin Wrong indentation (change editor settings). Remove empty line.


Apologies for not having answered before! Is right you indicate me, I had not noticed. I will make the correction! Thank You!


thanks!


Ha! fair. I did put a log message but you're probably right. Cheers.


Assuming there's no real reason to keep the storageKeyOfBookFolder in a temporary folder name, lgtm! This folder name has 57 characters.


It's required for PS3 edat support, why this thrown away ? :)


I never found out why. Fee free to look into it.

On Thu, Mar 26, 2015 at 9:26 PM, Wyclif Luyima <notifications@github.com>
wrote:

> I would rather find out why the table doesn't exist versus manually
> inserting these rows, only liquibase should write to this table, we should
> never do have to do so
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-standalone/commit/e4e7d927239a45294672bf8a9f4d5e8d30d7cb0c#commitcomment-10418533>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


Sounds good


давайте лучше будем явно коннекты писать руками, и слоты назовем по стайлгайду


You're welcome! The pleasure was mine.


Yes, he is.

Thanks!


Easier to search for, and I have like 20 matches in code for `HttpStatus.`.


_posts is not yet a collection, so Jekyll will flip out if you have it specified as one I believe.

The `permalink` directive should work in 2.5, but may not in 2.4. You're using 3.0 beta2? Hm...


(Most in tests.)


These should be full words, not dash separated words. Also please make sure you're using the images you've uploaded


Same here


Hmm.  I don't like OAuth, but I'll acquiesce.


And here


And one more time


Beim Scribus-Start wird ein Dialog gezeigt, womit ein neues Dokument erstellt werden kann. Falls dieser schon geschlossen wurde, kann er durch das Menü _Datei > Neues Dokument_ wieder angezeigt werden.


I have no time to look into this, all am saying is that we should not be writing to the liquibasechangelog table


Remove from app namespace.


cc #2 for additional content when this commit goes away from a force push


@fguth @gibatronic why is hidden? it's a unnecessary class, use media queries to handle that and don't forget to be mobile first.


For the Watson Color Swatches, text shouldn't be included. Please remove the text from on top of the color swatches. Please also break them up into their own bits in the file so we don't have instructions in the images. Best practice: never include text in images.


I would create a bug for this case


@fguth take the time to remove all the unnecessary classes: section


> imagescience is officially developed in and distributed by my group. The Fiji team has permission to ship it with Fiji.

To represent that better, we may want to create an `imagescience` GitHub org, for which you are the sole admin. Then you can do releases only when you want to, and update the version in Fiji only when you want to, by submitting PRs to the Fiji project. That way, the Fiji maintainers cannot cause any interference in your plans, and the project organization will better reflect reality.

The alternative—having your project hosted in the `fiji` org—implies that the Fiji maintainers are responsible for keeping your code up-to-date with Fiji dependencies, cutting new releases, etc., which does not sound like our current understanding as you describe it.

> Why does the version number of a package need to be updated if nothing changed in the package itself?

Firstly, by "package" you mean JAR library, right? Not Java package?

As @hinerm mentioned, [a similar discussion arose recently](http://imagej.net/pipermail/imagej-devel/2015-March/002481.html) and we concluded that you're right: the version number should usually only change in response to changes within that project's code only. There are still edge cases (e.g., same source compiling to different bytecode), but I expect that in the future they will be quite rare.


I think that a de-sluggification should be okay and predictibale...and this technically prohibits valid utf8 characters like ☃ :smile:


I see that there's an ASE file as a resource, but it's not uploaded/linked. Can we get this uploaded/linked before moving forward?


"# flake8: noqa" tells flake8 to skip the whole file.  "# noqa" tells it to skip a single line.  I suspect that is what you meant to have occur here and below.


@emeijering @hinerm @ctrueden according to our refined understanding of the what SemVer can do and what not, the tertiary (PATCH) version number deserves an upgrade when dependencies of a project are changed (e.g. ImageJ version update) as this qualifies as a patch.  I guess the 2.5.0 upgrade predates this insight and now the discussion is about how to deal with the fallout.

The short summary of the SemVer discussion is: Version should speak about changes to the public API of an artifact. Note that this is the best SemVer can do.  It is still largely impossible to reason whether it is safe to consume an update or not.

I cannot say anything about how to deal with the situation here.  The upgrade to 2.5 was a mistake. One could consume it or fight it.  Compare

https://github.com/fiji/legacy-imglib1/commits/master

and the discussion

https://github.com/bigdataviewer/SPIM_Registration/issues/10#issuecomment-78988053


Since you are making a manual change anyway and there is no edit for the pools, just set the mlimit to 100000.  I just tried it in a VM and it works fine.  Could even change the plugin to allow this.  Then you don't need to use a percentage or change the units.

-----Original Message-----
From: "nohatssir" <notifications@github.com>
Sent: Thursday, March 26, 2015 1:08pm
To: "OpenMediaVault-Plugin-Developers/openmediavault-unionfilesystems" <openmediavault-unionfilesystems@noreply.github.com>
Cc: "Aaron Murray" <aaron@ryecoinc.net>
Subject: Re: [openmediavault-unionfilesystems] add mlimit for mhddfs (11f8b5b)

It lacks ACLs.

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenMediaVault-Plugin-Developers/openmediavault-unionfilesystems/commit/11f8b5b15103fdcd9b09041f3ea99ae1846abcc7#commitcomment-10418162


I completely agree. I was taking too much time without a break through. So
i eventually did this to unblock Joseph. I would gladly call upon another
pair of eyes to help look into it.

On Thu, Mar 26, 2015 at 9:32 PM, Wyclif Luyima <notifications@github.com>
wrote:

> I have no time to look into this, all am saying is that we should not be
> writing to the liquibasechangelog table
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-standalone/commit/e4e7d927239a45294672bf8a9f4d5e8d30d7cb0c#commitcomment-10418677>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


look right where I commented: .section-culture__more, is it?


Unfortunately the stage reading list endpoint expects a full stage cluster, including stage oauth.  I'd like to move this into the Firefox Account datum, but it's low priority.


I think we should just put in the documentation the possible scenarios when it returns Not Authorized status code. I don't want to give a specific message for each case.


Please also include http://leaverou.github.io/contrast-ratio/ as a color contrast resource. It is much more robust and includes estimates for transparency


How in the name of all that is holy did tab literals get inserted?


@HaraldZealot что то не получается нормально строку с пробелами ввести....


Can you put a code comment next to those occurrences and briefly explain what is going on there? A year down the road, nobody will remember :sun:


The text on the color swatches is (mostly) not accessible. As per the [design language](http://www.ibm.com/design/language/framework/visual/color.shtml) anything <50 cannot have white text on it and anything >50 cannot have black text on it.


Great, thank. Really sorry about that: https://github.com/joyent/nodejs-advisory-board/pull/32


OK


@Nevcairiel: What does this do exactly? Does it ensure that subtitles are correctly demuxed if you seek directly at time when a sub is supposingly already being displayed?


Quick note so we don't miss a "# flake8: noqa"


The Data Scientist's Toolbox Course Project


Or else kill it at the source:

```
# spec_helper.rb

def ActiveRecord::Base.connection.quote_column_name(name)
  @quoted_column_names[name] ||= name.freeze
end

def ActiveRecord::Base.connection.quote_table_name(name)
  @quoted_table_names[name] ||= name.freeze
end
```

(Or maybe just for MySQL since that's the weird one?)


fgets??


Indentation


No need to `.to_s` here, right?


I think race condition will be excluded partially with skipping .run() when job is disabled. Also i think this happens because `checkTriggers()` is not synchronised with `.start()`


Correction: "Issue #26" in commit notes should be issue #24.


CK_NOT_DESIGNATED_INITIALIZER_ATTRIBUTE carries a little more semantic in the message it emits saying it's not the designated initializer. That's pretty much it but extra semantic in errors is always nice :)


why we have classes that are not titles? .section-culture__more, .section__arrow, ...?


`display: block;
 zoom: 1; ` etc.


No need for `.to_s` here, right?


This is bad. This is DiscoveryPageStore's concern. Moreover, why fetching all fetchStorefrontListings on a review change? Just fetch the listing that changed.


This shell-script is good overall. Can you add a couple comments here about why each of these steps is necessary?


se pare ca daca listele sunt goale si taburile sunt selectate da eroarea java.lang.IllegalStateException asa ca am pus un try catch


Проста калі пачынаеце працаваць з fgets,  то паўсюль трэба карыстацца fgets


Oops, sorry, I moved my comment to the pull request as it seemed more appropriate.

In any case, thanks for the context.


For me compile time expansion is a lot better than static switch case. But since @alexmarsev has an opinion on everything, he will surly tell us :D


the same + space after : `margin: 0;`


Line 184 breaks the 80 column limit. Aside from that, all of these look like really good changes.


@cl-a-us: In the original layout there was some kind of highlight box. Do you know, if this highlight box is available for our pages?


Set proper indentation (2 spaces)


gh


Remove empty line


like


I'm not sure what you're suggesting?  `SignerListSet` transactions are the first time we increase the `OwnerCount` in steps larger than one.  So I added this `increaseOwnerCount` method that allows me to add `howMuch` to the `OwnerCount`.  This specific check is looking for overflow of the 32-bit `OwnerCount`.  We test for overflow by seeing if we add `howMuch` to `current_count` (resulting in `new_count`) and get a result less than the old `current_count`.  Are you suggesting a better way for me to do that check?


Wrong indentation, read about nesting elements


Why not upgrading to 4.0.0.RELEASE?


testy systemu code review


Could you use `notes`in the loop since it's declared before and squash the commits?


the same situtation


That could work. In that case I would remove the limit though since HDD sizes continue to grow. Another way to go would be to allow the input with the unit like "1000g", "20%", "500m". Validate it with a simple regex. That way it's kind of flexible but doesn't add more fields. Thoughts on that?

I'm not sure which units are available at the moment, but the regex would look like this I guess:
```
/^\d+(m|g|%)$/
```


Hey Falko,

My submission was for the Arrays only! I'm still working on the other two (Dictionaries & NumberHandler). Kind of struggling with those ones, but I appreciate your feedback.


3 empty lines


Empty lines


mmm. ,, not sure why i was doing that! ,, but of cures u are right
the acutely work on this lib was in the RoR project ,, but it's in a private repo

here is a snippet on what I was working on
```html
 <script>
 "use strict";
	document.onreadystatechange = function () {
		if (document.readyState == "complete") {
            EPUBJS.filePath = "js/libs/";
            EPUBJS.cssPath = "css/";
            var reader = ePubReader('<%= @epub_path %>');
        }
    };
</script>
```


@parkr: yes, they don't work in 2.5.3 either in 3.0.0-beta2. But, I set `date` in the front matter for updates yet `:date` doesn't work either.

So, to summarize, here's the situation here:

1) We need `posts` to be rendered in `/blog/:year/:month/:day/:title`.
2) We need 2 other collections to render in their specific folder like `/updates/:year/:month/:day/` and `/newsletters/:year/:month/:day/title/`. Can't make it work due to the issue with `permalink` not picking up `date` from the front matter.
3) We need pagination for posts and collections but in different folders. Say, `/blog/page/:num`, `/updates/page/:num` and `newsletters/page/:num`

The problem with 1, is that if I specify `permalink: /blog/:year/:month/:day/:title` globally in _config.yml, then the other pages will not render as pretty :/

For the 2nd issue, I guess I could specify the permalink in the front matter since that seems to work. Though it's not foolproof unfortunately. That is why I wanted it automated.

For 3, I have no idea how to fix this unfortunately :/

Sorry for the long post. I really appreciate your time.


I totally agree with you :+1:  I'm hoping to redo this pull as soon as possible :)


This assignment was focused around utilizing a varied Huffman encoding scheme to compress literature. The goal was to utilize our own data structures as often as possible.


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


@fguth @gibatronic care to dont add unnecessary complexity, maybe we only need a base/general/shared.scss file instead of a file for each kind of element.


empty line. Read about CSS3 Vendor Prefixes


#2727

I guess they're going to do that in 1.3.0.


@mattleff This should be `border-radius: 8px 8px 0 0;` so both corners are rounded, right?


The compiler might very well do that for us and just inline the simplified function call. Anyway we probably won't have many targets anyway so I guess verbosity isn't so much of a problem.


@lizlux, I know this has been removed now but FYI, you can condense this and make the regex execute faster by combining common components, e.g.:
```php
$moduleBody = preg_replace('/<\/?(html|body)>/', '', $moduleBody);
```


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


TeamCity xBIM :: XbimExchange :: Build [Build 156](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1229&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:47


This allows for correct displaying when adding a new sobject with more tasks than already exist.


Okay, thanks.  See commit 4a2ff2b7f27e569032f6c5881e17cb053b32ccc7.


blz.. valeu pelo help :smile:


fghf


I guess that makes sense.


TeamCity xBIM :: XbimExchange :: Build [Build 157](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1230&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:34


Would it look like this?
![brand color meaning](brand-color-meaning.png)


Gah. I thought I pulled this one with `--rebase`. Oh well :P


@jonatack Removed `to_s`, previously was doing `include?` check which needs version to be `String`. Thank you!


&& src[i] != '\0'

but otherwise, looks good.

On 26 March 2015 at 13:49, Fabian Greffrath <notifications@github.com>
wrote:

> How about something along these lines?
>
>     size_t i;
>
>     for (i = 0; i < dest_size - 1 && src[i]; i++)
>     {
>         dest[i] = src[i];
>     }
>     dest[i] = '\0';
>
>     return src[i] == '\0';
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chocolate-doom/chocolate-doom/commit/62b5c602821cee8ac703ebe6362f3e1fc6d9ad3c#commitcomment-10417741>
> .
>

--
Simon Howard
https://soulsphere.org/


maj


Wrong indentation, read about proper nesting. ` <div><p>Paragraph</p></div> ` is not readable. Try
`<div>`
`  <p>Paragraph</p> `
`</div>`


not marking it as a test is a bad option. If the test cannot pass, you should rather mark it as incomplete so that it at least appear in the phpunit output


I understand @wluyima, however we failed to establish the reason why the changeset was not executed, so @dkayiwa's commit helped alot since we are almost running our of time before 1.11.1

We can monitor this to see why, by the way, it looks like this was happening when checked out the tag, the standalone built well on 1.11.2-SNAPSHOT


That is ok, I just harmonized the two types for you (blue and white buttons)
Btw.: I didn't invented the layout of the buttons.
I can live with fakelinkbutton class here.


Did you mean to delete radtrack/ui/LaserInterface.py or just rename it like radtrack/ui/BunchInterface.py?


I did it inline to make it difficult to handle that error. It's really something just to prevent a panic rather than something that should be used in regular use. The caller really should be validating the server list is not empty if there's a possibility of it.


It isn't "that" much more verbose...


I'm curious, did you feel that it was cheating because the students can see each others code in an open workflow? I could see how from a professor's perspective, that'd be a barrier. When a student finds employment at a startup or software-based company, the same open workflow becomes the basis for self-driven learning and collaboration—at the speed of light.

Thank you so much for attending. Please, feel free to contact if you have any ideas or questions regarding GitHub or an open design workflow. I hope to lead more intermediate workshops in the near future.

:metal:


The other setting on opts is lost here, such as moduleName
So, when I run the command with "-o abced.js", the generated file still use "parser" as the module name.


@Snugug for the color swatches, I know the final site won't have the text on the image, this was just a mockup I did to show what the visual example would look like. I intended for it to be rendered out in css. Should I just delete the image, and write that as a place holder?


El index del wall quedo feo con esto, @ballcue -.-


Why not keep the number field and add a combobox next to the number field with a g, m, and % in it?

-----Original Message-----
From: "nohatssir" <notifications@github.com>
Sent: Thursday, March 26, 2015 1:44pm
To: "OpenMediaVault-Plugin-Developers/openmediavault-unionfilesystems" <openmediavault-unionfilesystems@noreply.github.com>
Cc: "Aaron Murray" <aaron@ryecoinc.net>
Subject: Re: [openmediavault-unionfilesystems] add mlimit for mhddfs (11f8b5b)

That could work. In that case I would remove the limit though since HDD sizes continue to grow. Another way to go would be to allow the input with the unit like "1000g", "20%", "500m". Validate it with a simple regex. That way it's kind of flexible but doesn't add more fields. Thoughts on that?

I'm not sure which units are available at the moment, but the regex would look like this I guess:
```
/^\d+(m|g|%)$/
```

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenMediaVault-Plugin-Developers/openmediavault-unionfilesystems/commit/11f8b5b15103fdcd9b09041f3ea99ae1846abcc7#commitcomment-10418956


`.` shouldn't be escaped if it's inside `[]`. Give me one broken example so that I can see what went wrong.


You should be getting the CVE placeholder at that URL, although I agree that's not helpful.

Official announcement is here:

https://groups.google.com/forum/#!topic/httprb/jkb4oxwZjkU

We can switch to that


![screen shot 2015-03-26 at 1 53 26 pm](https://cloud.githubusercontent.com/assets/9538528/6854473/8c15aec2-d3bf-11e4-8621-fa0a61028fab.png)
Here it says 50 is accessible with black text


This is on Windows/MinGW.
I merged this commit, rebuilt, and noticed there was no more music in the menu screen. And I got these debug messages:
```
Initializing resource manager and core classes
Game data directory: D:/TSC/tsc/data
User data directory: C:/Users/Georger/AppData/Roaming\tsc
User cache directory: C:/Users/Georger/AppData/Roaming\tsc\cache
User config directory: C:/Users/Georger/AppData/Roaming\tsc
Initializing Package Manager
Configuration file is 'C:/Users/Georger/AppData/Roaming\tsc\config.xml'.
Translation locale set to 0 0
Translation locale is Portuguese_Brazil.1252
Translation support with gettext set to:
        Directory D:/TSC/tsc/data\translations
        Codeset: UTF-8
        Text domain: Secretchronicles
CEGUI log file is at 'C:/Users/Georger/AppData/Roaming\tsc\cache\cegui.log'.
Loading campaigns
Loaded campaign file: D:/TSC/tsc/data\campaigns\smc.tsccpn
Loaded campaign file: D:/TSC/tsc/data\campaigns\tutorial.tsccpn

Setting up level player
Loading levels
Applying preferences
Loading world from directory 'D:/TSC/tsc/data\worlds\test_1'
Loading world from directory 'D:/TSC/tsc/data\worlds\world_1'
Loading world from directory 'D:/TSC/tsc/data\worlds\world_2'
Loading world from directory 'D:/TSC/tsc/data\worlds\world_3'
Loading world from directory 'D:/TSC/tsc/data\worlds\world_4'
Loaded level: D:/TSC/tsc/data\levels\menu_brown_1.tsclvl
Preloading images and sounds...
Done preloading images and sounds.
** Couldn't load music file : D:/TSC/tsc/data\music\game/menu.ogg **
Wrote preferences file 'C:/Users/Georger/AppData/Roaming\tsc\config.xml'.
Unloaded level: D:/TSC/tsc/data\levels\menu_brown_1.tsclvl
** Last known SDL Error : Not an Ogg Vorbis audio stream **
```


Superflous semicolon?


Most of the variables in these loops are ambiguously named. Changing them to more descriptive names would increase readability and reduce bug-inducing hazards.
Ex: p -> each_marker, q -> some_iteration_i_didn't_get...


This version does "one-tailed" analysis, for lack of a better term


Don't we want to ensure that each projection is in fact unique across the whole model? It would appear that you are creating here a project of type and then in line 447 you do the same. If the two type arguments are equal, then we now have two equivalent projections for the same type.

I think a model-level ProjectionResolver is required.


Great!


Fix code style (indentation, space after : )


bad bad bad


This will allow us to do retries without string-parsing the error to see what the error code was


You can get rid of this :+1:


8.3-Release-WrapUp (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1168 tests passed)

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


This loop splits up tsv data by marker, correct?


And i loose motivation to work on flyspray if someone makes changes without discussion on things I made before with something in mind.


If we introduce a model-global projection resolver, this may not be the best place for this.


OH GOSH LAGGGGGG?


Review complete


this should probably use the now index instead of the ordinal value


Sorry, should have been more clear. The secondary colors is where I'm seeing issues. You're good on the primary and greyscale colors


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


As in previous site indentation, empty lines


Jcecconi: Here you have a code to remove comments in several lines, do not like too much the "while" i use, but....

         if ( strpos($this->lines[$l], "/*") !== false && strpos($this->lines[$l], "*/") !== false){
/*remove one line comment*/
            $this->lines[$l] = "";
         }else if (strpos($this->lines[$l], "/*") !== false) {
/*remove multiline comment*/
            while(strpos($this->lines[$l], "*/") === false ){
               $this->lines[$l] = "";
               $l++;
            }
            $this->lines[$l] = "";
         }


For what are those empty lines ;) ? Fix indentation, nesting


I'm fine with all options. Just thought it would require less changes to change to a regex. Choose which one you feel is best! :)


`![Brand Color Meanings](/images/brand-color-meaning.png)`

Although this will be another thing I'm going to update to CSS, yah?


TeamCity xBIM :: XbimExchange :: Build [Build 158](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1231&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:02:04


Por favor dejen de romperlo :rage4:


You can actually just add this to the development group below line 51. Either way, it still works!


@battlemidget -- just noticed that this went away, months later :\
This fades the green color around the deploy button from bright green to light green and eventually the highlight goes away. How does it break headless mode?


This change was made in 1.1.x where moving from 3.2.x to 4.0.0 really wouldn't be appropriate.


If we move to a global-model projection resolver, the type/capability matching logic would be handy to have in the projection class itself.


you mean it can't have white? if the text is black 90, and the background is blue 40: 90-40=50


In addition to the line comments, the velocity calculation section could be commented better to make it easier to work on. @wjosam87


@HaraldZealot  а как тогда лучше ввести вопрос?


VSHPWH (shenb) - i386-Windows-7-VisualStudio-12: Tests Failed

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


The compilation logic is quite related to colorforth, and it's more in that vein than any other, inching toward what I might have called spokenforth. So if your view of Forth is ANSI - no. But yes, it is forth.


Spanish translation updated.


They can have black text, but not white text, yah.


This revision is clean.


is this better?
![watson-color-swatches-08](https://cloud.githubusercontent.com/assets/9538528/6854727/fde96934-d3c0-11e4-8b7e-31b5592cb081.png)


Yup


can I export these as one image since it is a placeholder for rendered swatches? Or do you need the individually ?


We don't actually need this line.  Stop.js will be included elsewhere.


Whoa dude, that is such an awesome idea :D


One's fine!


Just got CAF MDSS booting and all lag at boot is gone.