ahh m_audioClock != m_clock

m_audioClock is a pts driver to audio packets having no pts itself. this is not what's desired here.


Is this something for upstream?


Ah okay - that makes sense then!


Ok, I did it for you in this branch:

https://github.com/Immemor/RaidCore/commit/31c8c856250c2a1409c5f04351d67eaccce4038e

But you must replace the "XXXXXX" by the keyword 'cancel' in Chinese.
From which region in China are you ? I went many times for my previous job in china (Beijing, Chengdu, hangzhou, etc...)


The card image displayed in the explorer, deck editor and duel decks screens is now obtained on a non-EDT thread. This makes the UI much more responsive. To see the difference, open Explorer and scroll through the list of cards by holding down the the up/down arrow or page-up/down keys. Most notably it prevents the long CardBuilder initialization process affecting the UI - currently in 1.69 if you are scrolling through the images and it selects a missing image the whole UI freezes while CardBuilder starts up.

Part of this process is to display a new throbber (I know, but that is the [official term](https://en.wikipedia.org/wiki/Throbber) apparently) widget I have had lying around for ages. This lets you take any image and animate it by rotating it continuously.

![screenshot00432](https://cloud.githubusercontent.com/assets/2443519/12392253/80405b90-bde5-11e5-8311-6a129b5dcb82.jpg)


Wait :-) you assign two times m_audioClock ... still  confused I am.


Should we be submitting these for upsteam?


According to UN, the official name is still FYROM,
former Yugoslav Republic of Macedonia.
http://www.un.org/en/members/

You are right about Iran. I will correct it.


因为该函数的调用者为游戏的主线程，主线程的优先级为 10，而新线程默认是以父线程的优先级作为优先级的，也就是说这个线程默认的优先级也是 10，所以在这里限制到中间的优先级。


Why not make it private instead? This way you are exposing a public property called `_model`.


Why `new`? Should we only be catching `Default::default`? And if so, we don't need the implements_trait check.


@wsdjeg OpenSUSE


Get this spec failure:

```
Failures:

  1) ResponseController when submitting a followup message logged in as the request owner when not sent preview or reedit params displays an error if the request has been closed to new responses
     Failure/Error: expect(deliveries.size).to eq(0)

       expected: 0
            got: 1

       (compared using ==)
     # ./spec/controllers/response_controller_spec.rb:193:in `block (5 levels) in <top (required)>'

Finished in 15.03 seconds (files took 26.53 seconds to load)
22 examples, 1 failure

Failed examples:

rspec ./spec/controllers/response_controller_spec.rb:184 # ResponseController when submitting a followup message logged in as the request owner when not sent preview or reedit params displays an error if the request has been closed to new responses

Randomized with seed 46647
```

I don't get this spec failure in the previous commit (d1320ee0060ef4c30b679725686537ba57de0429), but its the same failure as https://github.com/mysociety/alaveteli/commit/7e0477b402cec6f3241f939db3be7ecf6a5cc9bd#commitcomment-15481085


1. 如果二個數都一樣會顯示什麼
2. 你其實c不需要給default 值
3. 變數請命名有意義的字 不要用a b c 這種
4. test result 請想盡辦法使用各種組合

基本上 ok


try this

> 生成大端序模式的代码


evtl. sollte in in apply_profile per default auf etx_ffeth schalten wenn kein wert gesetzt wurde.


I think this should use `get()` instead of `find()`. The `ideaModules` element must be there: if it's not, that's an error. Conversely, setting `_model.project.modules` to `null` makes no sense (and would most probably result in an `NPE` further down the line).


wow, @behdad222


From looking at the test there's absolutely no way of telling what's the actual interpreter used, besides putting your faith in tox. So I thought it would be nice as a simple smoke test that ensures the version you asked for is used. Since Travis writes "Python 3.5" on all tests.


Wtf


This page is just for record something.


Your the best!

Thanks JC.  This is wonderful to have your help.


This afternoon I'll compile the CM kernel with the patch and I'll compare both kernels.


Sure, then why not :)


Are you sure this works with at least gcc and visual studio?
I usually use ```#elif defined(_X86_) || defined(__i386__) || defined(__386__)``` which should work with gcc, vs, clang and mingw afaik


I was wondering if we need device for the login, if I run rake routes it's device(15) against Monban (6).


mmm in my mind I thought:

1. move https://github.com/mozilla-b2g/gaia/blob/002fc618166f2887753b254a8b5e7d6195a86ed8/apps/keyboard/test/marionette/dismiss_test.js#L27-L33 into the System app's lib/keyboard
2. Change the calls in the System marionette test to use the copy in system's lib/keyboard
3. Load it directly in our test and use it

So that we mutualize things :)


這邊不能改成 1~4, 因為使用
slickGoTo 時，後面接的是 0~n 所以一定要從0 開始


I lkie this :smile:


This actually doesn't change anything at the moment, do you intend to change `format_currency` (or allow the number of digits to be changed)?

Anyway, it's kind of weird that the holdings table is already formatted, we parse it back to a float and then format it again. I think I'll change it in the API so that the API consistently spits out Decimals.


@kwin while that will technically work, using an optional dependency in a DS component causes a really ugly stack trace in the log. Not to mention that the configurations would not be portable. And again I think working around that issue would lead to very hard to read code. And yes, the issue is with a post-6.1 update to the scr bundle, not the version actually shipped in 6.1.


But what do you think, is it better to move thing towards c++11 or there are some applications/usages in which the c++11 standards may pose a problem?


I don’t know, I thought that suggesting `unwrap_or_default()` when possible instead of `unwrap_or_else(Foo::new)` was better. It’s also probably safe to assume that a parameter-less `new` method will return the same as `default`. But I can amend that it if you want.
The `implements_trait` function can be useful in other lints too, eg. #498 or https://github.com/Manishearth/rust-clippy/issues/416#issuecomment-151633544.


We should use a library for generating CSVs. Look at Super CSV library.


Fine with the document, I'm just wondering from development perspective, what does it return? Can you also write tests with adding specifically audio files please? I will need to work on Community Maps supporting them, so making sure it works on the other side, not just importing from Sapelli, is importing.


Well...I know what was the only way i figured out to create a new shape that shares a rootshape. It is not possible to do it with simple transitions because of the sharedData.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 133](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=689&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


No it wont.  Just the same as how G0 F200 sets G0 moves to have a 200 feedrate for the rest of the job
And G1 F1000 sets G1 moves to have 1000fr for the rest of the job

Each stores their own S/F/POS data


bør count være 2?


? It's just obsolete and i suffer from OCD. :roto2:


awesome :smiley:


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 133](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=689&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:34


@jancborchardt *push*


Sounds fair


Este tipo de literales debes guardarlos en atributos de la clase (si vas a utilizarlos desde fuera pueden ser static).

```
private String PRINT_MESSAGE = "Documento A3 para carpetilla de control gestor\nse ha guardado en su directorio \"Borrame\"";
private String PRINT_MESSAGE_TITLE = "Impresión de documentos";
```
Luego cuando los utilices queda mucho más limpio

```
showMessageDialog(null, PRINT_MESSAGE, PRINT_MESSAGE_TITLE, INFORMATION_MESSAGE);
```
De todas formas lo ideal sería tenerlos en catálogos de mensajes localizados :)


Nice :+1: good work


No, but you  pushed with it spelled wrong (;  Is spelled as ExpErimental, and you have it as ExpIrimental  (E vs I)


moreover as I said (maybe not clearly) you'll also need to wait for the keyboard displaying and hiding, that's why I was suggesting to add a `waitForKeyboardFrameHidden` tool in the same system's lib (and we'll need to use likely `waitForKeyboardFrameDisplayed` too, as the integration test failure seems to tell us).

Please ping me on IRC If you need any clarification.


GCC requires that variables be declared at the beginning of the block.


You know, you kind of did already, we all know BCL is crap


wasn't syntax error. Probably compiler bug or my fault ?


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 134](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=690&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


Por lo que veo este atributo sólo se utiliza en este método y no tiene getter ni setter, por lotanto parece que no  debería ser un atributo.


其實你的 terminal 那邊有報錯，sass 找不到變數哦
<img width="533" alt="2016-01-18 9 28 21" src="https://cloud.githubusercontent.com/assets/1787867/12392497/7a8624c4-be2a-11e5-9bc6-f7917586bb68.png">


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 134](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=690&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **FAILURE**
Summary: Compilation error: Xbim.COBie\Xbim.COBie.csproj (new); compilation error: Xbim.COBieLiteUK.Client\Xbim.COBieLiteUK.Client.csproj (new) Build time: 00:00:50


Hallo !
Thank you for the useful CalcArea extension.
Only frustrating limit I find is that it doesn't apply to lines. Would it be possible to add a way of calculating the LENGTH of a LINE while it's being created or edited ? That would be great :-)
Congratulations, anyway.


I just change from ubuntu 16.04 to 14.04 ,16.04 has too many bugs


Lo mismo, ¿por qué un atributo?


Thanks Adam. I'll fix that up!


Shouldn't this be
```go
if err != nil && err != distribution.ErrBlobUnknown {
```
Also the error message here could be more explicit than `Failed to find blob` (a failed search - `ErrBlobUnknown` also implies a failed search).


Pushed to ln5/tjr-tjr-grammar-suggestions-1, please let me know if you think this is not what you want.


Thanks! Will fix!


¿Aportan algo estos comentarios al final de cada línea? El nombre del método parece suficientemente descriptivo. En mi opinión solamente ensucian el código (y además requieren mantenimiento)


@pjvohra DWMR of #22. Ship code. Ship it!


Note: 2.5.3 is out.


Here is the change.


Just deleted hotplug ability. Monitoring stuff is still there.


大哥的2是甚麼意思?


Este tipo de cosas son propensas a errores y son muy feas para mantener. Si TipoContrato tiene lógica debería estar aislado en una clase y tú preguntarle cosas. Todo lo que sean tipos deberían ser enumerados y nunca Strings. La comparación de Strings es en Runtime y da muchos dolores de cabeza.


numberToStore -> numberOfSamplesToStore


- 這三行排版不對, 前面空白太多
- 跟大哥說的一樣, 變數請取有意義的
- 最後三行可以簡化成一行
```
System.out.printf("平均分數 = %.2f",(a+b+c)/3);
```
請重新上傳一次修改好的版本


unack, hey the code looks good :) However the commit message does not comply to our standards, please read up on them at the link given by GitCop. Don't forget the `Fixes <full bug URL>` note. You can use `git commit --amend` to edit the commit message.

Also you might want to adjust your author as it doesn't get recognized by github as your work.


It was a conflict cause i edited out your ## wx phoenix comments

The Rigth way to do things like that is to use the tree (gitk --all) and
inspect what's going on before issuing commands.

Metalic, so long as you insist on using those silly guis you will always
end up having conflicts like this. Look at the branch you made:
https://github.com/wrye-bash/wrye-bash/compare/metallicow-wx-phoenix-compatibility-jan16

has duplicate commits and what not....

Frankly - *use git directly:*

*git checkout dev*
*git pull*
gitk --all

Then look at the tree and try to figure out what you pulled

*Use the tree use the tree use the tree - * and let me know here with a
screen shot. Don't yet delete the rebased branch as it is correct while
what you pushed is a mess. So delete what you pushed, and post me a
sdcreenshot of your tree so I can explain to you. Doing this *despite my
very limited time* just to help you out (cause once you grok what goes on
you will be much more of a help and be able to really contribute some code)

On Mon, Jan 18, 2016 at 6:48 AM, Metallicow <notifications@github.com>
wrote:

> Not sure why this was a pull conflict. Null switcheroo commit. It said no
> can revert merged. so umm hmmm whatever.
> Anyhow looks like the branch renaming and push worked fine and is
> up-to-date again with dev. so phoenix testing can continue being hacked at
> provided something elsewhere in bash isn't broken at this point in time(my
> last attempt) with a dev checkout & drag/drop.
> Reproduce Notes to self: used TortoiseGit->Pull;
> TortoiseGit->Switch/Checkout... for this one with the create new branch
> checkbox checked. TortoiseGit resolve conflicts winmerge-like-gui and then
> Published branch from GitHub Desktop Gui. And now I'm here...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wrye-bash/wrye-bash/commit/367c2a35318051380d256a80d0524e72206e2032#commitcomment-15499141>
> .
>


Literales everywhere! Recuerda, un código limpio no contiene literales hardcodeados ni magic numbers :)


This has to be $licence->getUser()


All specs pass at this point


hehe


@teckjon Using the non-minified versions of JS files makes debugging far easier. You can flip to the .min versions when you ship. Use > `<script src="https://code.jquery.com/jquery-2.1.3.js"></script>`


This is not used.


不用給初始值就是如下, 但是我覺得養成習慣都給初始值也可以. 不然就是像我說的簡化成那樣
```
boolean c;
```


TeamCity pull requests :: netty [Build 1049](https://garage.netty.io/teamcity/viewLog.html?buildId=4855&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6381, ignored: 23 Build time: 01:26:35


is -> are


@teckjon DWMR. Nice work - thank you. Ship code.


Sometimes I think we should remake the framework to get rid of little oddities like this, then I remember I'm super lazy.


Currently, this only affects the the card image displayed in the explorer, deck editor and duel decks screens. The card image displayed is determined by the first match using the following rule
```
 custom -> crop -> original -> missing proxy
```
Please note that the crop image should be _exactly_ the same name as the original, ie ``xyz.jpg``, not ``xyz.crop.jpg``.


了解了!! 真的可以不用給耶!!


I had assumed they would be more likely to reject this as a pure windows
improvement if we can't show that it has some benefit on *nix.


This reminds me that sometimes my `nix-*` tools fail to connect to the nix daemon – probably when it isn't running, as it always succeeds when I retry.


@teckjon Spelling is often the issue - crazy but true.


Aprovecha el mecanismo de excepciones de Java. Capturar una expcepción y devolver null para evaluarlo en la llamada es más sucio que throwear la excepción y tratarla cuando la llames.

Si no vas a trararla puedes capturar la expceción checked y lanzar una unchecked (RuntimeException).

Por cierto, capturar Exception no suele ser buen negocio, debes capturar siempre al nivel más bajo posible que tenga sentido tratar o burbujear la excepción.


This is great! Thanks, Adam!


哦哦哦哦 我知道為你為什麼要設 1到4 了

0想設為首頁


Julian, is this the only change required for OLE to work on JDK 1.8? OLE currently supports only JDK 1.7 and Tomcat 7; We have to figure out and and when we can work on supporting the next versions.


"case study" => "take part"


Do not use inline CSS


navnet på monitors er ikke unikt


Un catch vacío... ¿si peta qué pasa?


Here you're sending an empty subdir if there's no subdir argument. This is how i would do it instead:
```
let attr = [{key: "folder", val: folder}]
if (typeof subdir == "function") {
   cb = subdir
   subdir = null
}else {
   attr.push({key: "sub", val: subdir})
}
```


Percibo mucha lógica de tratado de fechas... java.util.Date ya hace todo por ti. Procura no reinventar la rueda :)


Collection<String> or Iterable<String> should be used. Also some documentation would be good: What should the input format be, what exceptions are thrown when, whats the output.


Este tipo de estructuras condicionales en función del OS son bastante recurrentes... Parece buena idea encapsularlo en una clase


per step


Does not follow PSR-2 formatting rules.


Divide y vencerás. Si hay tanta lógica de comprobación puedes utilizar métodos privados para evaluar cada una de las partes. Así tendrás todo más modularizado y una visión global a más alto nivel donde puedes comprobar de un vistazo que significa "comprobar datos"


why `routed` and not `router`? is it a typo?


I still believe game should crash here, possibly with assert or exception. Reading wrong objects is of no use.


What is the reason for lowercaseing the filename?


Inicializaciones fuera del constructor


typo :smile:
`inititialization` -> `initialization`


:+1: :smile_cat:


Thanks! I was just about to do something with the Hydra failure.


Este tipo de controles se deben hacer con estados. Los estados los defines en un enumerado


Unfortunately you seem to have forgotten that realloc preserves the existing malloc'd data, new allocation or not. Yes, the behaviour of 0 is really dumb. However, when you realloc, you're not throwing the data away, you're simply acquiring more memory.


Output after fix:

# ./kafka-server-start.sh
USAGE: ./kafka-server-start.sh [-daemon] server.properties
# ./zookeeper-server-start.sh
USAGE: ./zookeeper-server-start.sh [-daemon] zookeeper.properties


@douglaslondrina .... the quickest workaround is to make a local copy of the podspec before the above change, and then to use that in your Podfile. Example:
```
pod 'AWSiOSSDK', :podspec => './AWSiOSSDK.podspec.json'
```


ack


No utilices referencias a ArrayList. Es una implementación y desde el consumidor solamente nos importa que implemente List.

Si dos líneas más abajo vas a referenciar a una lista que devuelve el modelo aprovecha la declaración para inicializar.


I'm not seeing where the default icon is provided - am I missing something?


Unused property


You are so beautiful


Looks good @skarbat , other than my minor queries.


ghvguh


yes! I'm not the only one!


Tenemos que hablar muy seriamente de fechas ;)


Some parallel work on this one ... :/  This will be available in the official 0.89, regardless of these changes. I'll decline the pull request, since the issue they handle has already been handled.


IMO it would be nice to use more explicit variable names so that one doesn't have to read the definition to understand what is being referred to in later statements. Something like ```scroll_ofs``` would likely be good if I understand the idea properly.


attento @previ che hai lasciato "saluta" qui e quindi causa errore se uno aggiorna. devi mettere ""


Typo alert!


Also it might be actually journald which is inadvertently enabling this stuff, not the kernel: https://bugzilla.redhat.com/show_bug.cgi?id=1227379

Also in case this enables the audit codepaths in the kernel system call handling (I don't know/haven't checked), we'd really want it disabled since it's a measurable performance hit: https://fedorahosted.org/fesco/ticket/1311


Cadena vacía es un literal
```
private static final String EMPTY_STRING = "";
```

Pista: Apache te da muchas de esas cosas en StringUtils ;)


Immediate changes needed in next version of YAML based on my today's following inputs and assessments:

1- Remove 404 response code from : '/conferences/{accesscode}/participants':
2- Will recheck 404 response code in complete API to list out
3- In place of 403 use 401 for all unauthenticated requests
4- JoinRequest - remove both required fields. also remove weburl from it seeing it will be automatically a property of a resource.
5- access code as string every where instead of int
6- Merge POST operation from : "/conferences/{accesscode}/participant" to  "/conferences/{accesscode}/participants"
7- Default conferencetype to AW in ConferenceRequest object
8- Revert haspassword change in ConferenceRequest.
9- Remove Weburl input parameter in JoinRequest


Try to avoid using calc value


Simplicity of Git


I think that we have to separate between the sending of feedback and the storing of SCTs and chains -- the "MUST NOT send" above isn't the real problem, is it? It's the storing of the SCTs and chains that is.


@zensh, yeah, regression on patterns with nested parentheses:

```
trie.define('(abc(cat|dog))');
SyntaxError: Invalid regular expression: Unmatched ')'
```

I'm sorry that I provoked you to release a new version... I just use route-trie in my [RotorJS](https://github.com/kuraga/rotorjs). Moreover, my library is modular so I [use](https://github.com/kuraga/rotorjs/blob/master/test/middlewares/trieTest.js) tests more than code. Well, I've made your tests standard. :smile:


krasava


Can we phrase "just received" in a way that is less ambigous?


@stefan-niedermann you can find the original at https://github.com/owncloud/core/blob/master/core/img/favicon-touch.svg :)


Agreed. My comment wasn't about the 400 it was about the text description. it seems to indicate that the only bad input parameter is a phone number? Based on your understanding and mine why does it not simply say. "400 - when any invalid input argument is specified" or something more generic. Or is the only thing that can ever be invalid here is a phone number?


Raw type warining! List y ArrayList son parametrizados, debes decirle de qué datos son tus listas


Tested. I can't connect to the login-server when event_notification set to yes?


y u so wet


I am not sure, if I fully understand your suggestion: CalcArea works on Polygon Layers, that means, an Area or Perimeter can only be calculated, when a Polygon feature exists. For this case, a minimum of three points are necessary. If you start a new feature with a first click and move your mouse over the map window, you have only a line feature until you click a second time on the map window. After that you have three points (the two clicks and the mouse position) -> Perimeter and Area are defined. Is what you mean, that CalcArea should determine the length of the line, before the second click on the Map Window occurs?

Von: GeogramBE [mailto:notifications@github.com]
Gesendet: Montag, 18. Jänner 2016 14:30
An: klausib/CalcArea
Cc: Batlogg Nikolaus
Betreff: Re: [CalcArea] New exclusions (ef12d26)

Hallo !
Thank you for the useful CalcArea extension.
Only frustrating limit I find is that it doesn't apply to lines. Would it be possible to add a way of calculating the LENGTH of a LINE while it's being created or edited ? That would be great :-)
Congratulations, anyway.

—
Reply to this email directly or view it on GitHub<https://github.com/klausib/CalcArea/commit/ef12d2668c26bc3deba0f901f9c05e606585881f#commitcomment-15506885>.


What is the reason for sampling this observable every 50 milliseconds?


If all this trace counter stuff is awkward for refactoring, then we should maybe have a separate patch to remove it since it's generally orthogonal to the overview tabs. For an interim it may be ok to just leave the one 3d trace tab.


Refactorings and improvements


try to be more specific with the variable names, try to use the full length name :)


Can we get rid of the "and/or" here?


¿Vas a mostrar null en una aplicación gráfica? Seguro que hay algún mensaje mucho más friendly que null (no todos somos developers ;) )


try to leave a space concatenations of values as example "...Name: " + results... name + "..."


Makes sense. By the way, this is throwaway code.


Yes.. 401 will be a valid response for every api when no token is passed.
403 is to be used when you have a token but you are denied access to this resource.


try to avoid using ids, we usually use classes only to avoid specificity issues


No necesitabas crear un servidor nuevo, si corres el servidor original con `node server.js` el _broker_ con mosca sobre http ya estaría creado, no deberías necesitar más líneas que las siguientes:

```javascript

var mqtt = require('mqtt')

 var cliente = mqtt.connect("ws://localhost:8000")

 cliente.subscribe("mqtt/demo")
 cliente.on("message", function(topic, message) {
 	   console.log(message.toString());
   })

 setInterval(function () {
	    cliente.publish("mqtt/demo", "notificacion periodica mandada!")
     }, 1000)
```


Thanks! I totally forgot to add that. I appreciate your work a lot but it is not entirely working i'm going to edit all the files, make a commit and publish it to npm. :+1:


Add to the test string `before_compact` some string value with `\n` char in it. And double check that it was not removed.


Ok, I'll have a look. But mind you that there currently is no such these for video either (it is commented out).


Not sure how I can ask the question differently.


На это кто-то завязан?


二哥的寫法會跑不動耶


please apply this consistantly across all the apis.


А чего оно в libs/ тогда делает?


Maybe replace `.\{2\}` with `..` ?


Este tipo de literales te los da Java
```
Boolean.TRUE.toString().toUpperCase()
```


I thought about doing that, but it seemed too much, since in my imagination there would be no problem to send an empty `sub`. That's probably why testing is necessary.


Isn't it possible to manage that in another way ?
My log file is full of "Event send to Loggly"...


I don't think you want to allow people to configure something out of their control?


Este método es ultra poderoso! Divide y vencerás ;)


@tazmanska Perfect ! If it seems too long/hard and don't want to loose time into. Just tell me ! I'll do the work !

i will release the version 3.0.0 these days. Current Segment Time Comparison have huge performance impact on activity processing page. So added a ribbon message to explain that Segment Time Comparison features have been disabled by default (only options unticked) due to performance issues. You can still re-enable them in options. Segment Time Comparison features will be automatically re-enabled when fixed.

Still thanks !!

Tom


Cheers for the heads up. I'll create an issue now and take a look through it.


I don't understand why 'me' is difficult. it's just a lookup on your end? please explain why this would be complicated.


fixed


Accept


Так оно его целиком что ли засосет? Мож чего-то более конкрентного хватить?
например: com.google.android.gms:play-services-identity или что там ты юзаешь?


¿Qué es Funciones? Puede ser absolutamente cualquier cosa! Si es algún tipo de utilidad relacionada con algo haz su propia clase de Utilidades (ej: DateUtils, ContractsUtils...)

Tiene toda la pinta que además deberían ser métodos estáticos.


Is OK but try to use 1 media block for all the stuff that should be inside it.


Hello: I want a video tutorial Thanks


In the meantime, here's an example of a response for a audio file upload:
```
{
  "id": 4,
  "name": "bleh",
  "description": "blah",
  "created_at": "2016-01-18T13:56:42.442366Z",
  "creator": {
    "id": 2,
    "display_name": "Matthias"
  },
  "isowner": true,
  "url": "/assets/user-uploads/audio/beatles_lvbYFof.mp3",
  "thumbnail_url": null,
  "file_type": "AudioFile"
}
```


True; my logic was that since it's a fairly new service (ca. 3 months), the limit may change.


Raw type!


I think upstream Git is sensitized enough by now to care even about pure Windows improvements ;-)


The webURL will be eliminated from this api as soon as we start building the new UI! this is only temporary and is to be removed.


Obviously there's no thumbnail, as you rightly suspected.


I think things might end up a bit simpler if gputop_i915_per_oa_overview_open() were to take a guid or maybe a query pointer to avoid having to walk through the array within the implementation of `gputop_i915_perf_oa_overview_open()` to match the corresponding query. If we add a `priv` or `query` member to struct tab, then where the tabs are allocated we can save a pointer to the query for these callbacks.


Такое неплохо бы вычищать перед коммитом


I think it was difficult to test videos as they're being uploaded straight to YouTube.

That's fine, if it does not throw an error, empty "thumbnail_url" should be OK.


Sé homogéneo, en el método anterior inicilizas en la declaración (que por cierto es más limpio). ¿Por qué aquí no haces lo mismo?


Jeje, no es tan tan sencillo, si clicas en los enlaces del [reto](https://docs.google.com/document/d/198ERYC6jnouGFp_a0xasA2T-RJqfVKM6gSrJhQ27coA/edit) verás que apuntan al uso de la API de notificationes nativa, que no son alerts.

Los alerts no generan notificaciones en móviles modernos y no son recomendables en general.

Aquí hay un codepen de qué es lo que se buscaba en su lugar:
http://codepen.io/yuujii/pen/pvjroY

Y una imagen del resultado previo:
<img width="1279" alt="screen shot 2016-01-18 at 15 01 28" src="https://cloud.githubusercontent.com/assets/5166506/12393185/6d367976-bdf4-11e5-8632-82e235764057.png">


Doesn't seems right for me to use a ul to render a form. Try to do it without the ul :)


hmmm, contrato es un contratoDAO... llamarle contrato a la variable puede inducir a error (los DAO y los BussnesObjects son cosas de naturaleza completamente distinta, escapa de la ambigüedad)


Thanks @Ealdwulf - It's not finished yet, I'm actually working on the steps you point out :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11845](https://teamcity.mesosphere.io/viewLog.html?buildId=107099&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:08:51


This collection should have a consistent ordering. If mutability is not going to be a requirement in the future, `ImmutableSet` should be a great solution.


IIRC, It's because it augments the base widget, thus the passive tense.


How about the following, which also relaxes it to a SHOULD which I think is more in line with what we should ask from UAs. (It could be very difficult to keep track of where a SCT was received...)

If the client chooses to do so, it will create or use an existing TLS channel to the server; the client may have recieved SCTs over this channel -- it SHOULD NOT send any SCTs (or constructed certificate chains) from that connection.


True, the client could send and the server could have the smarter logic to ignore.


А SEND_MULTIPLE почему не хотим?


c比較需要給初始值

java 還好


👍 thanks!


Pinterest ???


Looks like you miss spell "canola"?!


the client MAY require additional validity requirements on the connection, such as an unrevoked certificate or accompaniment by a valid SCT.


Al evaluar expresiones condicionales tan grandes deberías guardarlos en booleanos:

```
boolean esContratoFormacion = miContrato.getTipoctto().contains("Formación");
boolean esJornadaParcial = miContrato.getJor_tipo().contains("Parcial");
...
if( esContratoFormacon || esJornadaParcial...
```
¿No te resulta mucho más legible?

Con los números igual:
```
List<Integer> tiposValoracion = new ArrayList<Integer>();
tiposValoracion.add(300);
tiposValoracion.add(600);
tiposValoracion.add(700);
...
if (tiposValoracion.contains(miContrato.getTipoValoracion()) == false...
```
Y los magic numbers también fuera del cuerpo del método.


I think we can aim to change the remote protocol so that it works in terms of guid instead of ID. We will need some new protocol for advertising a list of supported guids which the remote ui can use to look up meta data in the XML files and it is then able to use those guids for requests back to the server. For now this makes sense though - maybe just a comment mentioning that this is only needed as a stop gap.


Лишний begin/end.


После прогарда тут будет гавно


@bbx10 have you considered sending a PR to the official repo: https://github.com/felis/USB_Host_Shield_2.0? I guess this driver could be implemented without much trouble. I will then work on adding support for the Arduino Due to all drivers.


It is not intended to be an operator, nor does implementations of ManagedState. However all these components need to be notified about beginning and end of window. This is why they implement Operator interface. In usage they are not operators, but they need to follow the window paradigm as the operator.

I can see one problem with these classes implementing Operator interface is that with tools that find out all the operators in the library they will show up as well.

If that is the concern, I can change it to not extend an Operator


El problema de no encapsular los literales numéricos es que no sabes qué signifca ese 300 y si mañana cambia, tienes que cambiarlo en todos los sitios donde haya otro 300 (y puede ser que no sea el mismo 300 en todas partes por lo tanto te volverás loco) jeje


Sem problema mano.. o que vale é a intensão :)


Я бы тут ещё добавил проверку, что в тексте ошибки есть слово title (т.е. `raise_error(ArgumentError, /title/)` примерно)


Recuerdo que habíamos hablado de encapsular todos esos parámetros. Recibir un Date y un Empleado quería mucho más limpio


mUrl если это не константа


Nice code cleaning, danx :)


Bien, este debe ser el código más nuevo. Esa es la idea! :+1:


here it is
```
it.only('should parse xmlns', function() {
    var obj = {
        test: {
            '$xmlns': 'foo',
            '$xmlns:y': 'moo'
        }
    };
    var str = jxon.jsToString(obj);
    assert.equal(str, '<test xmlns="foo" xmlns:y="moo"/>');
});
```


Ganesh,

What is the purpose of get_cert_subject_unknown_subfield?  Why would the user call this?

If it is for testing only rename it by prefixing it with an underscore - that is the common practice for indicating that a method/object is for internal use only.   Otherwise it's part of the API.


lanzar una UnsuportedOperationException como haces en otros sitios está realmente bien :)


Эти обычно удобно до вызова super логать. Но тут дело вкуса больше


Inetnta separar tu código del código generado


ReferenceError: markers_ is not defined


Going home


Se pueden capturar varias excepciones en un mismo try


Thanks :)


I was going to do this too, :+1:


You could use [$timeout](https://docs.angularjs.org/api/ng/service/$timeout).


"projects", "access_key" - в константы. нехорошо хардкодить. Особенно в нескольких местах


No me digas! jamás lo hubiese advinado! jejeje ;)

Los comentarios son el mal! En caso de utilizarlos no deben decir qué haces ni cómo lo haces, en todo caso `por qué lo haces


Fixed a few bugs.  (char *) to (const char *).  Also, fixed "config" command and removed the bit masks for the config function.  Now it just shows all the configurable variables automatically when "config" is typed.


break the chain


Let's keep it consistent, so a variable and a comment with source file path (don't delete it, we might need it for modifications). This should be done by a tool one day, not manually.


Este es el único contexto donde no te doy caña si no quieres encapsular los literales... pero de todas formas yo lo haría uglamente


See inline.


I did not see a question


Неосилил по какому принципу final расставлены. Выглядит как просто разножопица


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25690/testReport/) for this commit**


Bien por usar un logger! pero no haces nada más que mostrar un error. Que falla el guardar el documento parece algo bastante crítico que merece la pena tratar/informar ¿no crees?


Estos literales los quiero bien encapsuladitos jejeje


new ArrayList<>()
Кому эти буквы лишние надо?


@beanstown106 you can't make hardware modifications?!?!!? //sarcasm


for (final currProject : projects)
?


good job. @CzhCN1


Removed .DS_STORE


`xlsx` не нужен


If it would take longer than 1s to load the page, the login to Discourse would fail?


тут вообще не используется


mmm si possono fare le discussioni sulle righe di codice


Ah. Excellent.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#GPN2pDPrQzu0p_9QPFqIng


@vyncem That looks good. It's been a while since we redeployed the website so it is not surprising that it needed these changes. Great job!


这个 长满杂草的地方


а, вижу, не прав


This is not use right now, but I have been the whole morning trying to get the logs printed to the standard output with eliot. I have tried with write instead of log, creating that custom message, and it didn't seem to make any difference. I went back to message.log


Yo hasta preferiría no utilizar stringBuilder aquí.

```
		String query = 	"SELECT " +
						"	FIELD1, " +
						"	FIELD2, " +
						"	FIELD3 " +
						"FROM " +
						"	FOOTABLE " +
						"WHERE " +
						"	FIELD1 = 1";
```


I think we can call it just "Response", as it already implies generality. What do you think?


I did not intend to change `format_currency`, but it might be a good idea to make the numbers of digits configurable (now that we have a settings-file) - what do you think?

Changing the API so it spits out Decimals would be the best idea here. I changed it/implemented this because the API might change to do just that in the future.


求助：
下载您提供的android_bootable_recovery  android_device_wingtech_wt88047 android_device_cyanogen_msm8916-common proprietary_vendor_wingtech android_kernel_cyanogen_msm8916 这些代码去编译wt88047的TWRP，编译出来一直亮灭屏，没法启动起来recovery，一直重启recovery这个进程，但是我adb进入rec模式，手动命令运行recovery却是可以使用的，能否问下这个是什么原因引起的吗？谢谢了


NUNCA utilices propiedades de orden en una lista para obtener cosas. No aprotan ninguna semántica y son un caos para mantener. Encapsula y utiliza getters


`subject`


@justinedelson Thanks for the explanation, but when I tried it out in the context of https://issues.apache.org/jira/browse/SLING-4161 a mandatory reference in a DS component with a class which could not be resolved at run time did not emit any warnings. Instead the according service was just not starting. With which version of AEM/SCR did you try that out?

Having two different implementations for the two approaches to deactivate components seems totally fine to me (and rather straightforward). To me it looks cleaner than invoking an ECMA script from only one service and within that script just try out two approaches (kind of hard to debug as well)


gosta! :+1:


I can't find where the logs are going. I am missing some eliot initialisation I think, but I don't know what


тут видимо тоже `subject`, только параметры не те


good job


и здесь


Называть полезно более информаивно чем "c". Читабельность говорят повышает...


Trabajar con fechas en String en lugar de con Dates es currar el doble sin tener ninguna ventaja


I don't see an answer :-) We are stalled I think :-)


Estos mñetodos deberían estar en su propia clase de Utils como DNIUtils o algo así, y deberían ser estáticos


@malirod See 525bc5a


@pombredanne yes, it raised an error in python 3 (`range(1000)` does not create `list` in python 3), so i added a casting line to a `list` for a quick fix :)


Смысл этой строки? На одну ниже нельзя объявить?


what happens if you move localized here?


Apparently the right way to include multiple licenses is this:

    { "license" : "(ISC OR GPL-3.0)" }

source: <https://docs.npmjs.com/files/package.json>


this throws `ReflectionException in Container.php line 741: Class blade.compiler does not exist`

it should be
`$app->make('view')->getEngineResolver()->resolve('blade')->getCompiler();`


things like `drawLine` are called very often so I'd prefer simple loops instead of closures.


Can you please adapt the syntax of including multiple licenses according to the npmjs docs? Thanks!


@pombredanne typo... thanks! i will fix this later!


Esto lo habíamos hecho de manera didáctica. Los meses del año ya te los da Java. Si no te los da en el idioma que quieres puedes hacer una clase de utils que los traduzca pero no trabajes con literales, te obliga a comparar contra ellos y no sabes en tiempo de compilación contra qué estás currando
En todo caso (y tampoco porque los meses son algo demasiado estándar como para que los tengas que picar tú)
```
public enum Month {
  JANUARY ("enero"),
   ...
}
```


Here you can just do self.read_from_server ;)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11846](https://teamcity.mesosphere.io/viewLog.html?buildId=107103&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:47


no mezcles naming camelCase con underscored. La convención en java es siempre utilizar camelCase


Yes this is a problem and I already solved it by this commit https://github.com/jkonecny12/anaconda/commit/7968a1bff739e3d32d443706a3b4e451705efa24


best to be more specific about these if you can, a few people are still running with prereleases


It's the correct branch name for libseccomp, and the correct package name for the audit-libs development files for Fedora.  This is a generated file, though.


Thanks for fixing those. :+1:


Absolutamente excesiva responsabilidad... esta tabla es absurdamente grande


626 líneas para describir un mapeado de una fila de una base de datos es excesivo


en cambio este es super friendly! de un vistazo sé qué guarda :)


Nice one!


Well, there also is a question about what this package should do, be the same as the base parser in all julia versions, or be the latest julia parser but run on all julia versions. I think we probably want the latter, so I should probably remove this conditional.


If the image somehow has a different go compiler in /usr/bin, we need to prepend the location where we extracted the version we intend to use.  Otherwise, we might use a go compiler from /usr/bin.  I don't know if I'd call it a configuration issue.


Runnin --> Running
acually --> actually
swtich  --> switch


Este también está guay salvo por el nombre HorarioContrlab?? HorarioContraLaboratorio??? jejeje HorarioContratoLabolar no tiene nada de malo ¿no?


What kind of check?


BIEN! :+1:


The substitution should be transparent to consumers, i.e. `getClasspath` and `getProjectDependencies` should return the substituted dependencies, not the original ones.

Having a special subtype like `SubstitutedProjectDependency` could help display substitution information to the user if needed.


I don't see the latter as all that useful - why would I want to parse code for a version of Julia that's newer than what is running?


@malirod, I did it.


Debes mapear los tipos enumerados en BBDD a enums


If you don't then why not just use the old version of this package?


This seems to imply that if the client for whatever reason chooses to create a new TLS channel it should send the data regardless of age?

Also, is "TLS channel" well defined?


Good!


You can include a ps file from different places and it acts as "include_once"


Can we get this pushed to cocoapods?


@malirod, Ok, I did it.


Updates are automatic. If we stop maintaining the fidelity of this parser producing the same results as the scheme parser of the version of Julia that's running, then we should explicitly not support versions of Julia where that no longer holds.


Sieht gut aus :+1:


The actual user configuration happens in the Begin task


This is closer to the default upstream Dockerfile, which sets GOPATH to ``/go:/go/src/github.com/docker/docker/vendor``.  This should probably move ``/go`` to the front of the list, and if the compiler always knows where the standard library is, then we can just use the value from the default Dockerfile.


@malirod, Ok, I did it


What do you mean? This attack is about servers filling up client storage, is it not?


A lot better than spaghetti coding :+1: :smile:


Easiest is to insert a large string in the normalize method. Like: https://github.com/emberjs/ember.js/pull/12652


It'd be slightly more efficient to just do ```new List<TElement>();```, i.e. no capacity arg.


Ah, I didn't realize these had gone upstream.  Yes, I think so, if the current version doesn't work for other people, either.


closes #21


jo to by mohlo fungovat :)


@AGritsevich, where?


Amerian -> American


I agree. However, the existing model `EclipseProject` didn't allow for adding new entries through the method `getProjectDependencies()`.


Looks good.


Of course there is no problem with having methods like beginWindow, endWindow etc. and calling them when needed but explicitly declaring that it implements Operator is, in my opinion, more confusing than helpful in this case.

Generally speaking, such a declaration is made only when there are contexts where the object is used _only_ through that interface. For example, in a GUI context, a class may declare that it implements the Scrollable interface if there are contexts where the object is passed to a function taking a Scrollable parameter; that function then can only invoke methods of the Scrollable interface on the parameter. Another use case is if the object is an element of container of Scrollabe objects, e.g. List<Scrollable>. In the present instance it seems we have no such uses.


Thank you I'll change it.


This line should be guarded by is_not_dev_version


@malirod, Ok, I did it. About 3-rd paragrahp, for your solution need much more time. This I can't do. I think, my solution will work too.


可以使用更好用, 使参数更清晰的注解模板, 扩展性更强, 将参数变为JavaBean的形式, 更已读:
https://github.com/wyouflf/xUtils3/blob/master/sample/src/main/java/org/xutils/sample/http/BaiduParams.java


Audio upload test added: https://github.com/ExCiteS/geokey/commit/2017c0c663a104872d18fa9e176a4028dd0d10ad


Although you have specified the width at 30%, the padding and margins break the flow at lower resolutions. My suggestion would be to use styling sparingly to achieve the results you want. In other words, sometimes less is more.


Fair enough, I'm in favor of that then.


done


I think this line might need an -f flag, because it is trying to replace existing tags with new values for those tags.  WE got the message, for example "Conflict: Tag 1.14 is already set to image 61ace6f589b4e12a628b3d13f85810ccf41f39ce3c03f79eb73cc0759c7bff90, if you want to replace it, please use -f option"


But but... the for begins a block too! Oh well. Gotta remember that because it's standard procedure for me to put the declare into the for.


I'm confused about this fix, since I don't see a `profile` field in the `argo` dataset, nor in the object created from reading that file directly. Am I missing something here?
```r
> data(argo)
> str(argo)
Formal class 'argo' [package "oce"] with 3 slots
  ..@ metadata     :List of 5
  .. ..$ filename: chr "/Users/kelley/src/oce/create_data/argo/6900388_prof.nc"
  .. ..$ units   :List of 5
  .. .. ..$ temperature:List of 2
  .. .. .. ..$ unit :  expression(degree * C)
  .. .. .. ..$ scale: chr "ITS-90"
  .. .. ..$ salinity   :List of 2
  .. .. .. ..$ unit :  expression()
  .. .. .. ..$ scale: chr "PSS-78"
  .. .. ..$ longitude  :List of 2
  .. .. .. ..$ unit :  expression(degree * E)
  .. .. .. ..$ scale: chr ""
  .. .. ..$ latitude   :List of 2
  .. .. .. ..$ unit :  expression(degree * N)
  .. .. .. ..$ scale: chr ""
  .. .. ..$ pressure   :List of 2
  .. .. .. ..$ unit :  expression(dbar)
  .. .. .. ..$ scale: chr ""
  .. ..$ flags   :List of 3
  .. .. ..$ salinity   : chr [1:56, 1:207] "1" "1" "1" "1" ...
  .. .. ..$ temperature: chr [1:56, 1:207] "1" "1" "1" "1" ...
  .. .. ..$ pressure   : chr [1:56, 1:207] "1" "1" "1" "1" ...
  .. ..$ dataMode: chr [1:207] "D" "D" "D" "D" ...
  .. ..$ id      : chr "6900388"
  ..@ data         :List of 6
  .. ..$ time       : POSIXct[1:207], format: "2005-10-29 13:52:11" ...
  .. ..$ longitude  : num [1:207] -21.4 -21.9 -22.3 -22.4 -23.1 ...
  .. ..$ latitude   : num [1:207] 61 60.8 61.1 61.3 61.5 ...
  .. ..$ salinity   : num [1:56, 1:207] 35.2 35.2 35.2 35.2 35.2 ...
  .. ..$ temperature: num [1:56, 1:207] 9.71 9.71 9.71 9.71 9.71 ...
  .. ..$ pressure   : num [1:56, 1:207] 4.8 9.2 14.6 19.3 24 ...
  ..@ processingLog:List of 2
  .. ..$ time : POSIXct[1:2], format: "2016-01-17 06:33:00" ...
  .. ..$ value: chr [1:2] "create 'argo' object" "read.argo(file = file)"
```


Or better, we could try making it a user choice in the API which behavior they want - "parse as this version." Then it's a matter of picking a default, which I think the running version of julia would be a better-defined target for "fully compatible with Julia's built in parser" than whatever is the latest that has been implemented here.


Now I'm confused. First paragraph in this section already implies that there is a valid SCT present:

"The client MUST discard SCTs that are not signed by a log known to the client"

Can you explain more?


Will you also provide a patch to the openembedded-core mailing list?


Just wanted to make sure we have the same design in mind :)


Thanks for the tip @rk-yen , I've tried that but I'm still facing the same import error (#import "../AmazonServiceExceptionUnmarshaller.h" file not found).
Would you share what version of cocoapods you're using, or some other details? I've tried with several versions, none worked.


This is only to hide the Add part if you already have a download on demand element for every architecture in the repo (each DoD have to have a differente architecture)


We still need PHPUnit 4.8 for PHP 5.3.3 support.


Hi,you can try this method: https://github.com/DevriesL/SM-G9208_ImageBreaker/commit/87e72674ffe5f60c89f052dcd3f656b12f2a2c96


Will be fixed.


:ship: it


Should be 6.0?


Ah okay, I will add it back.


Cool!!


Is this escaping really necessary? It seems to render [correctly](https://nodejs.org/dist/latest-v5.x/docs/api/globals.html#globals_dirname).


Add `4.8` instead of `4.0` IMO.


Y'a le menu aussi... :)


What do you mean "automatically populated"? I'm also on OS X and don't have `SQLITE3_LIB_DIR` set in my environment.


Where?
I used dod in ids and not visible stuff...
dod == Download on Demand


Why is this FindBoost.cmake required?


Same as above


6.1 once we release 6.0 actually, will change


Half suspected you chucked these, I'm very upset


See inline comments.


Ok, I will


space after `:`


missing space after `img`


Yeah I didn't know about it either -- saw it on the Bootstrap site when copying their example code. 😀


Block is started after for and not in the initialization of for


turn back abort. Just call `exit(EXIT_FAILURE);` in the default branch instead of break


They were shit and failed pat


too big indentation, it was ok before


I see you're using ```` ```text ```` in some places, but leave it out on others. If there are no adverse effects of not specifying a type, maybe leave it out everywhere? (I think `text` is assumed to be default?)


They're awesome and just needed love


Yes. We need to decide on the timeout value and eventually think of some messages / spinner for the user.


Ok, I will move those to a fixture.


I propose to make small refactoring here. "struct" -> "class". Add public section.
The "_" suffix is missing in the `empty_log_queue_notifier` name.


Why Aravind Why (TдT)


I think the correct type for shell output is actually ```` ```console ````. It's not perfect, but better than ```` ```bash ````, I think. Compare:

#### bash
```bash
$ node
> require('url').parse('/status?name=ryan')
{
  href: '/status?name=ryan',
  search: '?name=ryan',
  query: 'name=ryan',
  pathname: '/status'
}
```
#### console
```console
$ node
> require('url').parse('/status?name=ryan')
{
  href: '/status?name=ryan',
  search: '?name=ryan',
  query: 'name=ryan',
  pathname: '/status'
}
```


보빈이 안드로이드 서버 둘다 하는거야? 덜덜


Missing "_"


Это очень странный if. Я не понял что он делает (


Could include an error message to show that there was an error getting the file.


:+1: Please merge this!


Ну вот тебе не лень было эту строку писать? Почему не внутри цикла?


oups , un peu trop minimaliste..


pak se teda jenom upravi at to bere to nastaveni backoff z env promenne


Should have been FM-3886


closes #18


this is now configurable


unnecessary spaces at the end of the line


I see some inconsistency regarding the level of indentation of comments like these ones. I think the extra level of indentation might throw off a linter (which I plan to integrate after this is landed). Should we remove the extra indentation or is this a common practice? @nodejs/docs


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11847](https://teamcity.mesosphere.io/viewLog.html?buildId=107112&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:07:42


I'm guessing this should be `case 'UNHANDLED':`?


@malirod , ok see 40be2e3ce80801963f84841c5df04d113d826c39


You shouldn't touch the template. Or add the .mo file. These shouldn't be here.


Add / to info > info/


Are we absolutely sure that we will always have a 0-indexed array here?


How many records does this insert into the privilege table?


If this has removed the need for the method, we should also delete it


not needed, just set the state to root like how login does in vidi


There's a serious regression presented in this commit.
Namely the `echo $var´ syntax had been used to trim any leading or preceding whitespaces, but with the new `printf "%s" "$var"´syntax those whitespaces are not cleaned out, which further causes problems in other parts of dracut functionalities. For example consider following debug output:

```
//usr/lib/dracut/dracut-functions.sh@688(check_vol_slaves): lvm lvs --noheadings -o vg_name /dev/mapper/os-root
/usr/lib/dracut/dracut-functions.sh@688(check_vol_slaves): _vg='  os  '
//usr/lib/dracut/dracut-functions.sh@690(check_vol_slaves): printf '%s\n' '  os  '
/usr/lib/dracut/dracut-functions.sh@690(check_vol_slaves): _vg='  os  '
/usr/lib/dracut/dracut-functions.sh@691(check_vol_slaves): [[ -n   os   ]]
//usr/lib/dracut/dracut-functions.sh@684(check_vol_slaves): lvm vgs --noheadings -o pv_name '  os  '
/usr/lib/dracut/dracut-functions.sh@699(check_vol_slaves): return 1
```

Where the problem is that that the volume group name "os" here contains whitespaces, from command line using command `lvm vgs --noheadings -o pv_name 'os'´ returns the correct physical volume.

By looking this commit it seems that there are plenty of other functions where similar problems may occur.


Obvious typo, but as others have said, I think we should skip this commit as renaming would destroy history.


Although images sometimes behave like block elements they are not. It is better practice to wrap these tags in a `<div>` or `<p>` to achieve the desired result.
https://developer.mozilla.org/en-US/docs/Web/HTML/Block-level_elements


const


k?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11848](https://teamcity.mesosphere.io/viewLog.html?buildId=107115&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:10:23


Ha! great oldschool job on this one :+1:


I wasn't sure why I wrote it like that. Sorry, I had meant that setting and passing `PKG_CONFIG_PATH` onto `cargo build` was no longer necessary; and I thought this might be important to note on `README.md` (or somewhere) for users who have been using an older version of rusqlite.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11849](https://teamcity.mesosphere.io/viewLog.html?buildId=107116&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:47


@avilaton movi lo que estaba en base.scss y sus includes a reset.scss y a forms.scss, ya que teniamos esos para ello.


Will render badly in text editors as it assumes you are starting bold texts. Leaves all the rest of the markdown, say, orange.


> Will render badly in text editors as it assumes you are starting bold texts. Leaves all the rest of the markdown, say, orange.


This is actually something that can be handled inside the model. Check out [rails validations](http://guides.rubyonrails.org/active_record_validations.html#why-use-validations-questionmark).


@avilaton esto lo quite y lo reemplace en el codigo por clases .FR y .FL que ya teniamos en el reset.scss.


It works like this but I think that we could have also track change of route in navbar component and reset the select mode if the route is not part of the navbar component

Then everything was tied to navbar


Shall the policy be to do it everywhere? Don't know much of the difference to bash, but at least bash rendered pipes correctly, see comment above in the PR. I have no strong opinion on this....


We are having trouble with some passwords containing the chars `+` or `\`, which are the only non alphanumeric in base64 (see https://en.wikipedia.org/wiki/Base64).

Shall we change this script with: `openssl rand -base64 12 | sed 's/+/0/;s/\//0/'` for instance?


I know it's not part of the immediate problem described by this JIRA, but I would suggest a general code cleanup to get rid of _while(true)_ in favor of  clear condition describing how this loop ends. The main issue I see is that its very hard to track all possible conditions with the amount of 'break' and 'continue' it has. On top of that _while(true)_ is not interruptible which means that there is a chance it will never end, so at the very least I would add _while(true && !Thread.currentThread().isInterrupted())


I want to do it more general but the option_active is not good so or so. I propose ``is_option_checked`` or ``option_checked``. What do you think?


I don't think passing `PKG_CONFIG_PATH` has ever been necessary, but if you leave it out, you'll get the version of SQLite that shipped with whatever version of OS X you're running on. I'd like to put together more comprehensive "how to build" documentation in general, particularly for Windows, but could include notes about this too.


[nits] 次回PR時にHelloStomp.prototypeのお名前も変更しておくと良いかもです〜。今回確かにHello Stomp Worldですがちょっとハンズオン意図と異なるのでw


This can actually be simplified with a little [rails help](http://guides.rubyonrails.org/active_record_validations.html#displaying-validation-errors-in-views).


Thanks. Can you provide a unit test for this as well?


According to the Doctrine PHPDoc and classes yes but I'll revert this just in case.


As long as it's valid markdown, I'd say that's a bug in your text editor.


Yes it's default. There are weird cases in `linguist` (supposedly) though, where all the rest of the markdown renders, say, green, because it does't logically close the code block (experienced in Atom). In order to mitigate this having it explicitly text worked.


@malirod see 2cded3e6e9be3770c0e60f0fcbb0158fcd1092ac


As per the guidelines, the title should have a full dotted border around it.


`if not file_path.startswith('#')`


Do we need to flip the horizontal margins? On my tests the results seemed horizontally mirrored.


@matt-riemer, @jfeaver, fixed failing tests in this commit. Not sure if this stuff was working actually, as cause of the failing tests were not incorrect tests themselves, but the code.


This comes from the original. There is inconsistency all over docs. When obvious I did them already, and did one in a later commit.

Shall I have a run through the whole thing? I if so, with which policy?


Well spotted


Is it possible that 'acceptGzip' is not even set on the request, at which case this would be NPE.


I don't think so -- we are fine with any of the 2xx responses and shouldn't care about 3xx either


Yes, it is only for testing. I will submit another pull request. Thanks.


Nice catch, thanks!


On second thought, I'd say we handle this in another PR, as to not stuff more things in an already hard to review PR. Leave it as is for now.


It results in unresolved external reference during linkage.
Inlined functions should be defined in the same module they are used (called) from according to C++ standard. MSVC and LLVM on VS (clang frontend coupled with MSVC backend) handles violation of this rule differently:
* *inline* functions referenced from external modules only leads to unresolved external for both compilers.
* *inline* functions referenced from both definition and external modules works with MSVC in all situations and with LLVM provided reference from definition module is not optimized out due to dead code removal.
* MSVC handles references to *__forceinline* functions similar to *inline* ones.
* references to *__forceinline* functions from external modules always remains unresolved with LLVM.


Thanks.

Possible improvement: To allow installing with `raco pkg install https://github.com/ps-tuebingen/grading-statistics` and then running with `racket -l grading-statistics`, it'd be better to move files into `grading-statistics`, while `info.rkt` stays in place. Not 100% sure.

We'll also need to avoid absolute paths for that, but one thing at a time.


These change is added to avoid the changes on Booking Reports


The availability section should extend from border to border. It looks like the padding and margins in the `.col` class may be the culprit here.


Styling changes need to be made in the .scss files and then the .css re-generated


I reviewed & good with UddiConnectionInfo_g0.xml changes.


Nit: Spaces around `=` and `-`, also on the line below.


Yes! Great catch! You can fix it and push. Thanks.


your spacing is still off on these


At this time, I just need to info to upgrade from an old Hibernate 3 setup (Jboss 5.1) to a new Hibernate 5 setup (Wildfly 10); I might be able to add a unit test in a few weeks, maybe months, as my priority is on fixing issues with the migration while contributing to the project any piece of code I need to ease that process.


Review : add test with Jasmin


Copyright should be Google then with authors below


no need for 'return'


Okay. Agree.


It's important to keep in mind that on the long run `CBA_missionTime` may diverge from `diag_tickTime` even if `accTime` is always 1 due to rounding.

This may not be important for most uses. I don't know how much impact it may have things that have a long delay like a explosive set to detonate after 30min.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1189](http://teamcity.monogame.net/viewLog.html?buildId=13731&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Grab the latest release from [here](http://staffwww.dcs.shef.ac.uk/people/R.Chisholm/lego/) I'll create a proper release ASAP (won't be able to accept your Pull request to do so though :( ).


@azachar thanks for adding my PR it's my first ever PR :dancer: glad I could help !


I'll change it to `checkbox_text` from what I can tell there is no `:param:` or similar in  description. Correct me if I'm wrong.


Should have been FM-3886. Doh :disappointed:


@andschwa Unfortunately, trying your method produces a less than ideal path.
```
PS C:\Users\Jeremy> [IO.Path]::GetFullPath("~\vimfiles\autoload\plug.vim")
C:\Users\Jeremy\~\vimfiles\autoload\plug.vim
```

@timjk Your solution seems fine to me, I'll fix the command in a moment. Amazing how much typing MS wants you to do to get a path...

```
PS C:\Users\Jeremy> $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath("~\vimfiles\autoload\plug.vim")
C:\Users\Jeremy\vimfiles\autoload\plug.vim
```


The thing about this is that future additions might not include the `text` type because few people will know about this issue. If this color bug doesn't affect our HTML output, I'd say leave the blocks without a type.


indentation is wrong


Please Rewrite with Sass


I ran into issues when compiling some ROS packages which require Boost's threading library on the target system, because the pthread library was not linked in the CMakeLists.txt.


I don't see where the Akka Streams actor-based implementation is being deprecated.  Is there something happening that I don't know about?  Here's a quote from the Akka Streams 2.0.2 documentation:

"There is simplified API you can use to combine sources and sinks with junctions like: Broadcast<T>,
Balance<T>, Merge<In> and Concat<A> without the need for using the Graph DSL. The combine method
takes care of constructing the necessary graph underneath."


For a bikeshed -- should this be plugin be called throttle_metadata_keys ?


`console` will have its own highlighting quirks one it encounters JS. I think not specifying a type here would be the best, until we find a better solution (which might require writing our own highlighting).


I think we should stick with either shows or series and re-name everything according to it.


Overall, nice job on your quiz, Paloma. There are just some fine tuning points on your styling you should address. Also, I saw some minor indentation issues. Strive to be perfect with your indentation. It will save you later when you have umpteen lines of code that you are trying to debug and need to crawl through. :)


Should these options be configurable?


In PR builds the SQ analysis runs in incremental mode i.e. the results are not persisted in the SQ database and - more importantly - the quality gate is not computed. So during PR builds we ignore this flag and give a warning (although a warning may be a bit too much ... please advise).


It is possible and in fact happens in the majority of test cases for the associated TestPostHTTP class.  In those cases equalsIgnoreCase will always be false and there should be no chance of an NPE in the current syntax unless I am overlooking something.


It also returns a list of key states.


Mmmm. There are like 10 places more in this controller where we don't catch any exceptions of the store method... Should we change all of them also?


Damn, good catch.  I'll update it.


I have made several commits since this one - you might want to take a look at them.  Whatever changes I make I want the API to be friendly to Java as well as Scala, especially since my "day job" is working in Java - haven't convinced my team to adopt Scala yet :(


LOL! Thanks man! I was actually going to clean it up a bit further, then submit a pull request if you didn't mind. This is the first time I've played with such an architecture (Angular + Material) and I love it! You did awesome with building this.

Give me a bit more to put some polish on it, and I'll submit a pull request if you're cool with that! Cheers, and thanks for building a (much needed) awesome launcher, that works great for me on Linux.


Most of the helpers here were copied from other locations.


"Retry" and the helpers below are new.


I believe line 1155 should not be 'p.gui' and instead be "p.linked_clone"


The topic+partition+offset case class is a good idea. Working on that.


although this fixes the problem I think we should also validate the annotJSON file before permitting the download. Should I open an extra issue for this?


Do you plan to upgrade this to GCC 5 when everything is working?


Зачем все эти catch - если у аккаунта не будет заполнено ни одного поля - он действительно кому-то нужен такой?


Integrated changes, updated version incoming


hehe ;)


Please fix typo.


`npm install gulp-filelog`


I see. Adding a test is very simple. You can download [our templates](http://in.relation.to/2016/01/14/hibernate-jpa-test-case-template/) and just add an entity that replicates this issue. It takes at most 10 minutes to do it.


Some of the helpers are common but there is no way to share code between tasks (unless I rely on the fact that tasks are dropped in the same location etc.) So this file should be a copy of the similarly named one from the PostTest task (i.e. nothing to see here, move along..)


Okay. Best then would be to anticipate the parser variable name and add it now. I think it will default to text then anyhow.

Pro:
* we don't need another PR for changing the blocks
* we can later distinguish it from text code blocks by just regexing ```` ```noderepl ````


That was badly used syntax it wasn't parameter description but the description of the whole method.
Parameters description is below.
There is one similar mistake too on "not display the checkbox". I'll fix that.


`npm install gulp-debug gulp-cond`


Зачем тут таймаут? Они же готовые из invokeAll приходят. Нет?


`npm install -g gulp-debug gulp-cond`


Good catch! We should change `Val` to `Value` in the preparagraph too, care to make the change? Thanks a lot.


Only the "breakBuild" param was added here, the rest are white space changes that GitHub isn't handling well.


`npm install -g gulp-if`


may as well remove this whole if block, it will never happen


Updated accordingly


True, and there are a few other places that do the same thing. (It had seemed a reasonable choice for an empty list when the source wasn't open and I hence hadn't seen it, and old habits can die hard).


Can you review this more today?  I merged the code late Friday but intentionally didn't push a release because I wanted some more :eyes: on it before doing so.  Today is a holiday in the US, so lets do a release tomorrow.


Actually, my bad. It was me who overlooked the current syntax"


you dont need to check stops or routes here


This will close #466


It's not just verbosity of filling the logs but also that I'm not sure it tells users anything valuable and is instead just for us when trying to debug something.


"project" - в константы


It can be a local variable instead of a class attribute.


Hi Anatoly,

On 01/18/2016 07:20 AM, anatoly techtonik wrote:
> What is "Win32 graphics mode"? Is it GDI or DirectDraw or ... ?

    It's GDI.  Somewhat low-level,  but there really wasn't a need for the
features (and bloat) provided by DirectDraw.

    BTW,  I've pulled in Mark Hessling's changes into my fork,  and will
be creating a pull request.  I don't know enough about Git and/or Github
to know it will simplify matters for you to pull everything at once.

-- Bill


Interesting, my intuition would be to call the inner thing a nested field, not the outer one. For example, in a point `{ value: { x: 5, y: 10 } }` I’d be inclined to call `x` and `y` nested fields.

Can we maybe completely avoid the term and talk just about fields containing arrays and objects? That seems to me like the least confusing option and it would actually match the implementation (where there are no nested fields as a first-class concept).

(Note that informally we also used the term “nested point”, which I find even more misleading. You didn’t use it, which is good.)


This sounds to me like it's up to subclasses to define the exact semantics. Why doesn't ReusableBuilder define `result` and `clear` with appropriate doc comments? If it's not possible to document those methods individually, I think this doc comment should describe the desired semantics (i.e. that calling `clear` directly after `result` allows reuse).


Oh, thanks for reminding! :-D
I will follow the syntax.


コメントを追加


only need to think about pairs size here


The change looks good except for my nitpick above.


@MrPowerGamerBR Okay, I got it.


not necessary implementing it right now

Am 18.01.16 um 16:14 schrieb Raphael Winkelmann:
>
> although this fixes the problem I think we should also validate the
> annotJSON file before permitting the download. Should I open an extra
> issue for this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IPS-LMU/EMU-webApp/commit/7fbcd0cbbb6bba37ad138d8f299897fc074406d7#commitcomment-15509585>.
>


Ah I see; for some reason I needed it for my machine, and relied on someone's advice: https://github.com/jgallagher/rusqlite/issues/57#issuecomment-146754873

Some documentation/guide on ensuring which SQLite is used would be incredibly helpful though :+1: I'm not really well-versed on using pkg config stuff.


無声音判定の無い採点


ok super!


This is neato @lewisnyman :revolving_hearts:


Очень полезный кусок кода )


Did this in e350d1b


This will break community users of the workspace.


I don't think this change is right.


Implemented that way ;-)


Не устал "project" копипастить? :)


Sure, https://github.com/quentin7b/android-location-tracker/issues/10 :)


Why do we need Direct config business service in this project ?


This should be a constant


Change it to more descriptive name .


Why do we need X12 business services in this project ?


The thing is - im using this often as an reference Project and had some laughs at interviews because of this (DOOM Launcher - with Angular Frontend) so i really want to keep the Materialdesign of the App.

But i can mention your Fork in my Readme when you want - maybe we can make some thoughts to "implement" skinning in a way but as you saw it isnt "that" easy because AngularMaterial uses its restricted themeProvider for this.


This is my first repository at Github. I'm not sure what to write yet. Let me explore the community first.


设置服务器端日志拦截器


There should be a test which executes this method


no need of ',' here


зачем каждый раз новый-то пихать?
if (getFragmentManager().findFragmentById(....) != null) {
....
}
не?


Why do we need AuditQuery Log business service in this project ?


:ok:


и почему не supportFragmentManager?


הערה על הקוד


Es war versehentlich eine Ursprungsversion enthalten, die nun vervollständigt wurde.


Same comment as above. Why do we need DirectConfig service endpoint in this project.


Do you think this indentation?
```
if not self._initialized \
   or ((old_selector or self._accordion.current_selector)
      and (old_selector is self._accordion.current_selector)):
```


Same as above comment.


Lol


Will be fixed.


no C++ comments


multi-line comment format is:
```
/*
 * line 1
 * line 2
*/
```


a more common kernel label seems to be 'exit'


Won't the error average out? Rule of large numbers?


Если первым параметром добавишь фрагмент который это дернул - я начну подозревать откуда этот коллбек. А так не понятно )


needs to be resolved before mailing


Hov, det var sgu da mig, der uploadede det


Wouldn't QSsl::SecureProtocols be a better fit?


QSsl::SecureProtocols is also available in Qt 4, so we may just not need this distinction here.
Additionally, you're iterating over ciphers, which I don't think have an effect on the protocol.


Do we want to decrease this to 1024?


Could place constant on the left to avoid assignment errors


Yeah, that looks like the mentioned [xmldom bug](https://github.com/jindw/xmldom/pull/45#issuecomment-15846059). This code should work in "all" browser (the xmlns stuff was a pita to get it work on IE if I remember correctly; there should be some tickets around about that).
Maybe we should workaround the xmldom issue by detecting a nodejs environment and treating that differently until it gets fixed?


initialization is not needed, globals are zero-initialized


Same comment as above, why do we need directconfig endpoint in this project.


`profile` is a magic variable.  I need to look in the docs for the specizliaed subset for argo, to see if it is described there.  (This is a bit like `ctd[["CT"]]`, which is not stored in the file.)


This is used only in one place, so this macro looks like an unnecessary indirection.


It can be added to something like the strftime_composite() test (I was just not liking the fact it isn't reversible (supported by the parser but not the generator) but it is useful none the less.
.


remove


@aomran


"sortcuts"

Wow, I must have been tired.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25704/testReport/) for this commit**


This should probably be `&& @backstab.include?(game_state['weapon_skill'])` again, which means maybe this logic can be shaken out and simplified.


Shouldn't we be checking that a job is complete before removing it? It's certainly possible that we need to wait on completing a job for some reason. We wouldn't want the job removed solely because it has passed some rather invisible time threshold, especially when that time value can change.


From what I've understood from our conversation, one of the main objectives of these notes is that the student does not feel lost and/or helpless.


I had some trouble with getting the distinction `execute_command` and `execute_in_cli`. Apparently it is the case that for one the output is suppressed, while in the other case the command is executed normally. My suggestion is to rename the signals to `execute_command_silent` and `execute_command`.


Be careful with types ending in a capital T, because monad transformers are traditionally named that, so most Haskell programmers will assume it's a monad transformer.


recyclerviewAccounts - ты где нибудь в гуловых пакетах видел заглавные ьуквы в названиях? ;)


Thank you. I missed that. Will be fixed.


@rk-yen sorry to bother again. It worked using cocoapods version 0.38.2. I think @alexandros-s removed the header_mappings_dir setting to comply with newer versions of c-pod, so maybe it will work against 1.0.0, I couldn't test now because I would had to refactor my complex podfile, but I'll try later.
Thanks for your attention.


Have we officially switched over to BLE and dropped WiFi Direct?


constexpr => const this is just a variable affectation.


`undefined` should be added instead of conditionally adding to keep the array sizes in sync, otherwise an instance could easily end up in a bad state.  For instance, if this region is added first:

`addParent('region', 'Some region', '1')`

followed by:

`addParent('region', 'Pennsylvania', '2', 'PA')`

Then internally, the region arrays will look like:

```
region = ['Some Region', 'Pennsylvania']
region_id = ['1', '2']
region_abbr = ['PA']
```

It won't be possible to know which region the abbreviation 'PA' belongs to.


Other than the one comment, looks good.


Can you make a pull request to https://github.com/w3c/web-platform-tests/ with this change? The people who originally wrote this server infrastructure should definitely have a chance to sign off on it.


    if (!success) {
        TGLog(@"Logout from server failed");
        //TODO: improve error handling: e.g. if the backend returns a 404 also remove current user and tread it as success
    }

    [self clearCurrentUserData];

to remove duplication :)


I cant compile it, see logs:
gcc  -g -O2 -Wall -Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wmissing-noreturn -fno-strict-aliasing   -o nfcapd nfcapd-nfcapd.o nfcapd-nf_common.o nfcapd-util.o nfcapd-minilzo.o nfcapd-nffile.o nfcapd-nfx.o nfcapd-nfxstat.o nfcapd-flist.o nfcapd-fts_compat.o nfcapd-nfstatfile.o nfcapd-launch.o nfcapd-nfnet.o nfcapd-collector.o nfcapd-netflow_v1.o nfcapd-netflow_v5_v7.o nfcapd-netflow_v9.o nfcapd-ipfix.o nfcapd-bookkeeper.o nfcapd-expire.o   -lresolv -lbz2
nfcapd-nf_common.o:(.rodata+0x59c): undefined reference to `String_UserID'
nfcapd-nf_common.o:(.rodata+0x5ac): undefined reference to `String_AppID'
collect2: error: ld returned 1 exit status
make[3]: *** [nfcapd] Error 1
make[3]: Leaving directory `/home/test/git/nfdump-69d8c18ca982eaf915112529e48da72dcb9198a4/bin'
make[2]: *** [all] Error 2
make[2]: Leaving directory `/home/test/git/nfdump-69d8c18ca982eaf915112529e48da72dcb9198a4/bin'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/test/git/nfdump-69d8c18ca982eaf915112529e48da72dcb9198a4'
make: *** [all] Error 2

Whats wrong?


Вот тут я уже реально не осилил, что OnAccountsSelectedListener может проапдейтить project. Пришлось лезть в код фрагмента


что это? Зачем оно???


Should be ``:param checkbox_text:``.


:finnadie:


I hope to employed by a company call Nutmeg


Oke, I wanted to make a difference between the display functions of adding a account and the transactions functions.
Can I better named them xxxTransactions then ?


Sounds good to me.


Oh, okay then. :)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1189](http://teamcity.monogame.net/viewLog.html?buildId=13731&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:35:01


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1190](http://teamcity.monogame.net/viewLog.html?buildId=13735&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:05


Maybe rename to "This is custom functionality implemented by the user"?


I think opposite when the `NULL`s are squashed it's harder to count number of parameters when you are trying to found what is this doing.
The best will be to add there comments what is what.
Do you agree?


странное название для интерфейса. BackButtonPressor хотя бы. Или OnBackButtonListener например


`Uncaught TypeError: Promise.defer is not a function(…)` on both extension's background page and any other page.

I use only Opera Developer (which updates automatically), but I have extension Browser Update Watchdog that notify about update and the problem begin with [version 36.0.2106.0](http://www.opera.com/blogs/desktop/2016/01/first-opera-developer-update-year/).
There was no any extension update or something, just browser update.
Few hours before everything was fine, then browser updated and any version of extension (dev or stable) won't work.

I think you searched for similar problems, aren't you? Maybe it's kind of [overwriting](https://github.com/TypeStrong/atom-typescript/issues/709) or [constructor](https://github.com/lloeki/ex-mode/pull/112) issue?
Or maybe even
```
/* exports */
app.Promise = Promise;
```
in `lib/wrapper/chrome/app.js:12`?


One more space. Like this:
```
if not self._initialized \
   or ((old_selector or self._accordion.current_selector)
       and (old_selector is self._accordion.current_selector)):
```


в жаве поля принято с m называть. mContext


Looks good to me.


:+1: :cocktail: :beer:


The name of this internally should be `Secrets`


That would work for me too. Or just add a comment before the call with the parameters replaced with some sane names like ``# parameters: name, g_type, flags, whatever, whatever2,...``


support nil


The Go name of this should be `Secrets`


Sorry for the many edits - im sleepy as hell and english isnt my mother tongue.


@jmcglone My pleasure! Thanks for the great guide and for letting us link out to it. :sparkles:


Раньше показывало весь фрейм, если мункин ставил один гриб. Теперь рисует отдельно фрейм для каждого гриба. И теперь можно отменять каждый спелл, раньше после первой отмены скрывал все фреймы.


@mikemcquaid What should the behaviour now be for ```brew fetch -fs giflib```? Should it download the source only? Cause right now it tries to download bootle and fails. My environment variables.

```
balrog:local n$ env|grep HOMEBREW
HOMEBREW_DEVELOPER=1
HOMEBREW_VERBOSE=1
HOMEBREW_SANDBOX=1
```


Merged


* The Perl 6 module deployment stanza

These are important to keep in mind in the present. If a module has a non-trivial build procedure but no perl6 dependencies it should be entirely possible to build/test/install without any other perl6 tools. Such build procedures would most certainly use a dependency to handle many things, while simplier ones that only fire off a few commands would probably use none; both cases can be served by a stand alone executable script (vs the current Build.pm that is meant to subclass a part of the package manager itself). I generally refer to this as a hook since it can serve this purpose but it doesn't technically have to be one.


* Also, support config directories

For instance zef puts its `config.json` in `resources/`, which means when installing from the $*CWD (before zef has been installed at all) it has access to that file via %?RESOURCES<config.json>. However let us assume that zef has actually already been installed and I do the same `install from $*CWD` to upgrade it; it might be the case that when I run `$*CWD.child('bin/zef') install .` I need it to use a special config that was previously installed instead of the default in $*CWD.child('resources/config.json') (rather than $*REPOS.grep('some repo').prefix.child('resources/config.json'), but it will always load the uninstalled one in the $*CWD/resources through %?RESOURCES. It can easily be argued that such a config should be saved somewhere like $*HOME where we can always figure out the actual path (to see if it exists), but maybe a type of %?RESOURCES that "falls back" to the $*CWD.child('resources') and uses $*REPOS.grep(some installable repo).prefix.child('resources') first (the opposite of what happens now) may prove useful. Then again such things could also be put into the Build phase if a proper hook/phase were defined.


It seems one of rakudos goals is to keep modules installed across upgrades (or to automatically reinstall them). Currently the tests are not installed. This brings up 2 questions: 1) if an installed module no longer supports the new version of rakudo (as declared in its META data) will it be installed anyway? What about its dependencies and rev-deps? 2) A rakudo update may unknowingly break an installed module to no longer pass its own tests or work properly, but without the tests installed there is no way to know this. If the tests were to be installed rakudo still needs a way to run the test harness after an update but before reinstalling modules, which doesn't seem like something it should have to concern itself with. I have no suggested course of action here, just the suggestion of possible problems with either installing or not-installing tests.


Please see 528d9d1fda91b162bc9bd8c082fa092740550e70 I added some tests and it is working(in NodeJS environment). Can you test it in the browser?


Tbh, I can live with it. It's not nice, but users don't care about nice code, but about nice UX. Not being able to rearrange a column for no good reason ... isn't exactly user friendly. As long as it's just this small piece of code in one single location ...


This is ugly.

Here is a slightly better version, but it's still pretty hacky. I really don't like the printTableT function.

    descriptionDeposit <- getLine
    putStrLn " "
    let newStore   = depositToAccountInStore amountToDeposit descriptionDeposit accountToDeposit accts
        newAcctBal = accountBalance $ newStore ! accountToWitdraw
    printBox (printTableT "Made Deposits" [(descriptionDeposit,amountToDeposit, newAcctBal)])
    putStrLn " "
    return newStore


Эти все DB-классы напрашиваются быть immutable а все сеттеры в билдер унести.


Don't include an interface if the count is 0.


Oh, not a problem at all if you want to use it as a reference, I completely understand! I've looked at your site, and you've done some impressive things, kudos.

I am happy to just maintain my own fork, and it's all up to you if you wanted to mention/link to it in your README. Thanks for the kind words about my design too, I actually came across the ThemeProvider right at the end of my work, but I am still happy how things turned out. Not sure if it's worth talking about theme support, but I would certainly be up for discussing down the road if the launcher became quite popular in the DOOM scene!

Not a worry at all about the edits, either!


Shouldn't this be `2 * padding`?


There are still problems with this.  The script is creating images major.minor for dev versions, such as 1.14 for version 1.14.0-dev.7.0.

And when it creates a new version with the same tag as a previous version, the image creation (or tagging?) stage fails with a message that the tag already exists, and you should use -f.
Here is an example:
Successfully built 9364988c8fac
Error response from daemon: Conflict: Tag 1.14.0-dev.7.1 is already set to image 9364988c8faca6c78b18aa40e322260c13072048302be75b1cec69c5aabc9de7, if you want to replace it, please use -f option

Until these are fixed, I will use the version from before the two merges, for releasing Dart on docker.


Not necessarily. Using finite precision. A + (B-A) != B, specially if B and A are similar, and the error might average to  zero or not depending on architecture and compile options.


Not related with this PR, but the explicit dep with Neutron and Nova suggest that that Manilla needs to be, in the barclamp view, after Nova (now is after Cinder)


I agree with you.


I meant that the actor-based API is going to be deprecated in the Reactive Kafka library. We are going to expose only the graph-based DSL.
Also, I overlooked that there are some commits ahead :) I'll take a look. Thumbs up for your team to get convinced to Scala, I was in your shoes once in one of my previous jobs and I know how it feels.


@backstab is not an array, so it will never be != Offhand Weapon (line 656) and == Offhand Weapon


also, this trailing comma will cause this to not validate as json


I think it would be nice having a test case for this?


+1, but maybe we extend this PR with another commit to fix the Manilla order in the view


I'll try to take a look at it by the end of the week; thanks for the information about the templates!


I believe this commit is closely related to this bug that is still present as of today on CM11: https://jira.cyanogenmod.org/browse/CYAN-5582 .
Can anybody make a test build with this commit onto the CM11 source branch, or directly merge it?
I have already checked and the baseline is identical.

CM11 for i9300 is almost perfect and while CM13 is under development, it will be quite a lot of time until it's ready for prime time.
Squashing this long standing bug could mean a lot for us that use this phone as their main device.

Thanks in advance


Moved to https://github.com/ps-tuebingen/grading-statistics.


我把BeautifulSoup库放在项目的根目录中了，运行前不需要额外安装BeautifulSoup库，这一行可以删掉：）


Another thing - it would be best if you could keep this as a Pull Request. Then I would be notified on new commits and we could discuss the  update set as a whole, then merge.


If that works then is perfect. But I think you can also load the Asserts module (http://codeception.com/docs/modules/Asserts#.Vp0J61MrLzI) via Config .yml like in this example:

class_name: UnitTester
modules:
    enabled:
        - Asserts
        - AcceptanceHelper

Not sure, not tested.


oke,

This is the only way I found to make the output in a tabular form with the name centered above it.
If you know a better way , Im open for suggestions.


Just curious as to why you are deprecating the actor-based DSL?


why no QList or QVector?


按照Python规范，这里应该分成两行书写


:laughing:


Added text and background styling to body tag.


Ok i will code the /banner create {delay} code now!


This is the problem with having everything in the one file. Leave it as it is for now until you've pulled things into different files, then you have more freedon with naming.

I'm not sure which ones you mean by transaction and which ones you mean by display.


Yep, I also think that I should add tests for all my last PR's..


No, because the only place `getBoundsZoom` is called with the `padding` parameter, in `_getBoundsCenterZoom`, the padding is the top-left padding *plus* the bottom-right padding.


Also, should this not be an `r => r.content.get('id')`? Since `r` is an instance of `ResolvedRecord` right?:

{
  isFulfilled: true,
  content: DS.Record{id: 'xyzy})
  index: 200,


OK, I will merge this branch into master and submit a pull request.


Naming is not particularly easy :) When I'm writing programs I choose my names very carefully, because they can quite often make your program easier or worse to undrestand.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1191](http://teamcity.monogame.net/viewLog.html?buildId=13739&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


probably the same with the webdriver module http://codeception.com/docs/modules/WebDriver#.Vp0KalMrLzI. Actually you have a function called $I->executeInSelenium. See: http://codeception.com/docs/modules/WebDriver#executeInSelenium


Ok I will code the /banner create {delay} code now!


nice! I like very much to be able to pass the Template :+1:


lowerCamelCase


Variables should have access modifiers


I found a bigger error due to this though. It uses `diag_tickTime` and sets `GVAR(lastTickTime)` after all other PFHs, so the time will actually run faster the more PFHs are added.


For now, I couldn't get success with gcc 5.2.


Excellent!


use "composer install" instead of "composer update"


We always have a repository id because Download on Demand elements belongs to them...
The only thing to do more in the Rails way will be using PUT instead of POST for updating and change it in the routes.


PrintItem, PrintList, printTable, accountRow, leftRight are used for displaying the input of doCreateAccounts
PrintItemT, PrintListT, printTableT, accountRowT, leftRightT are used for displaying the input of doDepositAccounts and doWithDrawlAccounts


For plan, I'll fix this problem which is listed up in #5.


As Konrad from Akka team mentioned, the actor-level API is the first one, exposed with akka-streams 1.0. It works on the lowest levels of abstraction. Since then, the Graph DSL has become the official "integration API" and Akka team recommends it. One of the main reasons is that it has some performance enhancements underneath (for example, to allow running all in a single thread when possible). From my benchmarks, the graph-based reactive kafka stream is faster and I want it to become the default one.


Recommit my favorit color


Hi,
Just out of curiosity how is OPENMVG_STD_UNORDERED_MAP defined/triggered ? I can't find how it's done.
As you are now using "using", you need C++11 support and as far as I understand  unordered_map is part of this new standard. Shouldn't we use unordered_map by default now ?  Is the OPENMVG_STD_UNORDERED_MAP used only for testing ?

Best,

C


Thank your for the compliments :]

I will look deeper into the themeProvider from AngularMaterial itself and look for a solution. I think skinning as a Feature would be a bad idea - but never thought that someone actually will do it.


Looks ok.


Game logic should be in a separate class from input output method, this way the output/input can easily be changed to another method such as using a GUI.


Shouldn't it be `__class__.__name__`, not `.name`?


Appreciate the pull request. But please can you only submit the files that are changed. This has a load of Umbraco dll's and other non needed DLL's.


I'll take this as a style guide for future commits


We do use this in a production application. With that said, I don't have an extensive collection of RTF files to test with. I simply wrap the import of RTF files in a try-catch-block and inform the user when importing failed for whatever reason.

There is no documentation other than the JavaDoc that's included with the source. There is also no JAR file from my version of the code. You'd have to copy/clone it into your source.


I did some quick naming so I could look if I get this working.
I can name printItemT  printTransaction and printListT  PrintTransactionList.
printTableT will be printTransactionsTable
AccountRowT will be TransactionRow
leftRightT will be leftRightTransactions


If statements should have curly brackets


Не стоит использовать croak в асинхроне


Re commit


Darn.  I can't figure out how to kill it - I've followed the steps at https://help.github.com/articles/remove-sensitive-data/, but it's still linkable.


Hm, why is this needed?


This is not the right thing to do. We'll have a different way of doing initial data in the future. Focus on the predictor tests, that can all be done in a unit test.


어얼 가덜 열심히 하네 :clap:


@prudhomm I am not sure that these changes are OK (also the similar ones in other modules). I mean we only enter here if if the user explicitely activated the module or it was activated by default, meaning that it is required.
Shouldn't we crash CMake at those points ? As something that the user wants was not found. He/she will be surprised that something that he/she enabled was not actually used.


thank for the finding! commit 2fd8588da51a0e43684852dcdabe327aca684a9f should fix it


Could place constant on the left to avoid assignment errors if you accidentally only use one = sign


Merged here.


This named route doesn't look right. I think trying to stick to the standard rails naming conventions as much as we can would be best.

I'd expect to see the following:

```
GET     /request/:id/followups/new        FollowupsController#new
POST    /request/:id/followups/preview    FollowupsController#preview
POST    /request/:id/followups            FollowupsController#create


^ Please remove the whitespaces around the `[`s. That's what's failing the CI test.


Looks ok.


Nit: no newline after <p>


Seems reasonable to me then.


I thought I had finish breaking down Skyrim saves, but looks like I am way behind inthe extra. This will need some work before properly displaying REFR in TES5View


Et en plus j'ai refait une coquille dans un commit qui parle de coquilles #coquilleCeption


This worked perfectly locally, what specifically was incorrect with it? I dislike the idea of adding an extra 3 required arguments for every request, including a GET. if you're doing a GET, you shouldn't need to specify a request_content.


I think you need a `bundle install` above this line.


Ups, you're absolutely right - this should be part of it (if kreadconfig and kreadconfig were not found, it simply disabled it


С этим полем что-то сильно не так. Не должно его тут быть. Каждый раз просле записи/чтения через parcek оно теряется, загружается видимо заново по необходимости, потом OutOfMemory ловим. Вот тут все написано как надо делать: http://developer.android.com/intl/ru/training/displaying-bitmaps/cache-bitmap.html


`implements Serializable`? (+ `toString` and `hashCode` methods and `serialVersionUID`)


Now is ok.
I think that is way to short URL (without .htaccess and short.php). In
the next week the makings of the first tests.

On 1/18/2016 3:42 AM, NorHei wrote:
> Please test a gain, as i am not at home and can not test from here.
>
> Am 18.01.2016 um 07:37 schrieb Krzysztof Winnicki:
> > |public function field_exists($table_name, $field_name) { $sql="SHOW
> > COLUMNS FROM `$table_name` LIKE '$field_name'"; $result =
> > $this->query($sql); $num = $result->numRows(); $exists = ($num > 0);
> > if($exists) return true; else return false; } |
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/WBCE/WebsiteBaker_CommunityEdition/commit/b38b465f2728c3e67f2abd5776c1d3f6ab140ec0#commitcomment-15499486>.
> >
>
> --
> Norbert Heimsath
> Mergelstraße 25
> 45478 Mülheim an der Ruhr
> Tel: 0208 55792
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WBCE/WebsiteBaker_CommunityEdition/commit/b38b465f2728c3e67f2abd5776c1d3f6ab140ec0#commitcomment-15502493>.
>


- Eric Bickerdyke (firework2117@gmail.com)